net: hns3: fix a -Wformat-nonliteral compile warning
[linux-2.6-microblaze.git] / drivers / net / ethernet / hisilicon / hns3 / hns3pf / hclge_main.c
index dd8405d..b2faebd 100644 (file)
 #define HCLGE_STATS_READ(p, offset) (*((u64 *)((u8 *)(p) + (offset))))
 #define HCLGE_MAC_STATS_FIELD_OFF(f) (offsetof(struct hclge_mac_stats, f))
 
-#define HCLGE_BUF_SIZE_UNIT    256
+#define HCLGE_BUF_SIZE_UNIT    256U
 #define HCLGE_BUF_MUL_BY       2
 #define HCLGE_BUF_DIV_BY       2
+#define NEED_RESERVE_TC_NUM    2
+#define BUF_MAX_PERCENT                100
+#define BUF_RESERVE_PERCENT    90
+
+#define HCLGE_RESET_MAX_FAIL_CNT       5
 
 static int hclge_set_mac_mtu(struct hclge_dev *hdev, int new_mps);
 static int hclge_init_vlan_config(struct hclge_dev *hdev);
+static void hclge_sync_vlan_filter(struct hclge_dev *hdev);
 static int hclge_reset_ae_dev(struct hnae3_ae_dev *ae_dev);
 static bool hclge_get_hw_reset_stat(struct hnae3_handle *handle);
 static int hclge_set_umv_space(struct hclge_dev *hdev, u16 space_size,
                               u16 *allocated_size, bool is_alloc);
 static void hclge_rfs_filter_expire(struct hclge_dev *hdev);
 static void hclge_clear_arfs_rules(struct hnae3_handle *handle);
+static enum hnae3_reset_type hclge_get_reset_level(struct hnae3_ae_dev *ae_dev,
+                                                  unsigned long *addr);
 
 static struct hnae3_ae_algo ae_algo;
 
@@ -441,8 +449,7 @@ static int hclge_tqps_update_stats(struct hnae3_handle *handle)
                queue = handle->kinfo.tqp[i];
                tqp = container_of(queue, struct hclge_tqp, q);
                /* command : HCLGE_OPC_QUERY_IGU_STAT */
-               hclge_cmd_setup_basic_desc(&desc[0],
-                                          HCLGE_OPC_QUERY_RX_STATUS,
+               hclge_cmd_setup_basic_desc(&desc[0], HCLGE_OPC_QUERY_RX_STATUS,
                                           true);
 
                desc[0].data[0] = cpu_to_le32((tqp->index & 0x1ff));
@@ -450,7 +457,7 @@ static int hclge_tqps_update_stats(struct hnae3_handle *handle)
                if (ret) {
                        dev_err(&hdev->pdev->dev,
                                "Query tqp stat fail, status = %d,queue = %d\n",
-                               ret,    i);
+                               ret, i);
                        return ret;
                }
                tqp->tqp_stats.rcb_rx_ring_pktnum_rcd +=
@@ -504,6 +511,7 @@ static int hclge_tqps_get_sset_count(struct hnae3_handle *handle, int stringset)
 {
        struct hnae3_knic_private_info *kinfo = &handle->kinfo;
 
+       /* each tqp has TX & RX two queues */
        return kinfo->num_tqps * (2);
 }
 
@@ -532,7 +540,7 @@ static u8 *hclge_tqps_get_strings(struct hnae3_handle *handle, u8 *data)
        return buff;
 }
 
