Commit 188793f0 authored by Jakub Kicinski's avatar Jakub Kicinski
Browse files

eth: hns3: migrate to new RXFH callbacks 



Migrate to new callbacks added by commit 9bb00786 ("net: ethtool:
add dedicated callbacks for getting and setting rxfh fields").

Reviewed-by: default avatarJijie <Shao&lt;shaojijie@huawei.com>
Link: https://patch.msgid.link/20250618203823.1336156-10-kuba@kernel.org


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 943c0ce3

drivers/net/ethernet/hisilicon/hns3/hnae3.h

+2 −2
Original line number Diff line number Diff line
@@ -690,9 +690,9 @@ struct hnae3_ae_ops { 
	int (*set_rss)(struct hnae3_handle *handle, const u32 *indir,

		       const u8 *key, const u8 hfunc);

	int (*set_rss_tuple)(struct hnae3_handle *handle,

			     struct ethtool_rxnfc *cmd);

			     const struct ethtool_rxfh_fields *cmd);

	int (*get_rss_tuple)(struct hnae3_handle *handle,

			     struct ethtool_rxnfc *cmd);

			     struct ethtool_rxfh_fields *cmd);



	int (*get_tc_size)(struct hnae3_handle *handle);

drivers/net/ethernet/hisilicon/hns3/hns3_common/hclge_comm_rss.c

+3 −3
Original line number Diff line number Diff line
@@ -151,7 +151,7 @@ EXPORT_SYMBOL_GPL(hclge_comm_set_rss_hash_key); 
int hclge_comm_set_rss_tuple(struct hnae3_ae_dev *ae_dev,

			     struct hclge_comm_hw *hw,

			     struct hclge_comm_rss_cfg *rss_cfg,

			     struct ethtool_rxnfc *nfc)

			     const struct ethtool_rxfh_fields *nfc)

