d/numam-dpdk
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

net/iavf: optimize promiscuous device operations

Browse Source 
This patch is to improve efficiency and eliminate code
redundancy of promiscuous ops.

Signed-off-by: Robin Zhang <robinx.zhang@intel.com>
Acked-by: Qi Zhang <qi.z.zhang@intel.com>
This commit is contained in:
Robin Zhang 2020-09-27 07:26:22 +00:00 committed by Ferruh Yigit
parent eb9444341f
commit 1e4d55a7fe
2 changed files with 19 additions and 43 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

49
drivers/net/iavf/iavf_ethdev.c
View File

@ -676,17 +676,9 @@ iavf_dev_promiscuous_enable(struct rte_eth_dev *dev)
struct iavf_adapter *adapter =
IAVF_DEV_PRIVATE_TO_ADAPTER(dev->data->dev_private);
struct iavf_info *vf = IAVF_DEV_PRIVATE_TO_VF(adapter);
int ret;
ret = iavf_config_promisc(adapter, true, vf->promisc_multicast_enabled);
if (!ret)
vf->promisc_unicast_enabled = true;
else if (ret == IAVF_NOT_SUPPORTED)
ret = -ENOTSUP;
else
ret = -EAGAIN;
return ret;
return iavf_config_promisc(adapter,
true, vf->promisc_multicast_enabled);
}
static int
@ -695,18 +687,9 @@ iavf_dev_promiscuous_disable(struct rte_eth_dev *dev)
struct iavf_adapter *adapter =
IAVF_DEV_PRIVATE_TO_ADAPTER(dev->data->dev_private);
struct iavf_info *vf = IAVF_DEV_PRIVATE_TO_VF(adapter);
int ret;
ret = iavf_config_promisc(adapter, false,
vf->promisc_multicast_enabled);
if (!ret)
vf->promisc_unicast_enabled = false;
else if (ret == IAVF_NOT_SUPPORTED)
ret = -ENOTSUP;
else
ret = -EAGAIN;
return ret;
return iavf_config_promisc(adapter,
false, vf->promisc_multicast_enabled);
}
static int
@ -715,17 +698,9 @@ iavf_dev_allmulticast_enable(struct rte_eth_dev *dev)
struct iavf_adapter *adapter =
IAVF_DEV_PRIVATE_TO_ADAPTER(dev->data->dev_private);
struct iavf_info *vf = IAVF_DEV_PRIVATE_TO_VF(adapter);
int ret;
ret = iavf_config_promisc(adapter, vf->promisc_unicast_enabled, true);
if (!ret)
vf->promisc_multicast_enabled = true;
else if (ret == IAVF_NOT_SUPPORTED)
ret = -ENOTSUP;
else
ret = -EAGAIN;
return ret;
return iavf_config_promisc(adapter,
vf->promisc_unicast_enabled, true);
}
static int
@ -734,17 +709,9 @@ iavf_dev_allmulticast_disable(struct rte_eth_dev *dev)
struct iavf_adapter *adapter =
IAVF_DEV_PRIVATE_TO_ADAPTER(dev->data->dev_private);
struct iavf_info *vf = IAVF_DEV_PRIVATE_TO_VF(adapter);
int ret;
ret = iavf_config_promisc(adapter, vf->promisc_unicast_enabled, false);
if (!ret)
vf->promisc_multicast_enabled = false;
else if (ret == IAVF_NOT_SUPPORTED)
ret = -ENOTSUP;
else
ret = -EAGAIN;
return ret;
return iavf_config_promisc(adapter,
vf->promisc_unicast_enabled, false);
}
static int

13
drivers/net/iavf/iavf_vchnl.c
View File

@ -842,10 +842,19 @@ iavf_config_promisc(struct iavf_adapter *adapter,
err = iavf_execute_vf_cmd(adapter, &args);
if (err)
if (err) {
PMD_DRV_LOG(ERR,
"fail to execute command CONFIG_PROMISCUOUS_MODE");
return err;
if (err == IAVF_NOT_SUPPORTED)
return -ENOTSUP;
return -EAGAIN;
}
vf->promisc_unicast_enabled = enable_unicast;
vf->promisc_multicast_enabled = enable_multicast;
return 0;
}
int