net/hns3: refactor converting descriptor error
authorLijun Ou <oulijun@huawei.com>
Fri, 22 Jan 2021 10:18:41 +0000 (18:18 +0800)
committerFerruh Yigit <ferruh.yigit@intel.com>
Fri, 29 Jan 2021 17:16:11 +0000 (18:16 +0100)
Use errno array instead of switch-case for refactor
the hns3_cmd_convert_err_code function.

Besides, we add a type for ROH(RDMA Over HCCS) check
cmdq return error in Kunpeng930 NIC hardware.

Signed-off-by: Lijun Ou <oulijun@huawei.com>
drivers/net/hns3/hns3_cmd.c
drivers/net/hns3/hns3_cmd.h

index 45dbead..9ed8161 100644 (file)
@@ -247,34 +247,32 @@ hns3_is_special_opcode(uint16_t opcode)
 static int
 hns3_cmd_convert_err_code(uint16_t desc_ret)
 {
-       switch (desc_ret) {
-       case HNS3_CMD_EXEC_SUCCESS:
-               return 0;
-       case HNS3_CMD_NO_AUTH:
-               return -EPERM;
-       case HNS3_CMD_NOT_SUPPORTED:
-               return -EOPNOTSUPP;
-       case HNS3_CMD_QUEUE_FULL:
-               return -EXFULL;
-       case HNS3_CMD_NEXT_ERR:
-               return -ENOSR;
-       case HNS3_CMD_UNEXE_ERR:
-               return -ENOTBLK;
-       case HNS3_CMD_PARA_ERR:
-               return -EINVAL;
-       case HNS3_CMD_RESULT_ERR:
-               return -ERANGE;
-       case HNS3_CMD_TIMEOUT:
-               return -ETIME;
-       case HNS3_CMD_HILINK_ERR:
-               return -ENOLINK;
-       case HNS3_CMD_QUEUE_ILLEGAL:
-               return -ENXIO;
-       case HNS3_CMD_INVALID:
-               return -EBADR;
-       default:
-               return -EREMOTEIO;
-       }
+       static const struct {
+               uint16_t imp_errcode;
+               int linux_errcode;
+       } hns3_cmdq_status[] = {
+               {HNS3_CMD_EXEC_SUCCESS, 0},
+               {HNS3_CMD_NO_AUTH, -EPERM},
+               {HNS3_CMD_NOT_SUPPORTED, -EOPNOTSUPP},
+               {HNS3_CMD_QUEUE_FULL, -EXFULL},
+               {HNS3_CMD_NEXT_ERR, -ENOSR},
+               {HNS3_CMD_UNEXE_ERR, -ENOTBLK},
+               {HNS3_CMD_PARA_ERR, -EINVAL},
+               {HNS3_CMD_RESULT_ERR, -ERANGE},
+               {HNS3_CMD_TIMEOUT, -ETIME},
+               {HNS3_CMD_HILINK_ERR, -ENOLINK},
+               {HNS3_CMD_QUEUE_ILLEGAL, -ENXIO},
+               {HNS3_CMD_INVALID, -EBADR},
+               {HNS3_CMD_ROH_CHECK_FAIL, -EINVAL}
+       };
+
+       uint32_t i;
+
+       for (i = 0; i < ARRAY_SIZE(hns3_cmdq_status); i++)
+               if (hns3_cmdq_status[i].imp_errcode == desc_ret)
+                       return hns3_cmdq_status[i].linux_errcode;
+
+       return -EREMOTEIO;
 }
 
 static int
index e40293b..6152f6e 100644 (file)
@@ -52,6 +52,7 @@ enum hns3_cmd_return_status {
        HNS3_CMD_HILINK_ERR     = 9,
        HNS3_CMD_QUEUE_ILLEGAL  = 10,
        HNS3_CMD_INVALID        = 11,
+       HNS3_CMD_ROH_CHECK_FAIL = 12
 };
 
 enum hns3_cmd_status {