Hello! I've found a small error in .../drivers/net/bonding/rte_eth_bond_api.c : Func rte_eth_bond_create: ret = rte_vdev_init(name, devargs); if (ret) return -ENOMEM; So, we get ENOMEM error regardless of the actual error code. This code would be more applicable. ret = rte_vdev_init(name, devargs); if (ret) return ret; Yours sincerely, Anatoly Dubinsky.
Hi, Anatoly Dubinsky
it looks you are right. Maybe you can send a patch to fix it.
Thanks, Xavier
On 2020/6/20 2:12, Анатолий Дубинский wrote:
> Hello!
>
> I've found a small error in .../drivers/net/bonding/rte_eth_bond_api.c :
>
> Func rte_eth_bond_create:
> ret = rte_vdev_init(name, devargs);
> if (ret)
> return -ENOMEM;
>
> So, we get ENOMEM error regardless of the actual error code.
>
> This code would be more applicable.
> ret = rte_vdev_init(name, devargs);
> if (ret)
> return ret;
>
>
> Yours sincerely, Anatoly Dubinsky.
>
>
>
Hello! I've found a small error in .../drivers/net/bonding/rte_eth_bond_api.c : Func rte_eth_bond_create: ret = rte_vdev_init(name, devargs); if (ret) return -ENOMEM; So, we get ENOMEM error regardless of the actual error code. This code would be more applicable. ret = rte_vdev_init(name, devargs); if (ret) return ret; Reported-by: Anatoly Dubinsky, ard@amicon.ru ________________________________ От: Анатолий Дубинский Отправлено: 19 июня 2020 г. 21:12 Кому: dev@dpdk.org Тема: small error in rte_eth_bond_api.c Hello! I've found a small error in .../drivers/net/bonding/rte_eth_bond_api.c : Func rte_eth_bond_create: ret = rte_vdev_init(name, devargs); if (ret) return -ENOMEM; So, we get ENOMEM error regardless of the actual error code. This code would be more applicable. ret = rte_vdev_init(name, devargs); if (ret) return ret; Yours sincerely, Anatoly Dubinsky.