net/hns3: fix log on flow director clear

If clear FDIR rules fail, the error code was logged, but the error code
was useless because it was the sum of all fail code.

This patch fixes it by log the success cnt and fail cnt.

Fixes: fcba820d9b9e ("net/hns3: support flow director")
Fixes: 8eed8acc812e ("net/hns3: add error code to some logs")
Cc: stable@dpdk.org
Signed-off-by: Chengwen Feng <fengchengwen@huawei.com>
Signed-off-by: Min Hu (Connor) <humin29@huawei.com>

diff --git a/drivers/net/hns3/hns3_fdir.c b/drivers/net/hns3/hns3_fdir.c
index 0ef0938..e116d87 100644 (file)
--- a/drivers/net/hns3/hns3_fdir.c
+++ b/drivers/net/hns3/hns3_fdir.c
@@ -1026,6 +1026,8 @@ int hns3_clear_all_fdir_filter(struct hns3_adapter *hns)
        struct hns3_fdir_info *fdir_info = &pf->fdir;
        struct hns3_fdir_rule_ele *fdir_filter;
        struct hns3_hw *hw = &hns->hw;
+       int succ_cnt = 0;
+       int fail_cnt = 0;
        int ret = 0;
 
        /* flush flow director */
@@ -1034,17 +1036,23 @@ int hns3_clear_all_fdir_filter(struct hns3_adapter *hns)
        fdir_filter = TAILQ_FIRST(&fdir_info->fdir_list);
        while (fdir_filter) {
                TAILQ_REMOVE(&fdir_info->fdir_list, fdir_filter, entries);
-               ret += hns3_fd_tcam_config(hw, true,
-                                          fdir_filter->fdir_conf.location,
-                                          NULL, false);
+               ret = hns3_fd_tcam_config(hw, true,
+                                         fdir_filter->fdir_conf.location,
+                                         NULL, false);
+               if (ret == 0)
+                       succ_cnt++;
+               else
+                       fail_cnt++;
                rte_free(fdir_filter);
                fdir_filter = TAILQ_FIRST(&fdir_info->fdir_list);
        }
 
-       if (ret) {
-               hns3_err(hw, "Fail to delete FDIR filter, ret = %d", ret);
+       if (fail_cnt > 0) {
+               hns3_err(hw, "fail to delete all FDIR filter, success num = %d "
+                        "fail num = %d", succ_cnt, fail_cnt);
                ret = -EIO;
        }
+
        return ret;
 }