]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/blobdiff - drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
net: hns3: refactor the hclge_get/set_rss function
[mirror_ubuntu-bionic-kernel.git] / drivers / net / ethernet / hisilicon / hns3 / hns3pf / hclge_main.c
index 80049223519f1b24232abf54a823aee4363dbd5a..131234dfb05ff565dd545d024a17589fe9c66e54 100644 (file)
@@ -17,6 +17,7 @@
 #include <linux/netdevice.h>
 #include <linux/pci.h>
 #include <linux/platform_device.h>
+#include <linux/if_vlan.h>
 #include <net/rtnetlink.h>
 #include "hclge_cmd.h"
 #include "hclge_dcb.h"
 static int hclge_set_mta_filter_mode(struct hclge_dev *hdev,
                                     enum hclge_mta_dmac_sel_type mta_mac_sel,
                                     bool enable);
+static int hclge_set_mtu(struct hnae3_handle *handle, int new_mtu);
 static int hclge_init_vlan_config(struct hclge_dev *hdev);
 static int hclge_reset_ae_dev(struct hnae3_ae_dev *ae_dev);
+static int hclge_update_led_status(struct hclge_dev *hdev);
 
 static struct hnae3_ae_algo ae_algo;
 
@@ -52,6 +55,8 @@ static const struct pci_device_id ae_algo_pci_tbl[] = {
        {0, }
 };
 
