From: Xiaoyun wang <cloud.wangxiaoyun@huawei.com>
To: <dev@dpdk.org>
Cc: <ferruh.yigit@intel.com>, <bluca@debian.org>,
<luoxianjun@huawei.com>, <luoxingyu@huawei.com>,
<zhouguoyang@huawei.com>, <yin.yinshi@huawei.com>,
<david.yangxiaoliang@huawei.com>, <zhaohui8@huawei.com>,
<zhengjingzhou@huawei.com>,
Xiaoyun wang <cloud.wangxiaoyun@huawei.com>
Subject: [dpdk-dev] [PATCH v1 5/5] net/hinic/base: modify return errors
Date: Sat, 27 Jun 2020 11:55:48 +0800 [thread overview]
Message-ID: <fe8c40d266837ffc8a145afc476115c89ad167a1.1593228634.git.cloud.wangxiaoyun@huawei.com> (raw)
In-Reply-To: <cover.1593228634.git.cloud.wangxiaoyun@huawei.com>
Modify return errors with linux system errors when
system requests fail.
Signed-off-by: Xiaoyun wang <cloud.wangxiaoyun@huawei.com>
---
drivers/net/hinic/base/hinic_pmd_cfg.c | 4 +-
drivers/net/hinic/base/hinic_pmd_hwdev.c | 16 ++---
drivers/net/hinic/base/hinic_pmd_hwif.c | 4 +-
drivers/net/hinic/base/hinic_pmd_niccfg.c | 104 +++++++++++++++---------------
drivers/net/hinic/base/hinic_pmd_nicio.c | 6 +-
5 files changed, 66 insertions(+), 68 deletions(-)
diff --git a/drivers/net/hinic/base/hinic_pmd_cfg.c b/drivers/net/hinic/base/hinic_pmd_cfg.c
index 2d25dc9..a0cc16d 100644
--- a/drivers/net/hinic/base/hinic_pmd_cfg.c
+++ b/drivers/net/hinic/base/hinic_pmd_cfg.c
@@ -179,7 +179,7 @@ static int get_cap_from_fw(struct hinic_hwdev *dev, enum func_type type)
if (err || dev_cap.mgmt_msg_head.status || !out_len) {
PMD_DRV_LOG(ERR, "Get capability from FW failed, err: %d, status: %d, out_len: %d",
err, dev_cap.mgmt_msg_head.status, out_len);
- return -EFAULT;
+ return -EIO;
}
parse_dev_cap(dev, &dev_cap, type);
@@ -201,7 +201,7 @@ static int get_cap_from_pf(struct hinic_hwdev *dev, enum func_type type)
if (err || dev_cap.mgmt_msg_head.status || !out_len) {
PMD_DRV_LOG(ERR, "Get capability from PF failed, err: %d, status: %d, out_len: %d",
err, dev_cap.mgmt_msg_head.status, out_len);
- return -EFAULT;
+ return -EIO;
}
parse_dev_cap(dev, &dev_cap, type);
diff --git a/drivers/net/hinic/base/hinic_pmd_hwdev.c b/drivers/net/hinic/base/hinic_pmd_hwdev.c
index a93f240..2af3bd9 100644
--- a/drivers/net/hinic/base/hinic_pmd_hwdev.c
+++ b/drivers/net/hinic/base/hinic_pmd_hwdev.c
@@ -475,7 +475,7 @@ static int wait_for_flr_finish(struct hinic_hwif *hwif)
rte_delay_ms(10);
} while (time_before(jiffies, end));
- return -EFAULT;
+ return -ETIME;
}
#define HINIC_WAIT_CMDQ_IDLE_TIMEOUT 1000
@@ -681,17 +681,15 @@ int hinic_set_interrupt_cfg(struct hinic_hwdev *hwdev,
u16 out_size = sizeof(msix_cfg);
int err;
- memset(&msix_cfg, 0, sizeof(msix_cfg));
- msix_cfg.mgmt_msg_head.resp_aeq_num = HINIC_AEQ1;
- msix_cfg.func_id = hinic_global_func_id(hwdev);
- msix_cfg.msix_index = (u16)interrupt_info.msix_index;
-
temp_info.msix_index = interrupt_info.msix_index;
-
err = hinic_get_interrupt_cfg(hwdev, &temp_info);
if (err)
- return -EINVAL;
+ return -EIO;
+ memset(&msix_cfg, 0, sizeof(msix_cfg));
+ msix_cfg.mgmt_msg_head.resp_aeq_num = HINIC_AEQ1;
+ msix_cfg.func_id = hinic_global_func_id(hwdev);
+ msix_cfg.msix_index = (u16)interrupt_info.msix_index;
msix_cfg.lli_credit_cnt = temp_info.lli_credit_limit;
msix_cfg.lli_tmier_cnt = temp_info.lli_timer_cfg;
msix_cfg.pending_cnt = temp_info.pending_limt;
@@ -1051,7 +1049,7 @@ int hinic_get_board_info(void *hwdev, struct hinic_board_info *info)
if (err || board_info.mgmt_msg_head.status || !out_size) {
PMD_DRV_LOG(ERR, "Failed to get board info, err: %d, status: 0x%x, out size: 0x%x",
err, board_info.mgmt_msg_head.status, out_size);
- return -EFAULT;
+ return -EIO;
}
memcpy(info, &board_info.info, sizeof(*info));
diff --git a/drivers/net/hinic/base/hinic_pmd_hwif.c b/drivers/net/hinic/base/hinic_pmd_hwif.c
index 4578b68..a485cf0 100644
--- a/drivers/net/hinic/base/hinic_pmd_hwif.c
+++ b/drivers/net/hinic/base/hinic_pmd_hwif.c
@@ -321,7 +321,7 @@ int wait_until_doorbell_flush_states(struct hinic_hwif *hwif,
rte_delay_ms(1);
} while (time_before(jiffies, end));
- return -EFAULT;
+ return -ETIME;
}
static int wait_until_doorbell_and_outbound_enabled(struct hinic_hwif *hwif)
@@ -343,7 +343,7 @@ static int wait_until_doorbell_and_outbound_enabled(struct hinic_hwif *hwif)
rte_delay_ms(1);
} while (time_before(jiffies, end));
- return -EFAULT;
+ return -ETIME;
}
u16 hinic_global_func_id(void *hwdev)
diff --git a/drivers/net/hinic/base/hinic_pmd_niccfg.c b/drivers/net/hinic/base/hinic_pmd_niccfg.c
index 67f6bc4..be6445d 100644
--- a/drivers/net/hinic/base/hinic_pmd_niccfg.c
+++ b/drivers/net/hinic/base/hinic_pmd_niccfg.c
@@ -55,7 +55,7 @@ int hinic_init_function_table(void *hwdev, u16 rx_buf_sz)
PMD_DRV_LOG(ERR,
"Failed to init func table, err: %d, status: 0x%x, out size: 0x%x",
err, function_table.mgmt_msg_head.status, out_size);
- return -EFAULT;
+ return -EIO;
}
return 0;
@@ -96,7 +96,7 @@ int hinic_get_base_qpn(void *hwdev, u16 *global_qpn)
PMD_DRV_LOG(ERR,
"Failed to get base qpn, err: %d, status: 0x%x, out size: 0x%x",
err, cmd_qpn.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
*global_qpn = cmd_qpn.base_qpn;
@@ -143,7 +143,7 @@ int hinic_set_mac(void *hwdev, u8 *mac_addr, u16 vlan_id, u16 func_id)
mac_info.mgmt_msg_head.status != HINIC_PF_SET_VF_ALREADY)) {
PMD_DRV_LOG(ERR, "Failed to set MAC, err: %d, status: 0x%x, out size: 0x%x",
err, mac_info.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
if (mac_info.mgmt_msg_head.status == HINIC_PF_SET_VF_ALREADY) {
@@ -198,7 +198,7 @@ int hinic_del_mac(void *hwdev, u8 *mac_addr, u16 vlan_id, u16 func_id)
mac_info.mgmt_msg_head.status != HINIC_PF_SET_VF_ALREADY)) {
PMD_DRV_LOG(ERR, "Failed to delete MAC, err: %d, status: 0x%x, out size: 0x%x",
err, mac_info.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
if (mac_info.mgmt_msg_head.status == HINIC_PF_SET_VF_ALREADY) {
PMD_DRV_LOG(WARNING, "PF has already set vf mac, Ignore delete operation.");
@@ -241,7 +241,7 @@ int hinic_get_default_mac(void *hwdev, u8 *mac_addr)
if (err || !out_size || mac_info.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to get mac, err: %d, status: 0x%x, out size: 0x%x",
err, mac_info.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
memmove(mac_addr, mac_info.mac, ETH_ALEN);
@@ -294,7 +294,7 @@ int hinic_update_mac(void *hwdev, u8 *old_mac, u8 *new_mac, u16 vlan_id,
mac_info.mgmt_msg_head.status != HINIC_PF_SET_VF_ALREADY)) {
PMD_DRV_LOG(ERR, "Failed to update MAC, err: %d, status: 0x%x, out size: 0x%x",
err, mac_info.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
if (mac_info.mgmt_msg_head.status == HINIC_PF_SET_VF_ALREADY) {
PMD_DRV_LOG(WARNING, "PF has already set vf mac, Ignore update operation");
@@ -338,7 +338,7 @@ int hinic_set_port_mtu(void *hwdev, u32 new_mtu)
if (err || !out_size || mtu_info.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to set mtu, err: %d, status: 0x%x, out size: 0x%x",
err, mtu_info.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -386,7 +386,7 @@ int hinic_add_remove_vlan(void *hwdev, u16 vlan_id, u16 func_id, bool add)
"Failed to %s vlan, err: %d, status: 0x%x, out size: 0x%x",
add ? "add" : "remove", err,
vlan_info.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -432,7 +432,7 @@ int hinic_config_vlan_filter(void *hwdev, u32 vlan_filter_ctrl)
"Failed to config vlan filter, vlan_filter_ctrl: 0x%x, err: %d, status: 0x%x, out size: 0x%x",
vlan_filter_ctrl, err,
vlan_filter.mgmt_msg_head.status, out_size);
- err = -EINVAL;
+ err = -EIO;
}
return err;
@@ -473,7 +473,7 @@ int hinic_set_rx_vlan_offload(void *hwdev, u8 en)
PMD_DRV_LOG(ERR,
"Failed to set rx vlan offload, err: %d, status: 0x%x, out size: 0x%x",
err, vlan_cfg.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -512,7 +512,7 @@ int hinic_get_link_status(void *hwdev, u8 *link_state)
if (err || !out_size || get_link.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to get link state, err: %d, status: 0x%x, out size: 0x%x",
err, get_link.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
*link_state = get_link.link_status;
@@ -554,7 +554,7 @@ int hinic_set_vport_enable(void *hwdev, bool enable)
if (err || !out_size || en_state.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to set vport state, err: %d, status: 0x%x, out size: 0x%x",
err, en_state.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -596,7 +596,7 @@ int hinic_set_port_enable(void *hwdev, bool enable)
if (err || !out_size || en_state.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to set phy port state, err: %d, status: 0x%x, out size: 0x%x",
err, en_state.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -660,7 +660,7 @@ int hinic_set_pause_config(void *hwdev, struct nic_pause_config nic_pause)
if (err || !out_size || pause_info.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to set pause info, err: %d, status: 0x%x, out size: 0x%x",
err, pause_info.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -685,7 +685,7 @@ int hinic_get_pause_info(void *hwdev, struct nic_pause_config *nic_pause)
if (err || !out_size || pause_info.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to get pause info, err: %d, status: 0x%x, out size: 0x%x\n",
err, pause_info.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
nic_pause->auto_neg = pause_info.auto_neg;
@@ -741,7 +741,7 @@ int hinic_dcb_set_ets(void *hwdev, u8 *up_tc, u8 *pg_bw,
PMD_DRV_LOG(ERR,
"Failed to set ets, err: %d, status: 0x%x, out size: 0x%x",
err, ets.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -773,7 +773,7 @@ int hinic_get_vport_stats(void *hwdev, struct hinic_vport_stats *stats)
PMD_DRV_LOG(ERR,
"Get vport stats from fw failed, err: %d, status: 0x%x, out size: 0x%x",
err, vport_stats_rsp.mgmt_msg_head.status, out_size);
- return -EFAULT;
+ return -EIO;
}
memcpy(stats, &vport_stats_rsp.stats, sizeof(*stats));
@@ -806,7 +806,7 @@ int hinic_get_phy_port_stats(void *hwdev, struct hinic_phy_port_stats *stats)
PMD_DRV_LOG(ERR,
"Failed to get port statistics, err: %d, status: 0x%x, out size: 0x%x",
err, port_stats_rsp.mgmt_msg_head.status, out_size);
- return -EFAULT;
+ return -EIO;
}
memcpy(stats, &port_stats_rsp.stats, sizeof(*stats));
@@ -863,7 +863,7 @@ int hinic_set_rss_type(void *hwdev, u32 tmpl_idx, struct nic_rss_type rss_type)
if (err || out_param != 0) {
PMD_DRV_LOG(ERR, "Failed to set rss context table");
- return -EFAULT;
+ return -EIO;
}
return 0;
@@ -891,7 +891,7 @@ int hinic_get_rss_type(void *hwdev, u32 tmpl_idx, struct nic_rss_type *rss_type)
PMD_DRV_LOG(ERR,
"Failed to get hash type, err: %d, status: 0x%x, out size: 0x%x",
err, ctx_tbl.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
rss_type->ipv4 = HINIC_RSS_TYPE_GET(ctx_tbl.context, IPV4);
@@ -931,7 +931,7 @@ int hinic_rss_set_template_tbl(void *hwdev, u32 tmpl_idx, u8 *temp)
PMD_DRV_LOG(ERR,
"Failed to set hash key, err: %d, status: 0x%x, out size: 0x%x",
err, temp_key.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -959,7 +959,7 @@ int hinic_rss_get_template_tbl(void *hwdev, u32 tmpl_idx, u8 *temp)
if (err || !out_size || temp_key.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to get hash key, err: %d, status: 0x%x, out size: 0x%x",
err, temp_key.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
memcpy(temp, temp_key.key, HINIC_RSS_KEY_SIZE);
@@ -1004,7 +1004,7 @@ int hinic_rss_set_hash_engine(void *hwdev, u8 tmpl_idx, u8 type)
if (err || !out_size || hash_type.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to get hash engine, err: %d, status: 0x%x, out size: 0x%x",
err, hash_type.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -1055,7 +1055,7 @@ int hinic_rss_set_indir_tbl(void *hwdev, u32 tmpl_idx, u32 *indir_table)
cmd_buf, &out_param, 0);
if (err || out_param != 0) {
PMD_DRV_LOG(ERR, "Failed to set rss indir table");
- err = -EFAULT;
+ err = -EIO;
goto free_buf;
}
@@ -1069,7 +1069,7 @@ int hinic_rss_set_indir_tbl(void *hwdev, u32 tmpl_idx, u32 *indir_table)
cmd_buf, &out_param, 0);
if (err || out_param != 0) {
PMD_DRV_LOG(ERR, "Failed to set rss indir table");
- err = -EFAULT;
+ err = -EIO;
}
free_buf:
@@ -1101,7 +1101,7 @@ int hinic_rss_get_indir_tbl(void *hwdev, u32 tmpl_idx, u32 *indir_table)
if (err || !out_size || rss_cfg.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to get indir table, err: %d, status: 0x%x, out size: 0x%x",
err, rss_cfg.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
hinic_be32_to_cpu(rss_cfg.indir, HINIC_RSS_INDIR_SIZE);
@@ -1139,7 +1139,7 @@ int hinic_rss_cfg(void *hwdev, u8 rss_en, u8 tmpl_idx, u8 tc_num, u8 *prio_tc)
if (err || !out_size || rss_cfg.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to set rss cfg, err: %d, status: 0x%x, out size: 0x%x",
err, rss_cfg.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -1180,7 +1180,7 @@ int hinic_rss_template_alloc(void *hwdev, u8 *tmpl_idx)
if (err || !out_size || template_mgmt.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to alloc rss template, err: %d, status: 0x%x, out size: 0x%x",
err, template_mgmt.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
*tmpl_idx = template_mgmt.template_id;
@@ -1223,7 +1223,7 @@ int hinic_rss_template_free(void *hwdev, u8 tmpl_idx)
if (err || !out_size || template_mgmt.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to free rss template, err: %d, status: 0x%x, out size: 0x%x",
err, template_mgmt.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -1296,7 +1296,7 @@ int hinic_set_rx_mode(void *hwdev, u32 enable)
if (err || !out_size || rx_mode_cfg.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to set rx mode, err: %d, status: 0x%x, out size: 0x%x",
err, rx_mode_cfg.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -1334,7 +1334,7 @@ int hinic_get_mgmt_version(void *hwdev, char *fw)
if (err || !out_size || fw_ver.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to get mgmt version, err: %d, status: 0x%x, out size: 0x%x\n",
err, fw_ver.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
snprintf(fw, HINIC_MGMT_VERSION_MAX_LEN, "%s", fw_ver.ver);
@@ -1365,7 +1365,7 @@ int hinic_set_rx_csum_offload(void *hwdev, u32 en)
PMD_DRV_LOG(ERR,
"Failed to set rx csum offload, err: %d, status: 0x%x, out size: 0x%x",
err, rx_csum_cfg.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -1395,7 +1395,7 @@ int hinic_set_rx_lro(void *hwdev, u8 ipv4_en, u8 ipv6_en, u8 max_wqe_num)
if (err || !out_size || lro_cfg.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to set lro offload, err: %d, status: 0x%x, out size: 0x%x",
err, lro_cfg.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -1427,7 +1427,7 @@ int hinic_set_anti_attack(void *hwdev, bool enable)
PMD_DRV_LOG(ERR, "Can't %s port Anti-Attack rate limit, err: %d, status: 0x%x, out size: 0x%x",
(enable ? "enable" : "disable"), err,
rate.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -1450,7 +1450,7 @@ int hinic_reset_port_link_cfg(void *hwdev)
if (err || !out_size || reset_cfg.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Reset port link configure failed, err: %d, status: 0x%x, out size: 0x%x",
err, reset_cfg.mgmt_msg_head.status, out_size);
- return -EFAULT;
+ return -EIO;
}
return 0;
@@ -1556,7 +1556,7 @@ int hinic_clear_vport_stats(struct hinic_hwdev *hwdev)
if (err || !out_size || clear_vport_stats.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to clear vport statistics, err: %d, status: 0x%x, out size: 0x%x",
err, clear_vport_stats.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -1586,7 +1586,7 @@ int hinic_clear_phy_port_stats(struct hinic_hwdev *hwdev)
PMD_DRV_LOG(ERR, "Failed to clear phy port statistics, err: %d, status: 0x%x, out size: 0x%x",
err, clear_phy_port_stats.mgmt_msg_head.status,
out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -1623,7 +1623,7 @@ int hinic_set_link_status_follow(void *hwdev,
PMD_DRV_LOG(ERR,
"Failed to set link status follow phy port status, err: %d, status: 0x%x, out size: 0x%x",
err, follow.mgmt_msg_head.status, out_size);
- return -EFAULT;
+ return -EIO;
}
return follow.mgmt_msg_head.status;
@@ -1649,7 +1649,7 @@ int hinic_get_link_mode(void *hwdev, u32 *supported, u32 *advertised)
PMD_DRV_LOG(ERR,
"Failed to get link mode, err: %d, status: 0x%x, out size: 0x%x",
err, link_mode.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
*supported = link_mode.supported;
@@ -1690,7 +1690,7 @@ int hinic_set_xsfp_tx_status(void *hwdev, bool enable)
"Failed to %s port xsfp status, err: %d, status: 0x%x, out size: 0x%x\n",
enable ? "Disable" : "Enable", err,
xsfp_status.mgmt_msg_head.status, out_size);
- return -EFAULT;
+ return -EIO;
}
return 0;
@@ -1723,7 +1723,7 @@ int hinic_flush_qp_res(void *hwdev)
if (err || !out_size || qp_res.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Failed to clear sq resources, err: %d, status: 0x%x, out size: 0x%x",
err, qp_res.mgmt_msg_head.status, out_size);
- return -EINVAL;
+ return -EIO;
}
return 0;
@@ -1808,7 +1808,7 @@ int hinic_set_fdir_filter(void *hwdev, u8 filter_type, u8 qid, u8 type_enable,
" enable: 0x%x, qid: 0x%x, filter_type_enable: 0x%x\n",
err, port_filer_cmd.mgmt_msg_head.status, out_size,
filter_type, enable, qid, type_enable);
- return -EFAULT;
+ return -EIO;
}
return 0;
@@ -1860,7 +1860,7 @@ int hinic_set_normal_filter(void *hwdev, u8 qid, u8 normal_type_enable,
" enable: 0x%x, qid: 0x%x, normal_type_enable: 0x%x, key:0x%x\n",
err, port_filer_cmd.mgmt_msg_head.status, out_size,
flag, enable, qid, normal_type_enable, key);
- return -EFAULT;
+ return -EIO;
}
return 0;
@@ -1908,7 +1908,7 @@ int hinic_set_fdir_tcam(void *hwdev, u16 type_mask,
if (err || !out_size || port_tcam_cmd.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Set tcam table failed, err: %d, status: 0x%x, out size: 0x%x",
err, port_tcam_cmd.mgmt_msg_head.status, out_size);
- return -EFAULT;
+ return -EIO;
}
return 0;
@@ -1945,7 +1945,7 @@ int hinic_clear_fdir_tcam(void *hwdev, u16 type_mask)
if (err || !out_size || port_tcam_cmd.mgmt_msg_head.status) {
PMD_DRV_LOG(ERR, "Clear tcam table failed, err: %d, status: 0x%x, out size: 0x%x",
err, port_tcam_cmd.mgmt_msg_head.status, out_size);
- return -EFAULT;
+ return -EIO;
}
return 0;
@@ -1964,7 +1964,7 @@ int hinic_add_tcam_rule(void *hwdev, struct tag_tcam_cfg_rule *tcam_rule)
if (tcam_rule->index >= HINIC_MAX_TCAM_RULES_NUM) {
PMD_DRV_LOG(ERR, "Tcam rules num to add is invalid");
- return -EFAULT;
+ return -EINVAL;
}
memset(&tcam_cmd, 0, sizeof(struct tag_fdir_add_rule_cmd));
@@ -1979,7 +1979,7 @@ int hinic_add_tcam_rule(void *hwdev, struct tag_tcam_cfg_rule *tcam_rule)
PMD_DRV_LOG(ERR,
"Add tcam rule failed, err: %d, status: 0x%x, out size: 0x%x",
err, tcam_cmd.mgmt_msg_head.status, out_size);
- return -EFAULT;
+ return -EIO;
}
return 0;
@@ -1998,7 +1998,7 @@ int hinic_del_tcam_rule(void *hwdev, u32 index)
if (index >= HINIC_MAX_TCAM_RULES_NUM) {
PMD_DRV_LOG(ERR, "Tcam rules num to del is invalid");
- return -EFAULT;
+ return -EINVAL;
}
memset(&tcam_cmd, 0, sizeof(struct tag_fdir_del_rule_cmd));
@@ -2013,7 +2013,7 @@ int hinic_del_tcam_rule(void *hwdev, u32 index)
PMD_DRV_LOG(ERR,
"Del tcam rule failed, err: %d, status: 0x%x, out size: 0x%x",
err, tcam_cmd.mgmt_msg_head.status, out_size);
- return -EFAULT;
+ return -EIO;
}
return 0;
@@ -2057,7 +2057,7 @@ static int hinic_mgmt_tcam_block(void *hwdev, u8 alloc_en,
PMD_DRV_LOG(ERR,
"Set tcam block failed, err: %d, status: 0x%x, out size: 0x%x",
err, tcam_block_info.mgmt_msg_head.status, out_size);
- return -EFAULT;
+ return -EIO;
}
if (alloc_en)
@@ -2108,7 +2108,7 @@ int hinic_flush_tcam_rule(void *hwdev)
PMD_DRV_LOG(ERR,
"Flush tcam fdir rules failed, err: %d, status: 0x%x, out size: 0x%x",
err, tcam_flush.mgmt_msg_head.status, out_size);
- err = -EFAULT;
+ err = -EIO;
}
return err;
@@ -2143,7 +2143,7 @@ int hinic_set_fdir_tcam_rule_filter(void *hwdev, bool enable)
"status: 0x%x, out size: 0x%x, enable: 0x%x",
err, port_tcam_cmd.mgmt_msg_head.status, out_size,
enable);
- return -EFAULT;
+ return -EIO;
}
if (port_tcam_cmd.mgmt_msg_head.status == HINIC_MGMT_CMD_UNSUPPORTED) {
diff --git a/drivers/net/hinic/base/hinic_pmd_nicio.c b/drivers/net/hinic/base/hinic_pmd_nicio.c
index 7ec3e4c..2914e99 100644
--- a/drivers/net/hinic/base/hinic_pmd_nicio.c
+++ b/drivers/net/hinic/base/hinic_pmd_nicio.c
@@ -314,7 +314,7 @@ static int init_sq_ctxts(struct hinic_nic_io *nic_io)
if (err || out_param != 0) {
PMD_DRV_LOG(ERR, "Failed to set SQ ctxts, err: %d",
err);
- err = -EFAULT;
+ err = -EIO;
break;
}
@@ -371,7 +371,7 @@ static int init_rq_ctxts(struct hinic_nic_io *nic_io)
cmd_buf, &out_param, 0);
if ((err) || out_param != 0) {
PMD_DRV_LOG(ERR, "Failed to set RQ ctxts");
- err = -EFAULT;
+ err = -EIO;
break;
}
@@ -422,7 +422,7 @@ static int clean_queue_offload_ctxt(struct hinic_nic_io *nic_io,
if ((err) || (out_param)) {
PMD_DRV_LOG(ERR, "Failed to clean queue offload ctxts");
- err = -EFAULT;
+ err = -EIO;
}
hinic_free_cmd_buf(hwdev, cmd_buf);
--
1.8.3.1
next prev parent reply other threads:[~2020-06-27 3:33 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-27 3:55 [dpdk-dev] [PATCH v1 0/5] fix promisc and tcam problem Xiaoyun wang
2020-06-27 3:55 ` [dpdk-dev] [PATCH v1 1/5] net/hinic/base: add out parameter for mgmt sync channel Xiaoyun wang
2020-06-27 3:55 ` [dpdk-dev] [PATCH v1 2/5] net/hinic/base: remove unused parameter Xiaoyun wang
2020-06-27 3:55 ` [dpdk-dev] [PATCH v1 3/5] net/hinic: fix setting promiscuous mode problem Xiaoyun wang
2020-06-27 3:55 ` [dpdk-dev] [PATCH v1 4/5] net/hinic: add tcam filter switch for FDIR Xiaoyun wang
2020-06-27 3:55 ` Xiaoyun wang [this message]
2020-07-07 19:42 ` [dpdk-dev] [PATCH v1 5/5] net/hinic/base: modify return errors Ferruh Yigit
2020-07-03 16:22 ` [dpdk-dev] [PATCH v1 0/5] fix promisc and tcam problem Ferruh Yigit
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=fe8c40d266837ffc8a145afc476115c89ad167a1.1593228634.git.cloud.wangxiaoyun@huawei.com \
--to=cloud.wangxiaoyun@huawei.com \
--cc=bluca@debian.org \
--cc=david.yangxiaoliang@huawei.com \
--cc=dev@dpdk.org \
--cc=ferruh.yigit@intel.com \
--cc=luoxianjun@huawei.com \
--cc=luoxingyu@huawei.com \
--cc=yin.yinshi@huawei.com \
--cc=zhaohui8@huawei.com \
--cc=zhengjingzhou@huawei.com \
--cc=zhouguoyang@huawei.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).