net: hns3: PF support get unicast MAC address space assigned by firmware
authorGuangbin Huang <huangguangbin2@huawei.com>
Tue, 14 Sep 2021 12:11:16 +0000 (20:11 +0800)
committerDavid S. Miller <davem@davemloft.net>
Tue, 14 Sep 2021 13:32:29 +0000 (14:32 +0100)
Currently, there are two ways for PF to set the unicast MAC address space
size: specified by config parameters in firmware or set to default value.

That's mean if the config parameters in firmware is zero, driver will
divide the whole unicast MAC address space equally to 8 PFs. However, in
this case, the unicast MAC address space will be wasted a lot when the
hardware actually has less then 8 PFs. And in the other hand, if one PF has
much more VFs than other PFs, then each function of this PF will has much
less address space than other PFs.

In order to ameliorate the above two situations, introduce the third way
of unicast MAC address space assignment: firmware divides the whole unicast
MAC address space equally to functions of all PFs, and calculates the space
size of each PF according to its function number. PF queries the space size
by the querying device specification command when in initialization
process.

The third way assignment is lower priority than specified by config
parameters, only if the config parameters is zero can be used, and if
firmware does not support the third way assignment, then driver still
divides the whole unicast MAC address space equally to 8 PFs.

Signed-off-by: Guangbin Huang <huangguangbin2@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/hisilicon/hns3/hnae3.h
drivers/net/ethernet/hisilicon/hns3/hns3_debugfs.c
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.h
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c

index 546a605303848aa1db81452707b4e89ab7b52bac..b100b63b13db94805bfd2c900887f8907b98f115 100644 (file)
@@ -341,6 +341,7 @@ struct hnae3_dev_specs {
        u8 max_non_tso_bd_num; /* max BD number of one non-TSO packet */
        u16 max_frm_size;
        u16 max_qset_num;
+       u16 umv_size;
 };
 
 struct hnae3_client_ops {
index 2b66c59f5eafd2c5df28213cd2193156cbe1baaf..d297f22f5af9155273a4030cda91fe3addba67b3 100644 (file)
@@ -924,6 +924,8 @@ hns3_dbg_dev_specs(struct hnae3_handle *h, char *buf, int len, int *pos)
                          dev_specs->max_tm_rate);
        *pos += scnprintf(buf + *pos, len - *pos, "MAX QSET number: %u\n",
                          dev_specs->max_qset_num);
+       *pos += scnprintf(buf + *pos, len - *pos, "umv size: %u\n",
+                         dev_specs->umv_size);
 }
 
 static int hns3_dbg_dev_info(struct hnae3_handle *h, char *buf, int len)
index 33244472e0d0e7e820e96a49e0cef8a81b2d0db8..cfbb7c51b0cb3a9b11872338f2dba521e508c2cc 100644 (file)
@@ -1188,7 +1188,9 @@ struct hclge_dev_specs_1_cmd {
        __le16 max_frm_size;
        __le16 max_qset_num;
        __le16 max_int_gl;
-       u8 rsv1[18];
+       u8 rsv0[2];
+       __le16 umv_size;
+       u8 rsv1[14];
 };
 
 /* mac speed type defined in firmware command */
index e55ba2e511b1a3eafb73437d53ae613d919f7be0..82deba80adfb2008bb876eb2220b7f3fc933f2dc 100644 (file)
@@ -1342,8 +1342,6 @@ static void hclge_parse_cfg(struct hclge_cfg *cfg, struct hclge_desc *desc)
        cfg->umv_space = hnae3_get_field(__le32_to_cpu(req->param[1]),
                                         HCLGE_CFG_UMV_TBL_SPACE_M,
                                         HCLGE_CFG_UMV_TBL_SPACE_S);
-       if (!cfg->umv_space)
-               cfg->umv_space = HCLGE_DEFAULT_UMV_SPACE_PER_PF;
 
        cfg->pf_rss_size_max = hnae3_get_field(__le32_to_cpu(req->param[2]),
                                               HCLGE_CFG_PF_RSS_SIZE_M,
@@ -1419,6 +1417,7 @@ static void hclge_set_default_dev_specs(struct hclge_dev *hdev)
        ae_dev->dev_specs.max_int_gl = HCLGE_DEF_MAX_INT_GL;
        ae_dev->dev_specs.max_frm_size = HCLGE_MAC_MAX_FRAME;
        ae_dev->dev_specs.max_qset_num = HCLGE_MAX_QSET_NUM;
+       ae_dev->dev_specs.umv_size = HCLGE_DEFAULT_UMV_SPACE_PER_PF;
 }
 
 static void hclge_parse_dev_specs(struct hclge_dev *hdev,
@@ -1440,6 +1439,7 @@ static void hclge_parse_dev_specs(struct hclge_dev *hdev,
        ae_dev->dev_specs.max_qset_num = le16_to_cpu(req1->max_qset_num);
        ae_dev->dev_specs.max_int_gl = le16_to_cpu(req1->max_int_gl);
        ae_dev->dev_specs.max_frm_size = le16_to_cpu(req1->max_frm_size);
+       ae_dev->dev_specs.umv_size = le16_to_cpu(req1->umv_size);
 }
 
 static void hclge_check_dev_specs(struct hclge_dev *hdev)
@@ -1460,6 +1460,8 @@ static void hclge_check_dev_specs(struct hclge_dev *hdev)
                dev_specs->max_int_gl = HCLGE_DEF_MAX_INT_GL;
        if (!dev_specs->max_frm_size)
                dev_specs->max_frm_size = HCLGE_MAC_MAX_FRAME;
+       if (!dev_specs->umv_size)
+               dev_specs->umv_size = HCLGE_DEFAULT_UMV_SPACE_PER_PF;
 }
 
 static int hclge_query_dev_specs(struct hclge_dev *hdev)
@@ -1548,7 +1550,10 @@ static int hclge_configure(struct hclge_dev *hdev)
        hdev->tm_info.num_pg = 1;
        hdev->tc_max = cfg.tc_num;
        hdev->tm_info.hw_pfc_map = 0;
-       hdev->wanted_umv_size = cfg.umv_space;
+       if (cfg.umv_space)
+               hdev->wanted_umv_size = cfg.umv_space;
+       else
+               hdev->wanted_umv_size = hdev->ae_dev->dev_specs.umv_size;
        hdev->tx_spare_buf_size = cfg.tx_spare_buf_size;
        hdev->gro_en = true;
        if (cfg.vlan_fliter_cap == HCLGE_VLAN_FLTR_CAN_MDF)