{

	struct hclge_comm_rss_input_tuple_cmd *req;

	struct hclge_desc desc;
@@ -422,7 +422,7 @@ int hclge_comm_set_rss_algo_key(struct hclge_comm_hw *hw, const u8 hfunc, 
}

EXPORT_SYMBOL_GPL(hclge_comm_set_rss_algo_key);



static u8 hclge_comm_get_rss_hash_bits(struct ethtool_rxnfc *nfc)

static u8 hclge_comm_get_rss_hash_bits(const struct ethtool_rxfh_fields *nfc)

{

	u8 hash_sets = nfc->data & RXH_L4_B_0_1 ? HCLGE_COMM_S_PORT_BIT : 0;
@@ -448,7 +448,7 @@ static u8 hclge_comm_get_rss_hash_bits(struct ethtool_rxnfc *nfc) 
}



int hclge_comm_init_rss_tuple_cmd(struct hclge_comm_rss_cfg *rss_cfg,

				  struct ethtool_rxnfc *nfc,

				  const struct ethtool_rxfh_fields *nfc,

				  struct hnae3_ae_dev *ae_dev,

				  struct hclge_comm_rss_input_tuple_cmd *req)

{

drivers/net/ethernet/hisilicon/hns3/hns3_common/hclge_comm_rss.h

+2 −2
Original line number Diff line number Diff line
@@ -108,7 +108,7 @@ void hclge_comm_get_rss_indir_tbl(struct hclge_comm_rss_cfg *rss_cfg, 
int hclge_comm_set_rss_algo_key(struct hclge_comm_hw *hw, const u8 hfunc,

				const u8 *key);

int hclge_comm_init_rss_tuple_cmd(struct hclge_comm_rss_cfg *rss_cfg,

				  struct ethtool_rxnfc *nfc,

				  const struct ethtool_rxfh_fields *nfc,

				  struct hnae3_ae_dev *ae_dev,

				  struct hclge_comm_rss_input_tuple_cmd *req);

u64 hclge_comm_convert_rss_tuple(u8 tuple_sets);
@@ -129,5 +129,5 @@ int hclge_comm_set_rss_hash_key(struct hclge_comm_rss_cfg *rss_cfg, 
int hclge_comm_set_rss_tuple(struct hnae3_ae_dev *ae_dev,

			     struct hclge_comm_hw *hw,

			     struct hclge_comm_rss_cfg *rss_cfg,

			     struct ethtool_rxnfc *nfc);

			     const struct ethtool_rxfh_fields *nfc);

#endif

drivers/net/ethernet/hisilicon/hns3/hns3_ethtool.c

+25 −8
Original line number Diff line number Diff line
@@ -978,6 +978,16 @@ static int hns3_set_rss(struct net_device *netdev, 
					rxfh->hfunc);

}



static int hns3_get_rxfh_fields(struct net_device *netdev,

				struct ethtool_rxfh_fields *cmd)

{

	struct hnae3_handle *h = hns3_get_handle(netdev);



	if (h->ae_algo->ops->get_rss_tuple)

		return h->ae_algo->ops->get_rss_tuple(h, cmd);

	return -EOPNOTSUPP;

}



static int hns3_get_rxnfc(struct net_device *netdev,

			  struct ethtool_rxnfc *cmd,

			  u32 *rule_locs)
@@ -988,10 +998,6 @@ static int hns3_get_rxnfc(struct net_device *netdev, 
	case ETHTOOL_GRXRINGS:

		cmd->data = h->kinfo.num_tqps;

		return 0;

	case ETHTOOL_GRXFH:

		if (h->ae_algo->ops->get_rss_tuple)

			return h->ae_algo->ops->get_rss_tuple(h, cmd);

		return -EOPNOTSUPP;

	case ETHTOOL_GRXCLSRLCNT:

		if (h->ae_algo->ops->get_fd_rule_cnt)

			return h->ae_algo->ops->get_fd_rule_cnt(h, cmd);
@@ -1275,15 +1281,22 @@ static int hns3_set_ringparam(struct net_device *ndev, 
	return ret;

}



static int hns3_set_rxnfc(struct net_device *netdev, struct ethtool_rxnfc *cmd)

static int hns3_set_rxfh_fields(struct net_device *netdev,

				const struct ethtool_rxfh_fields *cmd,

				struct netlink_ext_ack *extack)

{

	struct hnae3_handle *h = hns3_get_handle(netdev);



	switch (cmd->cmd) {

	case ETHTOOL_SRXFH:

	if (h->ae_algo->ops->set_rss_tuple)

		return h->ae_algo->ops->set_rss_tuple(h, cmd);

	return -EOPNOTSUPP;

}



static int hns3_set_rxnfc(struct net_device *netdev, struct ethtool_rxnfc *cmd)

{

	struct hnae3_handle *h = hns3_get_handle(netdev);



	switch (cmd->cmd) {

	case ETHTOOL_SRXCLSRLINS:

		if (h->ae_algo->ops->add_fd_entry)

			return h->ae_algo->ops->add_fd_entry(h, cmd);
@@ -2105,6 +2118,8 @@ static const struct ethtool_ops hns3vf_ethtool_ops = { 
	.get_rxfh_indir_size = hns3_get_rss_indir_size,

	.get_rxfh = hns3_get_rss,

	.set_rxfh = hns3_set_rss,

	.get_rxfh_fields = hns3_get_rxfh_fields,

	.set_rxfh_fields = hns3_set_rxfh_fields,

	.get_link_ksettings = hns3_get_link_ksettings,

	.get_channels = hns3_get_channels,

	.set_channels = hns3_set_channels,
@@ -2142,6 +2157,8 @@ static const struct ethtool_ops hns3_ethtool_ops = { 
	.get_rxfh_indir_size = hns3_get_rss_indir_size,

	.get_rxfh = hns3_get_rss,

	.set_rxfh = hns3_set_rss,

	.get_rxfh_fields = hns3_get_rxfh_fields,

	.set_rxfh_fields = hns3_set_rxfh_fields,

	.get_link_ksettings = hns3_get_link_ksettings,

	.set_link_ksettings = hns3_set_link_ksettings,

	.nway_reset = hns3_nway_reset,

drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c

+2 −2
Original line number Diff line number Diff line
@@ -4872,7 +4872,7 @@ static int hclge_set_rss(struct hnae3_handle *handle, const u32 *indir, 
}



static int hclge_set_rss_tuple(struct hnae3_handle *handle,

			       struct ethtool_rxnfc *nfc)

			       const struct ethtool_rxfh_fields *nfc)

{

	struct hclge_vport *vport = hclge_get_vport(handle);

	struct hclge_dev *hdev = vport->back;
@@ -4890,7 +4890,7 @@ static int hclge_set_rss_tuple(struct hnae3_handle *handle, 
}



static int hclge_get_rss_tuple(struct hnae3_handle *handle,

			       struct ethtool_rxnfc *nfc)

			       struct ethtool_rxfh_fields *nfc)

{

	struct hclge_vport *vport = hclge_get_vport(handle);

	u8 tuple_sets;