+MODULE_DEVICE_TABLE(pci, ae_algo_pci_tbl);
+
 static const char hns3_nic_test_strs[][ETH_GSTRING_LEN] = {
        "Mac    Loopback test",
        "Serdes Loopback test",
@@ -293,8 +298,24 @@ static const struct hclge_comm_stats_str g_mac_stats_string[] = {
                HCLGE_MAC_STATS_FIELD_OFF(mac_tx_512_1023_oct_pkt_num)},
        {"mac_tx_1024_1518_oct_pkt_num",
                HCLGE_MAC_STATS_FIELD_OFF(mac_tx_1024_1518_oct_pkt_num)},
-       {"mac_tx_1519_max_oct_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_1519_max_oct_pkt_num)},
+       {"mac_tx_1519_2047_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_1519_2047_oct_pkt_num)},
+       {"mac_tx_2048_4095_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_2048_4095_oct_pkt_num)},
+       {"mac_tx_4096_8191_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_4096_8191_oct_pkt_num)},
+       {"mac_tx_8192_12287_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_8192_12287_oct_pkt_num)},
+       {"mac_tx_8192_9216_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_8192_9216_oct_pkt_num)},
+       {"mac_tx_9217_12287_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_9217_12287_oct_pkt_num)},
+       {"mac_tx_12288_16383_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_12288_16383_oct_pkt_num)},
+       {"mac_tx_1519_max_good_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_1519_max_good_oct_pkt_num)},
+       {"mac_tx_1519_max_bad_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_1519_max_bad_oct_pkt_num)},
        {"mac_rx_total_pkt_num",
                HCLGE_MAC_STATS_FIELD_OFF(mac_rx_total_pkt_num)},
        {"mac_rx_total_oct_num",
@@ -329,8 +350,24 @@ static const struct hclge_comm_stats_str g_mac_stats_string[] = {
                HCLGE_MAC_STATS_FIELD_OFF(mac_rx_512_1023_oct_pkt_num)},
        {"mac_rx_1024_1518_oct_pkt_num",
                HCLGE_MAC_STATS_FIELD_OFF(mac_rx_1024_1518_oct_pkt_num)},
-       {"mac_rx_1519_max_oct_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_1519_max_oct_pkt_num)},
+       {"mac_rx_1519_2047_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_1519_2047_oct_pkt_num)},
+       {"mac_rx_2048_4095_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_2048_4095_oct_pkt_num)},
+       {"mac_rx_4096_8191_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_4096_8191_oct_pkt_num)},
+       {"mac_rx_8192_12287_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_8192_12287_oct_pkt_num)},
+       {"mac_rx_8192_9216_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_8192_9216_oct_pkt_num)},
+       {"mac_rx_9217_12287_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_9217_12287_oct_pkt_num)},
+       {"mac_rx_12288_16383_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_12288_16383_oct_pkt_num)},
+       {"mac_rx_1519_max_good_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_1519_max_good_oct_pkt_num)},
+       {"mac_rx_1519_max_bad_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_1519_max_bad_oct_pkt_num)},
 
        {"mac_tx_fragment_pkt_num",
                HCLGE_MAC_STATS_FIELD_OFF(mac_tx_fragment_pkt_num)},
@@ -358,6 +395,16 @@ static const struct hclge_comm_stats_str g_mac_stats_string[] = {
                HCLGE_MAC_STATS_FIELD_OFF(mac_rx_send_app_bad_pkt_num)}
 };
 
+static const struct hclge_mac_mgr_tbl_entry_cmd hclge_mgr_table[] = {
+       {
+               .flags = HCLGE_MAC_MGR_MASK_VLAN_B,
+               .ethter_type = cpu_to_le16(HCLGE_MAC_ETHERTYPE_LLDP),
+               .mac_addr_hi32 = cpu_to_le32(htonl(0x0180C200)),
+               .mac_addr_lo16 = cpu_to_le16(htons(0x000E)),
+               .i_port_bitmap = 0x1,
+       },
+};
+
 static int hclge_64_bit_update_stats(struct hclge_dev *hdev)
 {
 #define HCLGE_64_BIT_CMD_NUM 5
@@ -461,9 +508,41 @@ static int hclge_32_bit_update_stats(struct hclge_dev *hdev)
        return 0;
 }
 
+static int hclge_mac_get_traffic_stats(struct hclge_dev *hdev)
+{
+       struct hclge_mac_stats *mac_stats = &hdev->hw_stats.mac_stats;
+       struct hclge_desc desc;
+       __le64 *desc_data;
+       int ret;
+
+       /* for fiber port, need to query the total rx/tx packets statstics,
+        * used for data transferring checking.
+        */
+       if (hdev->hw.mac.media_type != HNAE3_MEDIA_TYPE_FIBER)
+               return 0;
+
+       if (test_bit(HCLGE_STATE_STATISTICS_UPDATING, &hdev->state))
+               return 0;
+
+       hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_STATS_MAC_TRAFFIC, true);
+       ret = hclge_cmd_send(&hdev->hw, &desc, 1);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "Get MAC total pkt stats fail, ret = %d\n", ret);
+
+               return ret;
+       }
+
+       desc_data = (__le64 *)(&desc.data[0]);
+       mac_stats->mac_tx_total_pkt_num += le64_to_cpu(*desc_data++);
+       mac_stats->mac_rx_total_pkt_num += le64_to_cpu(*desc_data);
+
+       return 0;
+}
+
 static int hclge_mac_update_stats(struct hclge_dev *hdev)
 {
-#define HCLGE_MAC_CMD_NUM 17
+#define HCLGE_MAC_CMD_NUM 21
 #define HCLGE_RTN_DATA_NUM 4
 
        u64 *data = (u64 *)(&hdev->hw_stats.mac_stats);
@@ -2207,8 +2286,11 @@ static int hclge_set_default_mac_vlan_mask(struct hclge_dev *hdev,
 
 static int hclge_mac_init(struct hclge_dev *hdev)
 {
+       struct hnae3_handle *handle = &hdev->vport[0].nic;
+       struct net_device *netdev = handle->kinfo.netdev;
        struct hclge_mac *mac = &hdev->hw.mac;
        u8 mac_mask[ETH_ALEN] = {0x00, 0x00, 0x00, 0x00, 0x00, 0x00};
+       int mtu;
        int ret;
 
        ret = hclge_cfg_mac_speed_dup(hdev, hdev->hw.mac.speed, HCLGE_MAC_FULL);
@@ -2242,11 +2324,25 @@ static int hclge_mac_init(struct hclge_dev *hdev)
        }
 
        ret = hclge_set_default_mac_vlan_mask(hdev, true, mac_mask);
-       if (ret)
+       if (ret) {
                dev_err(&hdev->pdev->dev,
                        "set default mac_vlan_mask fail ret=%d\n", ret);
+               return ret;
+       }
 
-       return ret;
+       if (netdev)
+               mtu = netdev->mtu;
+       else
+               mtu = ETH_DATA_LEN;
+
+       ret = hclge_set_mtu(handle, mtu);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "set mtu failed ret=%d\n", ret);
+               return ret;
+       }
+
+       return 0;
 }
 
 static void hclge_mbx_task_schedule(struct hclge_dev *hdev)
@@ -2785,13 +2881,20 @@ static void hclge_service_task(struct work_struct *work)
        struct hclge_dev *hdev =
                container_of(work, struct hclge_dev, service_task);
 
+       /* The total rx/tx packets statstics are wanted to be updated
+        * per second. Both hclge_update_stats_for_all() and
+        * hclge_mac_get_traffic_stats() can do it.
+        */
        if (hdev->hw_stats.stats_timer >= HCLGE_STATS_TIMER_INTERVAL) {
                hclge_update_stats_for_all(hdev);
                hdev->hw_stats.stats_timer = 0;
+       } else {
+               hclge_mac_get_traffic_stats(hdev);
        }
 
        hclge_update_speed_duplex(hdev);
        hclge_update_link_status(hdev);
+       hclge_update_led_status(hdev);
        hclge_service_complete(hdev);
 }
 
@@ -2878,31 +2981,6 @@ static u32 hclge_get_rss_indir_size(struct hnae3_handle *handle)
        return HCLGE_RSS_IND_TBL_SIZE;
 }
 
-static int hclge_get_rss_algo(struct hclge_dev *hdev)
-{
-       struct hclge_rss_config_cmd *req;
-       struct hclge_desc desc;
-       int rss_hash_algo;
-       int ret;
-
-       hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_RSS_GENERIC_CONFIG, true);
-
-       ret = hclge_cmd_send(&hdev->hw, &desc, 1);
-       if (ret) {
-               dev_err(&hdev->pdev->dev,
-                       "Get link status error, status =%d\n", ret);
-               return ret;
-       }
-
-       req = (struct hclge_rss_config_cmd *)desc.data;
-       rss_hash_algo = (req->hash_config & HCLGE_RSS_HASH_ALGO_MASK);
-
-       if (rss_hash_algo == HCLGE_RSS_HASH_ALGO_TOEPLITZ)
-               return ETH_RSS_HASH_TOP;
-
-       return -EINVAL;
-}
-
 static int hclge_set_rss_algo_key(struct hclge_dev *hdev,
                                  const u8 hfunc, const u8 *key)
 {
@@ -2941,7 +3019,7 @@ static int hclge_set_rss_algo_key(struct hclge_dev *hdev,
        return 0;
 }
 
-static int hclge_set_rss_indir_table(struct hclge_dev *hdev, const u32 *indir)
+static int hclge_set_rss_indir_table(struct hclge_dev *hdev, const u8 *indir)
 {
        struct hclge_rss_indirection_table_cmd *req;
        struct hclge_desc desc;
@@ -3037,12 +3115,11 @@ static int hclge_get_rss(struct hnae3_handle *handle, u32 *indir,
                         u8 *key, u8 *hfunc)
 {
        struct hclge_vport *vport = hclge_get_vport(handle);
-       struct hclge_dev *hdev = vport->back;
        int i;
 
        /* Get hash algorithm */
        if (hfunc)
-               *hfunc = hclge_get_rss_algo(hdev);
+               *hfunc = vport->rss_algo;
 
        /* Get the RSS Key required by the user */
        if (key)
@@ -3066,8 +3143,6 @@ static int hclge_set_rss(struct hnae3_handle *handle, const u32 *indir,
 
        /* Set the RSS Hash Key if specififed by the user */
        if (key) {
-               /* Update the shadow RSS key with user specified qids */
-               memcpy(vport->rss_hash_key, key, HCLGE_RSS_KEY_SIZE);
 
                if (hfunc == ETH_RSS_HASH_TOP ||
                    hfunc == ETH_RSS_HASH_NO_CHANGE)
@@ -3077,6 +3152,10 @@ static int hclge_set_rss(struct hnae3_handle *handle, const u32 *indir,
                ret = hclge_set_rss_algo_key(hdev, hash_algo, key);
                if (ret)
                        return ret;
+
+               /* Update the shadow RSS key with user specified qids */
+               memcpy(vport->rss_hash_key, key, HCLGE_RSS_KEY_SIZE);
+               vport->rss_algo = hash_algo;
        }
 
        /* Update the shadow RSS table with user specified qids */
@@ -3084,8 +3163,7 @@ static int hclge_set_rss(struct hnae3_handle *handle, const u32 *indir,
                vport->rss_indirection_tbl[i] = indir[i];
 
        /* Update the hardware */
-       ret = hclge_set_rss_indir_table(hdev, indir);
-       return ret;
+       return hclge_set_rss_indir_table(hdev, vport->rss_indirection_tbl);
 }
 
 static u8 hclge_get_rss_hash_bits(struct ethtool_rxnfc *nfc)
@@ -3358,6 +3436,11 @@ int hclge_bind_ring_with_vector(struct hclge_vport *vport,
                               hnae_get_bit(node->flag, HNAE3_RING_TYPE_B));
                hnae_set_field(tqp_type_and_id, HCLGE_TQP_ID_M,
                               HCLGE_TQP_ID_S, node->tqp_index);
+               hnae_set_field(tqp_type_and_id, HCLGE_INT_GL_IDX_M,
+                              HCLGE_INT_GL_IDX_S,
+                              hnae_get_field(node->int_gl_idx,
+                                             HNAE3_RING_GL_IDX_M,
+                                             HNAE3_RING_GL_IDX_S));
                req->tqp_type_and_id[i] = cpu_to_le16(tqp_type_and_id);
                if (++i >= HCLGE_VECTOR_ELEMENTS_PER_CMD) {
                        req->int_cause_num = HCLGE_VECTOR_ELEMENTS_PER_CMD;
@@ -3611,20 +3694,11 @@ static int hclge_ae_start(struct hnae3_handle *handle)
 {
        struct hclge_vport *vport = hclge_get_vport(handle);
        struct hclge_dev *hdev = vport->back;
-       int i, queue_id, ret;
+       int i, ret;
 
-       for (i = 0; i < vport->alloc_tqps; i++) {
-               /* todo clear interrupt */
-               /* ring enable */
-               queue_id = hclge_get_queue_id(handle->kinfo.tqp[i]);
-               if (queue_id < 0) {
-                       dev_warn(&hdev->pdev->dev,
-                                "Get invalid queue id, ignore it\n");
-                       continue;
-               }
+       for (i = 0; i < vport->alloc_tqps; i++)
+               hclge_tqp_enable(hdev, i, 0, true);
 
-               hclge_tqp_enable(hdev, queue_id, 0, true);
-       }
        /* mac enable */
        hclge_cfg_mac_mode(hdev, true);
        clear_bit(HCLGE_STATE_DOWN, &hdev->state);
@@ -3644,19 +3718,11 @@ static void hclge_ae_stop(struct hnae3_handle *handle)
 {
        struct hclge_vport *vport = hclge_get_vport(handle);
        struct hclge_dev *hdev = vport->back;
-       int i, queue_id;
+       int i;
 
-       for (i = 0; i < vport->alloc_tqps; i++) {
-               /* Ring disable */
-               queue_id = hclge_get_queue_id(handle->kinfo.tqp[i]);
-               if (queue_id < 0) {
-                       dev_warn(&hdev->pdev->dev,
-                                "Get invalid queue id, ignore it\n");
-                       continue;
-               }
+       for (i = 0; i < vport->alloc_tqps; i++)
+               hclge_tqp_enable(hdev, i, 0, false);
 
-               hclge_tqp_enable(hdev, queue_id, 0, false);
-       }
        /* Mac disable */
        hclge_cfg_mac_mode(hdev, false);
 
@@ -4193,6 +4259,91 @@ int hclge_rm_mc_addr_common(struct hclge_vport *vport,
        return status;
 }
 
+static int hclge_get_mac_ethertype_cmd_status(struct hclge_dev *hdev,
+                                             u16 cmdq_resp, u8 resp_code)
+{
+#define HCLGE_ETHERTYPE_SUCCESS_ADD            0
+#define HCLGE_ETHERTYPE_ALREADY_ADD            1
+#define HCLGE_ETHERTYPE_MGR_TBL_OVERFLOW       2
+#define HCLGE_ETHERTYPE_KEY_CONFLICT           3
+
+       int return_status;
+
+       if (cmdq_resp) {
+               dev_err(&hdev->pdev->dev,
+                       "cmdq execute failed for get_mac_ethertype_cmd_status, status=%d.\n",
+                       cmdq_resp);
+               return -EIO;
+       }
+
+       switch (resp_code) {
+       case HCLGE_ETHERTYPE_SUCCESS_ADD:
+       case HCLGE_ETHERTYPE_ALREADY_ADD:
+               return_status = 0;
+               break;
+       case HCLGE_ETHERTYPE_MGR_TBL_OVERFLOW:
+               dev_err(&hdev->pdev->dev,
+                       "add mac ethertype failed for manager table overflow.\n");
+               return_status = -EIO;
+               break;
+       case HCLGE_ETHERTYPE_KEY_CONFLICT:
+               dev_err(&hdev->pdev->dev,
+                       "add mac ethertype failed for key conflict.\n");
+               return_status = -EIO;
+               break;
+       default:
+               dev_err(&hdev->pdev->dev,
+                       "add mac ethertype failed for undefined, code=%d.\n",
+                       resp_code);
+               return_status = -EIO;
+       }
+
+       return return_status;
+}
+
+static int hclge_add_mgr_tbl(struct hclge_dev *hdev,
+                            const struct hclge_mac_mgr_tbl_entry_cmd *req)
+{
+       struct hclge_desc desc;
+       u8 resp_code;
+       u16 retval;
+       int ret;
+
+       hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_MAC_ETHTYPE_ADD, false);
+       memcpy(desc.data, req, sizeof(struct hclge_mac_mgr_tbl_entry_cmd));
+
+       ret = hclge_cmd_send(&hdev->hw, &desc, 1);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "add mac ethertype failed for cmd_send, ret =%d.\n",
+                       ret);
+               return ret;
+       }
+
+       resp_code = (le32_to_cpu(desc.data[0]) >> 8) & 0xff;
+       retval = le16_to_cpu(desc.retval);
+
+       return hclge_get_mac_ethertype_cmd_status(hdev, retval, resp_code);
+}
+
+static int init_mgr_tbl(struct hclge_dev *hdev)
+{
+       int ret;
+       int i;
+
+       for (i = 0; i < ARRAY_SIZE(hclge_mgr_table); i++) {
+               ret = hclge_add_mgr_tbl(hdev, &hclge_mgr_table[i]);
+               if (ret) {
+                       dev_err(&hdev->pdev->dev,
+                               "add mac ethertype failed, ret =%d.\n",
+                               ret);
+                       return ret;
+               }
+       }
+
+       return 0;
+}
+
 static void hclge_get_mac_addr(struct hnae3_handle *handle, u8 *p)
 {
        struct hclge_vport *vport = hclge_get_vport(handle);
@@ -4206,6 +4357,7 @@ static int hclge_set_mac_addr(struct hnae3_handle *handle, void *p)
        const unsigned char *new_addr = (const unsigned char *)p;
        struct hclge_vport *vport = hclge_get_vport(handle);
        struct hclge_dev *hdev = vport->back;
+       int ret;
 
        /* mac addr check */
        if (is_zero_ether_addr(new_addr) ||
@@ -4217,14 +4369,39 @@ static int hclge_set_mac_addr(struct hnae3_handle *handle, void *p)
                return -EINVAL;
        }
 
-       hclge_rm_uc_addr(handle, hdev->hw.mac.mac_addr);
+       ret = hclge_rm_uc_addr(handle, hdev->hw.mac.mac_addr);
+       if (ret)
+               dev_warn(&hdev->pdev->dev,
+                        "remove old uc mac address fail, ret =%d.\n",
+                        ret);
 
-       if (!hclge_add_uc_addr(handle, new_addr)) {
-               ether_addr_copy(hdev->hw.mac.mac_addr, new_addr);
-               return 0;
+       ret = hclge_add_uc_addr(handle, new_addr);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "add uc mac address fail, ret =%d.\n",
+                       ret);
+
+               ret = hclge_add_uc_addr(handle, hdev->hw.mac.mac_addr);
+               if (ret) {
+                       dev_err(&hdev->pdev->dev,
+                               "restore uc mac address fail, ret =%d.\n",
+                               ret);
+               }
+
+               return -EIO;
        }
 
-       return -EIO;
+       ret = hclge_mac_pause_addr_cfg(hdev, new_addr);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "configure mac pause address fail, ret =%d.\n",
+                       ret);
+               return -EIO;
+       }
+
+       ether_addr_copy(hdev->hw.mac.mac_addr, new_addr);
+
+       return 0;
 }
 
 static int hclge_set_vlan_filter_ctrl(struct hclge_dev *hdev, u8 vlan_type,
@@ -4560,16 +4737,21 @@ static int hclge_set_mtu(struct hnae3_handle *handle, int new_mtu)
        struct hclge_config_max_frm_size_cmd *req;
        struct hclge_dev *hdev = vport->back;
        struct hclge_desc desc;
+       int max_frm_size;
        int ret;
 
-       if ((new_mtu < HCLGE_MAC_MIN_MTU) || (new_mtu > HCLGE_MAC_MAX_MTU))
+       max_frm_size = new_mtu + ETH_HLEN + ETH_FCS_LEN + VLAN_HLEN;
+
+       if (max_frm_size < HCLGE_MAC_MIN_FRAME ||
+           max_frm_size > HCLGE_MAC_MAX_FRAME)
                return -EINVAL;
 
-       hdev->mps = new_mtu;
+       max_frm_size = max(max_frm_size, HCLGE_MAC_DEFAULT_FRAME);
+
        hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_CONFIG_MAX_FRM_SIZE, false);
 
        req = (struct hclge_config_max_frm_size_cmd *)desc.data;
-       req->max_frm_size = cpu_to_le16(new_mtu);
+       req->max_frm_size = cpu_to_le16(max_frm_size);
 
        ret = hclge_cmd_send(&hdev->hw, &desc, 1);
        if (ret) {
@@ -4577,6 +4759,8 @@ static int hclge_set_mtu(struct hnae3_handle *handle, int new_mtu)
                return ret;
        }
 
+       hdev->mps = max_frm_size;
+
        return 0;
 }
 
@@ -4624,21 +4808,36 @@ static int hclge_get_reset_status(struct hclge_dev *hdev, u16 queue_id)
        return hnae_get_bit(req->ready_to_reset, HCLGE_TQP_RESET_B);
 }
 
+static u16 hclge_covert_handle_qid_global(struct hnae3_handle *handle,
+                                         u16 queue_id)
+{
+       struct hnae3_queue *queue;
+       struct hclge_tqp *tqp;
+
+       queue = handle->kinfo.tqp[queue_id];
+       tqp = container_of(queue, struct hclge_tqp, q);
+
+       return tqp->index;
+}
+
 void hclge_reset_tqp(struct hnae3_handle *handle, u16 queue_id)
 {
        struct hclge_vport *vport = hclge_get_vport(handle);
        struct hclge_dev *hdev = vport->back;
        int reset_try_times = 0;
        int reset_status;
+       u16 queue_gid;
        int ret;
 
+       queue_gid = hclge_covert_handle_qid_global(handle, queue_id);
+
        ret = hclge_tqp_enable(hdev, queue_id, 0, false);
        if (ret) {
                dev_warn(&hdev->pdev->dev, "Disable tqp fail, ret = %d\n", ret);
                return;
        }
 
-       ret = hclge_send_reset_tqp_cmd(hdev, queue_id, true);
+       ret = hclge_send_reset_tqp_cmd(hdev, queue_gid, true);
        if (ret) {
                dev_warn(&hdev->pdev->dev,
                         "Send reset tqp cmd fail, ret = %d\n", ret);
@@ -4649,7 +4848,7 @@ void hclge_reset_tqp(struct hnae3_handle *handle, u16 queue_id)
        while (reset_try_times++ < HCLGE_TQP_RESET_TRY_TIMES) {
                /* Wait for tqp hw reset */
                msleep(20);
-               reset_status = hclge_get_reset_status(hdev, queue_id);
+               reset_status = hclge_get_reset_status(hdev, queue_gid);
                if (reset_status)
                        break;
        }
@@ -4659,7 +4858,7 @@ void hclge_reset_tqp(struct hnae3_handle *handle, u16 queue_id)
                return;
        }
 
-       ret = hclge_send_reset_tqp_cmd(hdev, queue_id, false);
+       ret = hclge_send_reset_tqp_cmd(hdev, queue_gid, false);
        if (ret) {
                dev_warn(&hdev->pdev->dev,
                         "Deassert the soft reset fail, ret = %d\n", ret);
@@ -4707,22 +4906,19 @@ static void hclge_set_flowctrl_adv(struct hclge_dev *hdev, u32 rx_en, u32 tx_en)
 
 static int hclge_cfg_pauseparam(struct hclge_dev *hdev, u32 rx_en, u32 tx_en)
 {
-       enum hclge_fc_mode fc_mode;
        int ret;
 
        if (rx_en && tx_en)
-               fc_mode = HCLGE_FC_FULL;
+               hdev->fc_mode_last_time = HCLGE_FC_FULL;
        else if (rx_en && !tx_en)
-               fc_mode = HCLGE_FC_RX_PAUSE;
+               hdev->fc_mode_last_time = HCLGE_FC_RX_PAUSE;
        else if (!rx_en && tx_en)
-               fc_mode = HCLGE_FC_TX_PAUSE;
+               hdev->fc_mode_last_time = HCLGE_FC_TX_PAUSE;
        else
-               fc_mode = HCLGE_FC_NONE;
+               hdev->fc_mode_last_time = HCLGE_FC_NONE;
 
-       if (hdev->tm_info.fc_mode == HCLGE_FC_PFC) {
-               hdev->fc_mode_last_time = fc_mode;
+       if (hdev->tm_info.fc_mode == HCLGE_FC_PFC)
                return 0;
-       }
 
        ret = hclge_mac_pause_en_cfg(hdev, tx_en, rx_en);
        if (ret) {
@@ -4731,7 +4927,7 @@ static int hclge_cfg_pauseparam(struct hclge_dev *hdev, u32 rx_en, u32 tx_en)
                return ret;
        }
 
-       hdev->tm_info.fc_mode = fc_mode;
+       hdev->tm_info.fc_mode = hdev->fc_mode_last_time;
 
        return 0;
 }
@@ -5185,6 +5381,12 @@ static int hclge_init_ae_dev(struct hnae3_ae_dev *ae_dev)
                return ret;
        }
 
+       ret = init_mgr_tbl(hdev);
+       if (ret) {
+               dev_err(&pdev->dev, "manager table init fail, ret =%d\n", ret);
+               return ret;
+       }
+
        hclge_dcb_ops_set(hdev);
 
        timer_setup(&hdev->service_timer, hclge_service_timer, 0);
@@ -5458,6 +5660,318 @@ static int hclge_set_channels(struct hnae3_handle *handle, u32 new_tqps_num)
        return ret;
 }
 
+static int hclge_get_regs_num(struct hclge_dev *hdev, u32 *regs_num_32_bit,
+                             u32 *regs_num_64_bit)
+{
+       struct hclge_desc desc;
+       u32 total_num;
+       int ret;
+
+       hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_QUERY_REG_NUM, true);
+       ret = hclge_cmd_send(&hdev->hw, &desc, 1);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "Query register number cmd failed, ret = %d.\n", ret);
+               return ret;
+       }
+
+       *regs_num_32_bit = le32_to_cpu(desc.data[0]);
+       *regs_num_64_bit = le32_to_cpu(desc.data[1]);
+
+       total_num = *regs_num_32_bit + *regs_num_64_bit;
+       if (!total_num)
+               return -EINVAL;
+
+       return 0;
+}
+
+static int hclge_get_32_bit_regs(struct hclge_dev *hdev, u32 regs_num,
+                                void *data)
+{
+#define HCLGE_32_BIT_REG_RTN_DATANUM 8
+
+       struct hclge_desc *desc;
+       u32 *reg_val = data;
+       __le32 *desc_data;
+       int cmd_num;
+       int i, k, n;
+       int ret;
+
+       if (regs_num == 0)
+               return 0;
+
+       cmd_num = DIV_ROUND_UP(regs_num + 2, HCLGE_32_BIT_REG_RTN_DATANUM);
+       desc = kcalloc(cmd_num, sizeof(struct hclge_desc), GFP_KERNEL);
+       if (!desc)
+               return -ENOMEM;
+
+       hclge_cmd_setup_basic_desc(&desc[0], HCLGE_OPC_QUERY_32_BIT_REG, true);
+       ret = hclge_cmd_send(&hdev->hw, desc, cmd_num);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "Query 32 bit register cmd failed, ret = %d.\n", ret);
+               kfree(desc);
+               return ret;
+       }
+
+       for (i = 0; i < cmd_num; i++) {
+               if (i == 0) {
+                       desc_data = (__le32 *)(&desc[i].data[0]);
+                       n = HCLGE_32_BIT_REG_RTN_DATANUM - 2;
+               } else {
+                       desc_data = (__le32 *)(&desc[i]);
+                       n = HCLGE_32_BIT_REG_RTN_DATANUM;
+               }
+               for (k = 0; k < n; k++) {
+                       *reg_val++ = le32_to_cpu(*desc_data++);
+
+                       regs_num--;
+                       if (!regs_num)
+                               break;
+               }
+       }
+
+       kfree(desc);
+       return 0;
+}
+
+static int hclge_get_64_bit_regs(struct hclge_dev *hdev, u32 regs_num,
+                                void *data)
+{
+#define HCLGE_64_BIT_REG_RTN_DATANUM 4
+
+       struct hclge_desc *desc;
+       u64 *reg_val = data;
+       __le64 *desc_data;
+       int cmd_num;
+       int i, k, n;
+       int ret;
+
+       if (regs_num == 0)
+               return 0;
+
+       cmd_num = DIV_ROUND_UP(regs_num + 1, HCLGE_64_BIT_REG_RTN_DATANUM);
+       desc = kcalloc(cmd_num, sizeof(struct hclge_desc), GFP_KERNEL);
+       if (!desc)
+               return -ENOMEM;
+
+       hclge_cmd_setup_basic_desc(&desc[0], HCLGE_OPC_QUERY_64_BIT_REG, true);
+       ret = hclge_cmd_send(&hdev->hw, desc, cmd_num);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "Query 64 bit register cmd failed, ret = %d.\n", ret);
+               kfree(desc);
+               return ret;
+       }
+
+       for (i = 0; i < cmd_num; i++) {
+               if (i == 0) {
+                       desc_data = (__le64 *)(&desc[i].data[0]);
+                       n = HCLGE_64_BIT_REG_RTN_DATANUM - 1;
+               } else {
+                       desc_data = (__le64 *)(&desc[i]);
+                       n = HCLGE_64_BIT_REG_RTN_DATANUM;
+               }
+               for (k = 0; k < n; k++) {
+                       *reg_val++ = le64_to_cpu(*desc_data++);
+
+                       regs_num--;
+                       if (!regs_num)
+                               break;
+               }
+       }
+
+       kfree(desc);
+       return 0;
+}
+
+static int hclge_get_regs_len(struct hnae3_handle *handle)
+{
+       struct hclge_vport *vport = hclge_get_vport(handle);
+       struct hclge_dev *hdev = vport->back;
+       u32 regs_num_32_bit, regs_num_64_bit;
+       int ret;
+
+       ret = hclge_get_regs_num(hdev, &regs_num_32_bit, &regs_num_64_bit);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "Get register number failed, ret = %d.\n", ret);
+               return -EOPNOTSUPP;
+       }
+
+       return regs_num_32_bit * sizeof(u32) + regs_num_64_bit * sizeof(u64);
+}
+
+static void hclge_get_regs(struct hnae3_handle *handle, u32 *version,
+                          void *data)
+{
+       struct hclge_vport *vport = hclge_get_vport(handle);
+       struct hclge_dev *hdev = vport->back;
+       u32 regs_num_32_bit, regs_num_64_bit;
+       int ret;
+
+       *version = hdev->fw_version;
+
+       ret = hclge_get_regs_num(hdev, &regs_num_32_bit, &regs_num_64_bit);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "Get register number failed, ret = %d.\n", ret);
+               return;
+       }
+
+       ret = hclge_get_32_bit_regs(hdev, regs_num_32_bit, data);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "Get 32 bit register failed, ret = %d.\n", ret);
+               return;
+       }
+
+       data = (u32 *)data + regs_num_32_bit;
+       ret = hclge_get_64_bit_regs(hdev, regs_num_64_bit,
+                                   data);
+       if (ret)
+               dev_err(&hdev->pdev->dev,
+                       "Get 64 bit register failed, ret = %d.\n", ret);
+}
+
+static int hclge_set_led_status_sfp(struct hclge_dev *hdev, u8 speed_led_status,
+                                   u8 act_led_status, u8 link_led_status,
+                                   u8 locate_led_status)
+{
+       struct hclge_set_led_state_cmd *req;
+       struct hclge_desc desc;
+       int ret;
+
+       hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_LED_STATUS_CFG, false);
+
+       req = (struct hclge_set_led_state_cmd *)desc.data;
+       hnae_set_field(req->port_speed_led_config, HCLGE_LED_PORT_SPEED_STATE_M,
+                      HCLGE_LED_PORT_SPEED_STATE_S, speed_led_status);
+       hnae_set_field(req->link_led_config, HCLGE_LED_ACTIVITY_STATE_M,
+                      HCLGE_LED_ACTIVITY_STATE_S, act_led_status);
+       hnae_set_field(req->activity_led_config, HCLGE_LED_LINK_STATE_M,
+                      HCLGE_LED_LINK_STATE_S, link_led_status);
+       hnae_set_field(req->locate_led_config, HCLGE_LED_LOCATE_STATE_M,
+                      HCLGE_LED_LOCATE_STATE_S, locate_led_status);
+
+       ret = hclge_cmd_send(&hdev->hw, &desc, 1);
+       if (ret)
+               dev_err(&hdev->pdev->dev,
+                       "Send set led state cmd error, ret =%d\n", ret);
+
+       return ret;
+}
+
+enum hclge_led_status {
+       HCLGE_LED_OFF,
+       HCLGE_LED_ON,
+       HCLGE_LED_NO_CHANGE = 0xFF,
+};
+
+static int hclge_set_led_id(struct hnae3_handle *handle,
+                           enum ethtool_phys_id_state status)
+{
+#define BLINK_FREQUENCY                2
+       struct hclge_vport *vport = hclge_get_vport(handle);
+       struct hclge_dev *hdev = vport->back;
+       struct phy_device *phydev = hdev->hw.mac.phydev;
+       int ret = 0;
+
+       if (phydev || hdev->hw.mac.media_type != HNAE3_MEDIA_TYPE_FIBER)
+               return -EOPNOTSUPP;
+
+       switch (status) {
+       case ETHTOOL_ID_ACTIVE:
+               ret = hclge_set_led_status_sfp(hdev,
+                                              HCLGE_LED_NO_CHANGE,
+                                              HCLGE_LED_NO_CHANGE,
+                                              HCLGE_LED_NO_CHANGE,
+                                              HCLGE_LED_ON);
+               break;
+       case ETHTOOL_ID_INACTIVE:
+               ret = hclge_set_led_status_sfp(hdev,
+                                              HCLGE_LED_NO_CHANGE,
+                                              HCLGE_LED_NO_CHANGE,
+                                              HCLGE_LED_NO_CHANGE,
+                                              HCLGE_LED_OFF);
+               break;
+       default:
+               ret = -EINVAL;
+               break;
+       }
+
+       return ret;
+}
+
+enum hclge_led_port_speed {
+       HCLGE_SPEED_LED_FOR_1G,
+       HCLGE_SPEED_LED_FOR_10G,
+       HCLGE_SPEED_LED_FOR_25G,
+       HCLGE_SPEED_LED_FOR_40G,
+       HCLGE_SPEED_LED_FOR_50G,
+       HCLGE_SPEED_LED_FOR_100G,
+};
+
+static u8 hclge_led_get_speed_status(u32 speed)
+{
+       u8 speed_led;
+
+       switch (speed) {
+       case HCLGE_MAC_SPEED_1G:
+               speed_led = HCLGE_SPEED_LED_FOR_1G;
+               break;
+       case HCLGE_MAC_SPEED_10G:
+               speed_led = HCLGE_SPEED_LED_FOR_10G;
+               break;
+       case HCLGE_MAC_SPEED_25G:
+               speed_led = HCLGE_SPEED_LED_FOR_25G;
+               break;
+       case HCLGE_MAC_SPEED_40G:
+               speed_led = HCLGE_SPEED_LED_FOR_40G;
+               break;
+       case HCLGE_MAC_SPEED_50G:
+               speed_led = HCLGE_SPEED_LED_FOR_50G;
+               break;
+       case HCLGE_MAC_SPEED_100G:
+               speed_led = HCLGE_SPEED_LED_FOR_100G;
+               break;
+       default:
+               speed_led = HCLGE_LED_NO_CHANGE;
+       }
+
+       return speed_led;
+}
+
+static int hclge_update_led_status(struct hclge_dev *hdev)
+{
+       u8 port_speed_status, link_status, activity_status;
+       u64 rx_pkts, tx_pkts;
+
+       if (hdev->hw.mac.media_type != HNAE3_MEDIA_TYPE_FIBER)
+               return 0;
+
+       port_speed_status = hclge_led_get_speed_status(hdev->hw.mac.speed);
+
+       rx_pkts = hdev->hw_stats.mac_stats.mac_rx_total_pkt_num;
+       tx_pkts = hdev->hw_stats.mac_stats.mac_tx_total_pkt_num;
+       if (rx_pkts != hdev->rx_pkts_for_led ||
+           tx_pkts != hdev->tx_pkts_for_led)
+               activity_status = HCLGE_LED_ON;
+       else
+               activity_status = HCLGE_LED_OFF;
+       hdev->rx_pkts_for_led = rx_pkts;
+       hdev->tx_pkts_for_led = tx_pkts;
+
+       if (hdev->hw.mac.link)
+               link_status = HCLGE_LED_ON;
+       else
+               link_status = HCLGE_LED_OFF;
+
+       return hclge_set_led_status_sfp(hdev, port_speed_status,
+                                       activity_status, link_status,
+                                       HCLGE_LED_NO_CHANGE);
+}
+
 static const struct hnae3_ae_ops hclge_ops = {
        .init_ae_dev = hclge_init_ae_dev,
        .uninit_ae_dev = hclge_uninit_ae_dev,
@@ -5509,6 +6023,9 @@ static const struct hnae3_ae_ops hclge_ops = {
        .set_channels = hclge_set_channels,
        .get_channels = hclge_get_channels,
        .get_flowctrl_adv = hclge_get_flowctrl_adv,
+       .get_regs_len = hclge_get_regs_len,
+       .get_regs = hclge_get_regs,
+       .set_led_id = hclge_set_led_id,
 };
 
 static struct hnae3_ae_algo ae_algo = {