-static u64 *hclge_comm_get_stats(void *comm_stats,
+static u64 *hclge_comm_get_stats(const void *comm_stats,
                                 const struct hclge_comm_stats_str strs[],
                                 int size, u64 *data)
 {
@@ -556,8 +564,7 @@ static u8 *hclge_comm_get_strings(u32 stringset,
                return buff;
 
        for (i = 0; i < size; i++) {
-               snprintf(buff, ETH_GSTRING_LEN,
-                        strs[i].desc);
+               snprintf(buff, ETH_GSTRING_LEN, "%s", strs[i].desc);
                buff = buff + ETH_GSTRING_LEN;
        }
 
@@ -648,8 +655,7 @@ static int hclge_get_sset_count(struct hnae3_handle *handle, int stringset)
        return count;
 }
 
-static void hclge_get_strings(struct hnae3_handle *handle,
-                             u32 stringset,
+static void hclge_get_strings(struct hnae3_handle *handle, u32 stringset,
                              u8 *data)
 {
        u8 *p = (char *)data;
@@ -657,21 +663,17 @@ static void hclge_get_strings(struct hnae3_handle *handle,
 
        if (stringset == ETH_SS_STATS) {
                size = ARRAY_SIZE(g_mac_stats_string);
-               p = hclge_comm_get_strings(stringset,
-                                          g_mac_stats_string,
-                                          size,
-                                          p);
+               p = hclge_comm_get_strings(stringset, g_mac_stats_string,
+                                          size, p);
                p = hclge_tqps_get_strings(handle, p);
        } else if (stringset == ETH_SS_TEST) {
                if (handle->flags & HNAE3_SUPPORT_APP_LOOPBACK) {
-                       memcpy(p,
-                              hns3_nic_test_strs[HNAE3_LOOP_APP],
+                       memcpy(p, hns3_nic_test_strs[HNAE3_LOOP_APP],
                               ETH_GSTRING_LEN);
                        p += ETH_GSTRING_LEN;
                }
                if (handle->flags & HNAE3_SUPPORT_SERDES_SERIAL_LOOPBACK) {
-                       memcpy(p,
-                              hns3_nic_test_strs[HNAE3_LOOP_SERIAL_SERDES],
+                       memcpy(p, hns3_nic_test_strs[HNAE3_LOOP_SERIAL_SERDES],
                               ETH_GSTRING_LEN);
                        p += ETH_GSTRING_LEN;
                }
@@ -682,8 +684,7 @@ static void hclge_get_strings(struct hnae3_handle *handle,
                        p += ETH_GSTRING_LEN;
                }
                if (handle->flags & HNAE3_SUPPORT_PHY_LOOPBACK) {
-                       memcpy(p,
-                              hns3_nic_test_strs[HNAE3_LOOP_PHY],
+                       memcpy(p, hns3_nic_test_strs[HNAE3_LOOP_PHY],
                               ETH_GSTRING_LEN);
                        p += ETH_GSTRING_LEN;
                }
@@ -696,10 +697,8 @@ static void hclge_get_stats(struct hnae3_handle *handle, u64 *data)
        struct hclge_dev *hdev = vport->back;
        u64 *p;
 
-       p = hclge_comm_get_stats(&hdev->hw_stats.mac_stats,
-                                g_mac_stats_string,
-                                ARRAY_SIZE(g_mac_stats_string),
-                                data);
+       p = hclge_comm_get_stats(&hdev->hw_stats.mac_stats, g_mac_stats_string,
+                                ARRAY_SIZE(g_mac_stats_string), data);
        p = hclge_tqps_get_stats(handle, p);
 }
 
@@ -744,9 +743,7 @@ static int hclge_query_function_status(struct hclge_dev *hdev)
                ret = hclge_cmd_send(&hdev->hw, &desc, 1);
                if (ret) {
                        dev_err(&hdev->pdev->dev,
-                               "query function status failed %d.\n",
-                               ret);
-
+                               "query function status failed %d.\n", ret);
                        return ret;
                }
 
@@ -806,7 +803,7 @@ static int hclge_query_pf_resource(struct hclge_dev *hdev)
                /* PF should have NIC vectors and Roce vectors,
                 * NIC vectors are queued before Roce vectors.
                 */
-               hdev->num_msi = hdev->num_roce_msi  +
+               hdev->num_msi = hdev->num_roce_msi +
                                hdev->roce_base_msix_offset;
        } else {
                hdev->num_msi =
@@ -1082,7 +1079,7 @@ static void hclge_parse_cfg(struct hclge_cfg *cfg, struct hclge_desc *desc)
        struct hclge_cfg_param_cmd *req;
        u64 mac_addr_tmp_high;
        u64 mac_addr_tmp;
-       int i;
+       unsigned int i;
 
        req = (struct hclge_cfg_param_cmd *)desc[0].data;
 
@@ -1144,7 +1141,8 @@ static int hclge_get_cfg(struct hclge_dev *hdev, struct hclge_cfg *hcfg)
 {
        struct hclge_desc desc[HCLGE_PF_CFG_DESC_NUM];
        struct hclge_cfg_param_cmd *req;
-       int i, ret;
+       unsigned int i;
+       int ret;
 
        for (i = 0; i < HCLGE_PF_CFG_DESC_NUM; i++) {
                u32 offset = 0;
@@ -1210,7 +1208,8 @@ static void hclge_init_kdump_kernel_config(struct hclge_dev *hdev)
 static int hclge_configure(struct hclge_dev *hdev)
 {
        struct hclge_cfg cfg;
-       int ret, i;
+       unsigned int i;
+       int ret;
 
        ret = hclge_get_cfg(hdev, &cfg);
        if (ret) {
@@ -1273,8 +1272,8 @@ static int hclge_configure(struct hclge_dev *hdev)
        return ret;
 }
 
-static int hclge_config_tso(struct hclge_dev *hdev, int tso_mss_min,
-                           int tso_mss_max)
+static int hclge_config_tso(struct hclge_dev *hdev, unsigned int tso_mss_min,
+                           unsigned int tso_mss_max)
 {
        struct hclge_cfg_tso_status_cmd *req;
        struct hclge_desc desc;
@@ -1586,7 +1585,8 @@ static int hclge_tx_buffer_alloc(struct hclge_dev *hdev,
 
 static u32 hclge_get_tc_num(struct hclge_dev *hdev)
 {
-       int i, cnt = 0;
+       unsigned int i;
+       u32 cnt = 0;
 
        for (i = 0; i < HCLGE_MAX_TC_NUM; i++)
                if (hdev->hw_tc_map & BIT(i))
@@ -1599,7 +1599,8 @@ static int hclge_get_pfc_priv_num(struct hclge_dev *hdev,
                                  struct hclge_pkt_buf_alloc *buf_alloc)
 {
        struct hclge_priv_buf *priv;
-       int i, cnt = 0;
+       unsigned int i;
+       int cnt = 0;
 
        for (i = 0; i < HCLGE_MAX_TC_NUM; i++) {
                priv = &buf_alloc->priv_buf[i];
@@ -1616,7 +1617,8 @@ static int hclge_get_no_pfc_priv_num(struct hclge_dev *hdev,
                                     struct hclge_pkt_buf_alloc *buf_alloc)
 {
        struct hclge_priv_buf *priv;
-       int i, cnt = 0;
+       unsigned int i;
+       int cnt = 0;
 
        for (i = 0; i < HCLGE_MAX_TC_NUM; i++) {
                priv = &buf_alloc->priv_buf[i];
@@ -1694,10 +1696,14 @@ static bool  hclge_is_rx_buf_ok(struct hclge_dev *hdev,
        }
 
        if (hnae3_dev_dcb_supported(hdev)) {
+               hi_thrd = shared_buf - hdev->dv_buf_size;
+
+               if (tc_num <= NEED_RESERVE_TC_NUM)
+                       hi_thrd = hi_thrd * BUF_RESERVE_PERCENT
+                                       / BUF_MAX_PERCENT;
+
                if (tc_num)
-                       hi_thrd = (shared_buf - hdev->dv_buf_size) / tc_num;
-               else
-                       hi_thrd = shared_buf - hdev->dv_buf_size;
+                       hi_thrd = hi_thrd / tc_num;
 
                hi_thrd = max_t(u32, hi_thrd, HCLGE_BUF_MUL_BY * aligned_mps);
                hi_thrd = rounddown(hi_thrd, HCLGE_BUF_SIZE_UNIT);
@@ -1746,7 +1752,7 @@ static bool hclge_rx_buf_calc_all(struct hclge_dev *hdev, bool max,
 {
        u32 rx_all = hdev->pkt_buf_size - hclge_get_tx_buff_alloced(buf_alloc);
        u32 aligned_mps = round_up(hdev->mps, HCLGE_BUF_SIZE_UNIT);
-       int i;
+       unsigned int i;
 
        for (i = 0; i < HCLGE_MAX_TC_NUM; i++) {
                struct hclge_priv_buf *priv = &buf_alloc->priv_buf[i];
@@ -1787,9 +1793,10 @@ static bool hclge_drop_nopfc_buf_till_fit(struct hclge_dev *hdev,
        /* let the last to be cleared first */
        for (i = HCLGE_MAX_TC_NUM - 1; i >= 0; i--) {
                struct hclge_priv_buf *priv = &buf_alloc->priv_buf[i];
+               unsigned int mask = BIT((unsigned int)i);
 
-               if (hdev->hw_tc_map & BIT(i) &&
-                   !(hdev->tm_info.hw_pfc_map & BIT(i))) {
+               if (hdev->hw_tc_map & mask &&
+                   !(hdev->tm_info.hw_pfc_map & mask)) {
                        /* Clear the no pfc TC private buffer */
                        priv->wl.low = 0;
                        priv->wl.high = 0;
@@ -1816,9 +1823,10 @@ static bool hclge_drop_pfc_buf_till_fit(struct hclge_dev *hdev,
        /* let the last to be cleared first */
        for (i = HCLGE_MAX_TC_NUM - 1; i >= 0; i--) {
                struct hclge_priv_buf *priv = &buf_alloc->priv_buf[i];
+               unsigned int mask = BIT((unsigned int)i);
 
-               if (hdev->hw_tc_map & BIT(i) &&
-                   hdev->tm_info.hw_pfc_map & BIT(i)) {
+               if (hdev->hw_tc_map & mask &&
+                   hdev->tm_info.hw_pfc_map & mask) {
                        /* Reduce the number of pfc TC with private buffer */
                        priv->wl.low = 0;
                        priv->enable = 0;
@@ -1835,6 +1843,55 @@ static bool hclge_drop_pfc_buf_till_fit(struct hclge_dev *hdev,
        return hclge_is_rx_buf_ok(hdev, buf_alloc, rx_all);
 }
 
+static int hclge_only_alloc_priv_buff(struct hclge_dev *hdev,
+                                     struct hclge_pkt_buf_alloc *buf_alloc)
+{
+#define COMPENSATE_BUFFER      0x3C00
+#define COMPENSATE_HALF_MPS_NUM        5
+#define PRIV_WL_GAP            0x1800
+
+       u32 rx_priv = hdev->pkt_buf_size - hclge_get_tx_buff_alloced(buf_alloc);
+       u32 tc_num = hclge_get_tc_num(hdev);
+       u32 half_mps = hdev->mps >> 1;
+       u32 min_rx_priv;
+       unsigned int i;
+
+       if (tc_num)
+               rx_priv = rx_priv / tc_num;
+
+       if (tc_num <= NEED_RESERVE_TC_NUM)
+               rx_priv = rx_priv * BUF_RESERVE_PERCENT / BUF_MAX_PERCENT;
+
+       min_rx_priv = hdev->dv_buf_size + COMPENSATE_BUFFER +
+                       COMPENSATE_HALF_MPS_NUM * half_mps;
+       min_rx_priv = round_up(min_rx_priv, HCLGE_BUF_SIZE_UNIT);
+       rx_priv = round_down(rx_priv, HCLGE_BUF_SIZE_UNIT);
+
+       if (rx_priv < min_rx_priv)
+               return false;
+
+       for (i = 0; i < HCLGE_MAX_TC_NUM; i++) {
+               struct hclge_priv_buf *priv = &buf_alloc->priv_buf[i];
+
+               priv->enable = 0;
+               priv->wl.low = 0;
+               priv->wl.high = 0;
+               priv->buf_size = 0;
+
+               if (!(hdev->hw_tc_map & BIT(i)))
+                       continue;
+
+               priv->enable = 1;
+               priv->buf_size = rx_priv;
+               priv->wl.high = rx_priv - hdev->dv_buf_size;
+               priv->wl.low = priv->wl.high - PRIV_WL_GAP;
+       }
+
+       buf_alloc->s_buf.buf_size = 0;
+
+       return true;
+}
+
 /* hclge_rx_buffer_calc: calculate the rx private buffer size for all TCs
  * @hdev: pointer to struct hclge_dev
  * @buf_alloc: pointer to buffer calculation data
@@ -1854,6 +1911,9 @@ static int hclge_rx_buffer_calc(struct hclge_dev *hdev,
                return 0;
        }
 
+       if (hclge_only_alloc_priv_buff(hdev, buf_alloc))
+               return 0;
+
        if (hclge_rx_buf_calc_all(hdev, true, buf_alloc))
                return 0;
 
@@ -2151,7 +2211,6 @@ static int hclge_init_msi(struct hclge_dev *hdev)
 
 static u8 hclge_check_speed_dup(u8 duplex, int speed)
 {
-
        if (!(speed == HCLGE_MAC_SPEED_10M || speed == HCLGE_MAC_SPEED_100M))
                duplex = HCLGE_MAC_FULL;
 
@@ -2169,7 +2228,8 @@ static int hclge_cfg_mac_speed_dup_hw(struct hclge_dev *hdev, int speed,
 
        hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_CONFIG_SPEED_DUP, false);
 
-       hnae3_set_bit(req->speed_dup, HCLGE_CFG_DUPLEX_B, !!duplex);
+       if (duplex)
+               hnae3_set_bit(req->speed_dup, HCLGE_CFG_DUPLEX_B, 1);
 
        switch (speed) {
        case HCLGE_MAC_SPEED_10M:
@@ -2314,6 +2374,17 @@ static int hclge_restart_autoneg(struct hnae3_handle *handle)
        return hclge_notify_client(hdev, HNAE3_UP_CLIENT);
 }
 
+static int hclge_halt_autoneg(struct hnae3_handle *handle, bool halt)
+{
+       struct hclge_vport *vport = hclge_get_vport(handle);
+       struct hclge_dev *hdev = vport->back;
+
+       if (hdev->hw.mac.support_autoneg && hdev->hw.mac.autoneg)
+               return hclge_set_autoneg_en(hdev, !halt);
+
+       return 0;
+}
+
 static int hclge_set_fec_hw(struct hclge_dev *hdev, u32 fec_mode)
 {
        struct hclge_config_fec_cmd *req;
@@ -2387,6 +2458,15 @@ static int hclge_mac_init(struct hclge_dev *hdev)
                return ret;
        }
 
+       if (hdev->hw.mac.support_autoneg) {
+               ret = hclge_set_autoneg_en(hdev, hdev->hw.mac.autoneg);
+               if (ret) {
+                       dev_err(&hdev->pdev->dev,
+                               "Config mac autoneg fail ret=%d\n", ret);
+                       return ret;
+               }
+       }
+
        mac->link = 0;
 
        if (mac->user_fec_mode & BIT(HNAE3_FEC_USER_DEF)) {
@@ -2457,7 +2537,7 @@ static int hclge_get_mac_link_status(struct hclge_dev *hdev)
 
 static int hclge_get_mac_phy_link(struct hclge_dev *hdev)
 {
-       int mac_state;
+       unsigned int mac_state;
        int link_stat;
 
        if (test_bit(HCLGE_STATE_DOWN, &hdev->state))
@@ -2531,7 +2611,7 @@ static void hclge_update_port_capability(struct hclge_mac *mac)
 
 static int hclge_get_sfp_speed(struct hclge_dev *hdev, u32 *speed)
 {
-       struct hclge_sfp_info_cmd *resp = NULL;
+       struct hclge_sfp_info_cmd *resp;
        struct hclge_desc desc;
        int ret;
 
@@ -2752,8 +2832,8 @@ static void hclge_enable_vector(struct hclge_misc_vector *vector, bool enable)
 static irqreturn_t hclge_misc_irq_handle(int irq, void *data)
 {
        struct hclge_dev *hdev = data;
+       u32 clearval = 0;
        u32 event_cause;
-       u32 clearval;
 
        hclge_enable_vector(&hdev->misc_vector, false);
        event_cause = hclge_check_event_cause(hdev, &clearval);
@@ -2859,8 +2939,7 @@ int hclge_notify_client(struct hclge_dev *hdev,
        struct hnae3_client *client = hdev->nic_client;
        u16 i;
 
-       if (!test_bit(HCLGE_STATE_NIC_REGISTERED, &hdev->state) ||
-           !client)
+       if (!test_bit(HCLGE_STATE_NIC_REGISTERED, &hdev->state) || !client)
                return 0;
 
        if (!client->ops->reset_notify)
@@ -2888,8 +2967,7 @@ static int hclge_notify_roce_client(struct hclge_dev *hdev,
        int ret = 0;
        u16 i;
 
-       if (!test_bit(HCLGE_STATE_ROCE_REGISTERED, &hdev->state) ||
-           !client)
+       if (!test_bit(HCLGE_STATE_ROCE_REGISTERED, &hdev->state) || !client)
                return 0;
 
        if (!client->ops->reset_notify)
@@ -3076,10 +3154,11 @@ static void hclge_do_reset(struct hclge_dev *hdev)
        }
 }
 
-static enum hnae3_reset_type hclge_get_reset_level(struct hclge_dev *hdev,
+static enum hnae3_reset_type hclge_get_reset_level(struct hnae3_ae_dev *ae_dev,
                                                   unsigned long *addr)
 {
        enum hnae3_reset_type rst_level = HNAE3_NONE_RESET;
+       struct hclge_dev *hdev = ae_dev->priv;
 
        /* first, resolve any unknown reset type to the known type(s) */
        if (test_bit(HNAE3_UNKNOWN_RESET, addr)) {
@@ -3243,6 +3322,7 @@ static bool hclge_reset_err_handle(struct hclge_dev *hdev, bool is_timeout)
 
                dev_info(&hdev->pdev->dev, "Upgrade reset level\n");
                hclge_clear_reset_cause(hdev);
+               set_bit(HNAE3_GLOBAL_RESET, &hdev->default_reset_request);
                mod_timer(&hdev->reset_timer,
                          jiffies + HCLGE_RESET_INTERVAL);
 
@@ -3271,6 +3351,25 @@ static int hclge_reset_prepare_up(struct hclge_dev *hdev)
        return ret;
 }
 
+static int hclge_reset_stack(struct hclge_dev *hdev)
+{
+       int ret;
+
+       ret = hclge_notify_client(hdev, HNAE3_UNINIT_CLIENT);
+       if (ret)
+               return ret;
+
+       ret = hclge_reset_ae_dev(hdev->ae_dev);
+       if (ret)
+               return ret;
+
+       ret = hclge_notify_client(hdev, HNAE3_INIT_CLIENT);
+       if (ret)
+               return ret;
+
+       return hclge_notify_client(hdev, HNAE3_RESTORE_CLIENT);
+}
+
 static void hclge_reset(struct hclge_dev *hdev)
 {
        struct hnae3_ae_dev *ae_dev = pci_get_drvdata(hdev->pdev);
@@ -3314,19 +3413,8 @@ static void hclge_reset(struct hclge_dev *hdev)
                goto err_reset;
 
        rtnl_lock();
-       ret = hclge_notify_client(hdev, HNAE3_UNINIT_CLIENT);
-       if (ret)
-               goto err_reset_lock;
-
-       ret = hclge_reset_ae_dev(hdev->ae_dev);
-       if (ret)
-               goto err_reset_lock;
 
-       ret = hclge_notify_client(hdev, HNAE3_INIT_CLIENT);
-       if (ret)
-               goto err_reset_lock;
-
-       ret = hclge_notify_client(hdev, HNAE3_RESTORE_CLIENT);
+       ret = hclge_reset_stack(hdev);
        if (ret)
                goto err_reset_lock;
 
@@ -3336,16 +3424,23 @@ static void hclge_reset(struct hclge_dev *hdev)
        if (ret)
                goto err_reset_lock;
 
+       rtnl_unlock();
+
+       ret = hclge_notify_roce_client(hdev, HNAE3_INIT_CLIENT);
+       /* ignore RoCE notify error if it fails HCLGE_RESET_MAX_FAIL_CNT - 1
+        * times
+        */
+       if (ret && hdev->reset_fail_cnt < HCLGE_RESET_MAX_FAIL_CNT - 1)
+               goto err_reset;
+
+       rtnl_lock();
+
        ret = hclge_notify_client(hdev, HNAE3_UP_CLIENT);
        if (ret)
                goto err_reset_lock;
 
        rtnl_unlock();
 
-       ret = hclge_notify_roce_client(hdev, HNAE3_INIT_CLIENT);
-       if (ret)
-               goto err_reset;
-
        ret = hclge_notify_roce_client(hdev, HNAE3_UP_CLIENT);
        if (ret)
                goto err_reset;
@@ -3393,7 +3488,7 @@ static void hclge_reset_event(struct pci_dev *pdev, struct hnae3_handle *handle)
                return;
        else if (hdev->default_reset_request)
                hdev->reset_level =
-                       hclge_get_reset_level(hdev,
+                       hclge_get_reset_level(ae_dev,
                                              &hdev->default_reset_request);
        else if (time_after(jiffies, (hdev->last_reset_time + 4 * 5 * HZ)))
                hdev->reset_level = HNAE3_FUNC_RESET;
@@ -3422,13 +3517,14 @@ static void hclge_reset_timer(struct timer_list *t)
        struct hclge_dev *hdev = from_timer(hdev, t, reset_timer);
 
        dev_info(&hdev->pdev->dev,
-                "triggering global reset in reset timer\n");
-       set_bit(HNAE3_GLOBAL_RESET, &hdev->default_reset_request);
+                "triggering reset in reset timer\n");
        hclge_reset_event(hdev->pdev, NULL);
 }
 
 static void hclge_reset_subtask(struct hclge_dev *hdev)
 {
+       struct hnae3_ae_dev *ae_dev = pci_get_drvdata(hdev->pdev);
+
        /* check if there is any ongoing reset in the hardware. This status can
         * be checked from reset_pending. If there is then, we need to wait for
         * hardware to complete reset.
@@ -3439,12 +3535,12 @@ static void hclge_reset_subtask(struct hclge_dev *hdev)
         *       now.
         */
        hdev->last_reset_time = jiffies;
-       hdev->reset_type = hclge_get_reset_level(hdev, &hdev->reset_pending);
+       hdev->reset_type = hclge_get_reset_level(ae_dev, &hdev->reset_pending);
        if (hdev->reset_type != HNAE3_NONE_RESET)
                hclge_reset(hdev);
 
        /* check if we got any *new* reset requests to be honored */
-       hdev->reset_type = hclge_get_reset_level(hdev, &hdev->reset_request);
+       hdev->reset_type = hclge_get_reset_level(ae_dev, &hdev->reset_request);
        if (hdev->reset_type != HNAE3_NONE_RESET)
                hclge_do_reset(hdev);
 
@@ -3511,6 +3607,7 @@ static void hclge_service_task(struct work_struct *work)
        hclge_update_port_info(hdev);
        hclge_update_link_status(hdev);
        hclge_update_vport_alive(hdev);
+       hclge_sync_vlan_filter(hdev);
        if (hdev->fd_arfs_expire_timer >= HCLGE_FD_ARFS_EXPIRE_TIMER_INTERVAL) {
                hclge_rfs_filter_expire(hdev);
                hdev->fd_arfs_expire_timer = 0;
@@ -3608,29 +3705,28 @@ static int hclge_set_rss_algo_key(struct hclge_dev *hdev,
                                  const u8 hfunc, const u8 *key)
 {
        struct hclge_rss_config_cmd *req;
+       unsigned int key_offset = 0;
        struct hclge_desc desc;
-       int key_offset;
+       int key_counts;
        int key_size;
        int ret;
 
+       key_counts = HCLGE_RSS_KEY_SIZE;
        req = (struct hclge_rss_config_cmd *)desc.data;
 
-       for (key_offset = 0; key_offset < 3; key_offset++) {
+       while (key_counts) {
                hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_RSS_GENERIC_CONFIG,
                                           false);
 
                req->hash_config |= (hfunc & HCLGE_RSS_HASH_ALGO_MASK);
                req->hash_config |= (key_offset << HCLGE_RSS_HASH_KEY_OFFSET_B);
 
-               if (key_offset == 2)
-                       key_size =
-                       HCLGE_RSS_KEY_SIZE - HCLGE_RSS_HASH_KEY_NUM * 2;
-               else
-                       key_size = HCLGE_RSS_HASH_KEY_NUM;
-
+               key_size = min(HCLGE_RSS_HASH_KEY_NUM, key_counts);
                memcpy(req->hash_key,
                       key + key_offset * HCLGE_RSS_HASH_KEY_NUM, key_size);
 
+               key_counts -= key_size;
+               key_offset++;
                ret = hclge_cmd_send(&hdev->hw, &desc, 1);
                if (ret) {
                        dev_err(&hdev->pdev->dev,
@@ -3995,7 +4091,8 @@ int hclge_rss_init_hw(struct hclge_dev *hdev)
        u16 tc_valid[HCLGE_MAX_TC_NUM];
        u16 tc_size[HCLGE_MAX_TC_NUM];
        u16 roundup_size;
-       int i, ret;
+       unsigned int i;
+       int ret;
 
        ret = hclge_set_rss_indir_table(hdev, rss_indir);
        if (ret)
@@ -4150,8 +4247,7 @@ int hclge_bind_ring_with_vector(struct hclge_vport *vport,
        return 0;
 }
 
-static int hclge_map_ring_to_vector(struct hnae3_handle *handle,
-                                   int vector,
+static int hclge_map_ring_to_vector(struct hnae3_handle *handle, int vector,
                                    struct hnae3_ring_chain_node *ring_chain)
 {
        struct hclge_vport *vport = hclge_get_vport(handle);
@@ -4168,8 +4264,7 @@ static int hclge_map_ring_to_vector(struct hnae3_handle *handle,
        return hclge_bind_ring_with_vector(vport, vector_id, true, ring_chain);
 }
 
-static int hclge_unmap_ring_frm_vector(struct hnae3_handle *handle,
-                                      int vector,
+static int hclge_unmap_ring_frm_vector(struct hnae3_handle *handle, int vector,
                                       struct hnae3_ring_chain_node *ring_chain)
 {
        struct hclge_vport *vport = hclge_get_vport(handle);
@@ -4190,8 +4285,7 @@ static int hclge_unmap_ring_frm_vector(struct hnae3_handle *handle,
        if (ret)
                dev_err(&handle->pdev->dev,
                        "Unmap ring from vector fail. vectorid=%d, ret =%d\n",
-                       vector_id,
-                       ret);
+                       vector_id, ret);
 
        return ret;
 }
@@ -4611,7 +4705,7 @@ static void hclge_fd_convert_meta_data(struct hclge_fd_key_cfg *key_cfg,
 {
        u32 tuple_bit, meta_data = 0, tmp_x, tmp_y, port_number;
        u8 cur_pos = 0, tuple_size, shift_bits;
-       int i;
+       unsigned int i;
 
        for (i = 0; i < MAX_META_DATA; i++) {
                tuple_size = meta_data_key_info[i].key_length;
@@ -4653,7 +4747,8 @@ static int hclge_config_key(struct hclge_dev *hdev, u8 stage,
        struct hclge_fd_key_cfg *key_cfg = &hdev->fd_cfg.key_cfg[stage];
        u8 key_x[MAX_KEY_BYTES], key_y[MAX_KEY_BYTES];
        u8 *cur_key_x, *cur_key_y;
-       int i, ret, tuple_size;
+       unsigned int i;
+       int ret, tuple_size;
        u8 meta_data_region;
 
        memset(key_x, 0, sizeof(key_x));
@@ -5255,13 +5350,12 @@ static int hclge_del_fd_entry(struct hnae3_handle *handle,
 
        if (!hclge_fd_rule_exist(hdev, fs->location)) {
                dev_err(&hdev->pdev->dev,
-                       "Delete fail, rule %d is inexistent\n",
-                       fs->location);
+                       "Delete fail, rule %d is inexistent\n", fs->location);
                return -ENOENT;
        }
 
-       ret = hclge_fd_tcam_config(hdev, HCLGE_FD_STAGE_1, true,
-                                  fs->location, NULL, false);
+       ret = hclge_fd_tcam_config(hdev, HCLGE_FD_STAGE_1, true, fs->location,
+                                  NULL, false);
        if (ret)
                return ret;
 
@@ -5978,7 +6072,7 @@ static int hclge_set_serdes_loopback(struct hclge_dev *hdev, bool en,
        return -EBUSY;
 }
 
-static int hclge_tqp_enable(struct hclge_dev *hdev, int tqp_id,
+static int hclge_tqp_enable(struct hclge_dev *hdev, unsigned int tqp_id,
                            int stream_id, bool enable)
 {
        struct hclge_desc desc;
@@ -5989,7 +6083,8 @@ static int hclge_tqp_enable(struct hclge_dev *hdev, int tqp_id,
        hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_CFG_COM_TQP_QUEUE, false);
        req->tqp_id = cpu_to_le16(tqp_id & HCLGE_RING_ID_MASK);
        req->stream_id = cpu_to_le16(stream_id);
-       req->enable |= enable << HCLGE_TQP_ENABLE_B;
+       if (enable)
+               req->enable |= 1U << HCLGE_TQP_ENABLE_B;
 
        ret = hclge_cmd_send(&hdev->hw, &desc, 1);
        if (ret)
@@ -6441,7 +6536,9 @@ static int hclge_set_umv_space(struct hclge_dev *hdev, u16 space_size,
 
        req = (struct hclge_umv_spc_alc_cmd *)desc.data;
        hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_MAC_VLAN_ALLOCATE, false);
-       hnae3_set_bit(req->allocate, HCLGE_UMV_SPC_ALC_B, !is_alloc);
+       if (!is_alloc)
+               hnae3_set_bit(req->allocate, HCLGE_UMV_SPC_ALC_B, 1);
+
        req->space_size = cpu_to_le32(space_size);
 
        ret = hclge_cmd_send(&hdev->hw, &desc, 1);
@@ -6530,8 +6627,7 @@ int hclge_add_uc_addr_common(struct hclge_vport *vport,
            is_multicast_ether_addr(addr)) {
                dev_err(&hdev->pdev->dev,
                        "Set_uc mac err! invalid mac:%pM. is_zero:%d,is_br=%d,is_mul=%d\n",
-                        addr,
-                        is_zero_ether_addr(addr),
+                        addr, is_zero_ether_addr(addr),
                         is_broadcast_ether_addr(addr),
                         is_multicast_ether_addr(addr));
                return -EINVAL;
@@ -6598,9 +6694,8 @@ int hclge_rm_uc_addr_common(struct hclge_vport *vport,
        if (is_zero_ether_addr(addr) ||
            is_broadcast_ether_addr(addr) ||
            is_multicast_ether_addr(addr)) {
-               dev_dbg(&hdev->pdev->dev,
-                       "Remove mac err! invalid mac:%pM.\n",
-                        addr);
+               dev_dbg(&hdev->pdev->dev, "Remove mac err! invalid mac:%pM.\n",
+                       addr);
                return -EINVAL;
        }
 
@@ -6641,18 +6736,16 @@ int hclge_add_mc_addr_common(struct hclge_vport *vport,
        hnae3_set_bit(req.entry_type, HCLGE_MAC_VLAN_BIT0_EN_B, 0);
        hclge_prepare_mac_addr(&req, addr, true);
        status = hclge_lookup_mac_vlan_tbl(vport, &req, desc, true);
-       if (!status) {
-               /* This mac addr exist, update VFID for it */
-               hclge_update_desc_vfid(desc, vport->vport_id, false);
-               status = hclge_add_mac_vlan_tbl(vport, &req, desc);
-       } else {
+       if (status) {
                /* This mac addr do not exist, add new entry for it */
                memset(desc[0].data, 0, sizeof(desc[0].data));
                memset(desc[1].data, 0, sizeof(desc[0].data));
                memset(desc[2].data, 0, sizeof(desc[0].data));
-               hclge_update_desc_vfid(desc, vport->vport_id, false);
-               status = hclge_add_mac_vlan_tbl(vport, &req, desc);
        }
+       status = hclge_update_desc_vfid(desc, vport->vport_id, false);
+       if (status)
+               return status;
+       status = hclge_add_mac_vlan_tbl(vport, &req, desc);
 
        if (status == -ENOSPC)
                dev_err(&hdev->pdev->dev, "mc mac vlan table is full\n");
@@ -6690,7 +6783,9 @@ int hclge_rm_mc_addr_common(struct hclge_vport *vport,
        status = hclge_lookup_mac_vlan_tbl(vport, &req, desc, true);
        if (!status) {
                /* This mac addr exist, remove this handle's VFID for it */
-               hclge_update_desc_vfid(desc, vport->vport_id, true);
+               status = hclge_update_desc_vfid(desc, vport->vport_id, true);
+               if (status)
+                       return status;
 
                if (hclge_is_all_function_id_zero(desc))
                        /* All the vfid is zero, so need to delete this entry */
@@ -7019,7 +7114,7 @@ static void hclge_enable_vlan_filter(struct hnae3_handle *handle, bool enable)
                handle->netdev_flags &= ~HNAE3_VLAN_FLTR;
 }
 
-static int hclge_set_vf_vlan_common(struct hclge_dev *hdev, int vfid,
+static int hclge_set_vf_vlan_common(struct hclge_dev *hdev, u16 vfid,
                                    bool is_kill, u16 vlan, u8 qos,
                                    __be16 proto)
 {
@@ -7086,12 +7181,13 @@ static int hclge_set_vf_vlan_common(struct hclge_dev *hdev, int vfid,
                if (!req0->resp_code)
                        return 0;
 
-               if (req0->resp_code == HCLGE_VF_VLAN_DEL_NO_FOUND) {
-                       dev_warn(&hdev->pdev->dev,
-                                "vlan %d filter is not in vf vlan table\n",
-                                vlan);
+               /* vf vlan filter is disabled when vf vlan table is full,
+                * then new vlan id will not be added into vf vlan table.
+                * Just return 0 without warning, avoid massive verbose
+                * print logs when unload.
+                */
+               if (req0->resp_code == HCLGE_VF_VLAN_DEL_NO_FOUND)
                        return 0;
-               }
 
                dev_err(&hdev->pdev->dev,
                        "Kill vf vlan filter fail, ret =%d.\n",
@@ -7715,11 +7811,20 @@ int hclge_set_vlan_filter(struct hnae3_handle *handle, __be16 proto,
        bool writen_to_tbl = false;
        int ret = 0;
 
-       /* when port based VLAN enabled, we use port based VLAN as the VLAN
-        * filter entry. In this case, we don't update VLAN filter table
-        * when user add new VLAN or remove exist VLAN, just update the vport
-        * VLAN list. The VLAN id in VLAN list won't be writen in VLAN filter
-        * table until port based VLAN disabled
+       /* When device is resetting, firmware is unable to handle
+        * mailbox. Just record the vlan id, and remove it after
+        * reset finished.
+        */
+       if (test_bit(HCLGE_STATE_RST_HANDLING, &hdev->state) && is_kill) {
+               set_bit(vlan_id, vport->vlan_del_fail_bmap);
+               return -EBUSY;
+       }
+
+       /* When port base vlan enabled, we use port base vlan as the vlan
+        * filter entry. In this case, we don't update vlan filter table
+        * when user add new vlan or remove exist vlan, just update the vport
+        * vlan list. The vlan id in vlan list will be writen in vlan filter
+        * table until port base vlan disabled
         */
        if (handle->port_base_vlan_state == HNAE3_PORT_BASE_VLAN_DISABLE) {
                ret = hclge_set_vlan_filter_hw(hdev, proto, vport->vport_id,
@@ -7727,16 +7832,53 @@ int hclge_set_vlan_filter(struct hnae3_handle *handle, __be16 proto,
                writen_to_tbl = true;
        }
 
-       if (ret)
-               return ret;
+       if (!ret) {
+               if (is_kill)
+                       hclge_rm_vport_vlan_table(vport, vlan_id, false);
+               else
+                       hclge_add_vport_vlan_table(vport, vlan_id,
+                                                  writen_to_tbl);
+       } else if (is_kill) {
+               /* When remove hw vlan filter failed, record the vlan id,
+                * and try to remove it from hw later, to be consistence
+                * with stack
+                */
+               set_bit(vlan_id, vport->vlan_del_fail_bmap);
+       }
+       return ret;
+}
 
-       if (is_kill)
-               hclge_rm_vport_vlan_table(vport, vlan_id, false);
-       else
-               hclge_add_vport_vlan_table(vport, vlan_id,
-                                          writen_to_tbl);
+static void hclge_sync_vlan_filter(struct hclge_dev *hdev)
+{
+#define HCLGE_MAX_SYNC_COUNT   60
 
-       return 0;
+       int i, ret, sync_cnt = 0;
+       u16 vlan_id;
+
+       /* start from vport 1 for PF is always alive */
+       for (i = 0; i < hdev->num_alloc_vport; i++) {
+               struct hclge_vport *vport = &hdev->vport[i];
+
+               vlan_id = find_first_bit(vport->vlan_del_fail_bmap,
+                                        VLAN_N_VID);
+               while (vlan_id != VLAN_N_VID) {
+                       ret = hclge_set_vlan_filter_hw(hdev, htons(ETH_P_8021Q),
+                                                      vport->vport_id, vlan_id,
+                                                      0, true);
+                       if (ret && ret != -EINVAL)
+                               return;
+
+                       clear_bit(vlan_id, vport->vlan_del_fail_bmap);
+                       hclge_rm_vport_vlan_table(vport, vlan_id, false);
+
+                       sync_cnt++;
+                       if (sync_cnt >= HCLGE_MAX_SYNC_COUNT)
+                               return;
+
+                       vlan_id = find_first_bit(vport->vlan_del_fail_bmap,
+                                                VLAN_N_VID);
+               }
+       }
 }
 
 static int hclge_set_mac_mtu(struct hclge_dev *hdev, int new_mps)
@@ -7763,7 +7905,7 @@ static int hclge_set_mtu(struct hnae3_handle *handle, int new_mtu)
 int hclge_set_vport_mtu(struct hclge_vport *vport, int new_mtu)
 {
        struct hclge_dev *hdev = vport->back;
-       int i, max_frm_size, ret = 0;
+       int i, max_frm_size, ret;
 
        max_frm_size = new_mtu + ETH_HLEN + ETH_FCS_LEN + 2 * VLAN_HLEN;
        if (max_frm_size < HCLGE_MAC_MIN_FRAME ||
@@ -7874,7 +8016,7 @@ int hclge_reset_tqp(struct hnae3_handle *handle, u16 queue_id)
        int reset_try_times = 0;
        int reset_status;
        u16 queue_gid;
-       int ret = 0;
+       int ret;
 
        queue_gid = hclge_covert_handle_qid_global(handle, queue_id);
 
@@ -7891,7 +8033,6 @@ int hclge_reset_tqp(struct hnae3_handle *handle, u16 queue_id)
                return ret;
        }
 
-       reset_try_times = 0;
        while (reset_try_times++ < HCLGE_TQP_RESET_TRY_TIMES) {
                /* Wait for tqp hw reset */
                msleep(20);
@@ -7930,7 +8071,6 @@ void hclge_reset_vf_queue(struct hclge_vport *vport, u16 queue_id)
                return;
        }
 
-       reset_try_times = 0;
        while (reset_try_times++ < HCLGE_TQP_RESET_TRY_TIMES) {
                /* Wait for tqp hw reset */
                msleep(20);
@@ -8000,7 +8140,7 @@ int hclge_cfg_flowctrl(struct hclge_dev *hdev)
 {
        struct phy_device *phydev = hdev->hw.mac.phydev;
        u16 remote_advertising = 0;
-       u16 local_advertising = 0;
+       u16 local_advertising;
        u32 rx_pause, tx_pause;
        u8 flowctl;
 
@@ -8125,7 +8265,8 @@ static void hclge_get_mdix_mode(struct hnae3_handle *handle,
        struct hclge_vport *vport = hclge_get_vport(handle);
        struct hclge_dev *hdev = vport->back;
        struct phy_device *phydev = hdev->hw.mac.phydev;
-       int mdix_ctrl, mdix, retval, is_resolved;
+       int mdix_ctrl, mdix, is_resolved;
+       unsigned int retval;
 
        if (!phydev) {
                *tp_mdix_ctrl = ETH_TP_MDI_INVALID;
@@ -8199,25 +8340,44 @@ static int hclge_init_nic_client_instance(struct hnae3_ae_dev *ae_dev,
 {
        struct hnae3_client *client = vport->nic.client;
        struct hclge_dev *hdev = ae_dev->priv;
+       int rst_cnt;
        int ret;
 
+       rst_cnt = hdev->rst_stats.reset_cnt;
        ret = client->ops->init_instance(&vport->nic);
        if (ret)
                return ret;
 
        set_bit(HCLGE_STATE_NIC_REGISTERED, &hdev->state);
-       hnae3_set_client_init_flag(client, ae_dev, 1);
+       if (test_bit(HCLGE_STATE_RST_HANDLING, &hdev->state) ||
+           rst_cnt != hdev->rst_stats.reset_cnt) {
+               ret = -EBUSY;
+               goto init_nic_err;
+       }
 
        /* Enable nic hw error interrupts */
        ret = hclge_config_nic_hw_error(hdev, true);
-       if (ret)
+       if (ret) {
                dev_err(&ae_dev->pdev->dev,
                        "fail(%d) to enable hw error interrupts\n", ret);
+               goto init_nic_err;
+       }
+
+       hnae3_set_client_init_flag(client, ae_dev, 1);
 
        if (netif_msg_drv(&hdev->vport->nic))
                hclge_info_show(hdev);
 
        return ret;
+
+init_nic_err:
+       clear_bit(HCLGE_STATE_NIC_REGISTERED, &hdev->state);
+       while (test_bit(HCLGE_STATE_RST_HANDLING, &hdev->state))
+               msleep(HCLGE_WAIT_RESET_DONE);
+
+       client->ops->uninit_instance(&vport->nic, 0);
+
+       return ret;
 }
 
 static int hclge_init_roce_client_instance(struct hnae3_ae_dev *ae_dev,
@@ -8225,6 +8385,7 @@ static int hclge_init_roce_client_instance(struct hnae3_ae_dev *ae_dev,
 {
        struct hnae3_client *client = vport->roce.client;
        struct hclge_dev *hdev = ae_dev->priv;
+       int rst_cnt;
        int ret;
 
        if (!hnae3_dev_roce_supported(hdev) || !hdev->roce_client ||
@@ -8236,14 +8397,38 @@ static int hclge_init_roce_client_instance(struct hnae3_ae_dev *ae_dev,
        if (ret)
                return ret;
 
+       rst_cnt = hdev->rst_stats.reset_cnt;
        ret = client->ops->init_instance(&vport->roce);
        if (ret)
                return ret;
 
        set_bit(HCLGE_STATE_ROCE_REGISTERED, &hdev->state);
+       if (test_bit(HCLGE_STATE_RST_HANDLING, &hdev->state) ||
+           rst_cnt != hdev->rst_stats.reset_cnt) {
+               ret = -EBUSY;
+               goto init_roce_err;
+       }
+
+       /* Enable roce ras interrupts */
+       ret = hclge_config_rocee_ras_interrupt(hdev, true);
+       if (ret) {
+               dev_err(&ae_dev->pdev->dev,
+                       "fail(%d) to enable roce ras interrupts\n", ret);
+               goto init_roce_err;
+       }
+
        hnae3_set_client_init_flag(client, ae_dev, 1);
 
        return 0;
+
+init_roce_err:
+       clear_bit(HCLGE_STATE_ROCE_REGISTERED, &hdev->state);
+       while (test_bit(HCLGE_STATE_RST_HANDLING, &hdev->state))
+               msleep(HCLGE_WAIT_RESET_DONE);
+
+       hdev->roce_client->ops->uninit_instance(&vport->roce, 0);
+
+       return ret;
 }
 
 static int hclge_init_client_instance(struct hnae3_client *client,
@@ -8286,12 +8471,6 @@ static int hclge_init_client_instance(struct hnae3_client *client,
                }
        }
 
-       /* Enable roce ras interrupts */
-       ret = hclge_config_rocee_ras_interrupt(hdev, true);
-       if (ret)
-               dev_err(&ae_dev->pdev->dev,
-                       "fail(%d) to enable roce ras interrupts\n", ret);
-
        return ret;
 
 clear_nic:
@@ -8315,6 +8494,9 @@ static void hclge_uninit_client_instance(struct hnae3_client *client,
                vport = &hdev->vport[i];
                if (hdev->roce_client) {
                        clear_bit(HCLGE_STATE_ROCE_REGISTERED, &hdev->state);
+                       while (test_bit(HCLGE_STATE_RST_HANDLING, &hdev->state))
+                               msleep(HCLGE_WAIT_RESET_DONE);
+
                        hdev->roce_client->ops->uninit_instance(&vport->roce,
                                                                0);
                        hdev->roce_client = NULL;
@@ -8324,6 +8506,9 @@ static void hclge_uninit_client_instance(struct hnae3_client *client,
                        return;
                if (hdev->nic_client && client->ops->uninit_instance) {
                        clear_bit(HCLGE_STATE_NIC_REGISTERED, &hdev->state);
+                       while (test_bit(HCLGE_STATE_RST_HANDLING, &hdev->state))
+                               msleep(HCLGE_WAIT_RESET_DONE);
+
                        client->ops->uninit_instance(&vport->nic, 0);
                        hdev->nic_client = NULL;
                        vport->nic.client = NULL;
@@ -8447,6 +8632,23 @@ static void hclge_flr_done(struct hnae3_ae_dev *ae_dev)
        set_bit(HNAE3_FLR_DONE, &hdev->flr_state);
 }
 
+static void hclge_clear_resetting_state(struct hclge_dev *hdev)
+{
+       u16 i;
+
+       for (i = 0; i < hdev->num_alloc_vport; i++) {
+               struct hclge_vport *vport = &hdev->vport[i];
+               int ret;
+
+                /* Send cmd to clear VF's FUNC_RST_ING */
+               ret = hclge_set_vf_rst(hdev, vport->vport_id, false);
+               if (ret)
+                       dev_warn(&hdev->pdev->dev,
+                                "clear vf(%d) rst failed %d!\n",
+                                vport->vport_id, ret);
+       }
+}
+
 static int hclge_init_ae_dev(struct hnae3_ae_dev *ae_dev)
 {
        struct pci_dev *pdev = ae_dev->pdev;
@@ -8607,6 +8809,22 @@ static int hclge_init_ae_dev(struct hnae3_ae_dev *ae_dev)
        INIT_WORK(&hdev->mbx_service_task, hclge_mailbox_service_task);
 
        hclge_clear_all_event_cause(hdev);
+       hclge_clear_resetting_state(hdev);
+
+       /* Log and clear the hw errors those already occurred */
+       hclge_handle_all_hns_hw_errors(ae_dev);
+
+       /* request delayed reset for the error recovery because an immediate
+        * global reset on a PF affecting pending initialization of other PFs
+        */
+       if (ae_dev->hw_err_reset_req) {
+               enum hnae3_reset_type reset_level;
+
+               reset_level = hclge_get_reset_level(ae_dev,
+                                                   &ae_dev->hw_err_reset_req);
+               hclge_set_def_reset_request(ae_dev, reset_level);
+               mod_timer(&hdev->reset_timer, jiffies + HCLGE_RESET_INTERVAL);
+       }
 
        /* Enable MISC vector(vector0) */
        hclge_enable_vector(&hdev->misc_vector, true);
@@ -8713,8 +8931,7 @@ static int hclge_reset_ae_dev(struct hnae3_ae_dev *ae_dev)
 
        ret = hclge_init_fd_config(hdev);
        if (ret) {
-               dev_err(&pdev->dev,
-                       "fd table init fail, ret=%d\n", ret);
+               dev_err(&pdev->dev, "fd table init fail, ret=%d\n", ret);
                return ret;
        }
 
@@ -8820,7 +9037,8 @@ static int hclge_set_channels(struct hnae3_handle *handle, u32 new_tqps_num,
        u16 tc_size[HCLGE_MAX_TC_NUM];
        u16 roundup_size;
        u32 *rss_indir;
-       int ret, i;
+       unsigned int i;
+       int ret;
 
        kinfo->req_rss_size = new_tqps_num;
 
@@ -9218,6 +9436,7 @@ static const struct hnae3_ae_ops hclge_ops = {
        .set_autoneg = hclge_set_autoneg,
        .get_autoneg = hclge_get_autoneg,
        .restart_autoneg = hclge_restart_autoneg,
+       .halt_autoneg = hclge_halt_autoneg,
        .get_pauseparam = hclge_get_pauseparam,
        .set_pauseparam = hclge_set_pauseparam,
        .set_mtu = hclge_set_mtu,
@@ -9234,6 +9453,7 @@ static const struct hnae3_ae_ops hclge_ops = {
        .set_vf_vlan_filter = hclge_set_vf_vlan_filter,
        .enable_hw_strip_rxvtag = hclge_en_hw_strip_rxvtag,
        .reset_event = hclge_reset_event,
+       .get_reset_level = hclge_get_reset_level,
        .set_default_reset_request = hclge_set_def_reset_request,
        .get_tqps_and_rss_info = hclge_get_tqps_and_rss_info,
        .set_channels = hclge_set_channels,