failsafe: Bug fix for the secondary process RX-TX support
Checks
Commit Message
Remove the vdev args check for secondary process which prevents the
secondary from attaching to the device created by the primary process
via the hotplug framework. This check was removed for other vdevs but
was missed for failsafe.
Fixes: 4852aa8f6e21 ("drivers/net: enable hotplug on secondary process")
Cc: qi.z.zhang@intel.com
Signed-off-by: Kumara Parameshwaran <kumaraparamesh92@gmail.com>
---
drivers/net/failsafe/failsafe.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
Comments
On 11/11/2021 12:44 PM, Kumara Parameshwaran wrote:
> Remove the vdev args check for secondary process which prevents the
> secondary from attaching to the device created by the primary process
> via the hotplug framework. This check was removed for other vdevs but
> was missed for failsafe.
>
> Fixes: 4852aa8f6e21 ("drivers/net: enable hotplug on secondary process")
> Cc: qi.z.zhang@intel.com
>
> Signed-off-by: Kumara Parameshwaran <kumaraparamesh92@gmail.com>
Renamed patch title as:
net/failsafe: fix secondary process probe
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
Applied to dpdk-next-net/main, thanks.
@@ -340,8 +340,7 @@ rte_pmd_failsafe_probe(struct rte_vdev_device *vdev)
INFO("Initializing " FAILSAFE_DRIVER_NAME " for %s",
name);
- if (rte_eal_process_type() == RTE_PROC_SECONDARY &&
- strlen(rte_vdev_device_args(vdev)) == 0) {
+ if (rte_eal_process_type() == RTE_PROC_SECONDARY) {
eth_dev = rte_eth_dev_attach_secondary(name);
if (!eth_dev) {
ERROR("Failed to probe %s", name);