* [dpdk-dev] small error in rte_eth_bond_api.c
@ 2020-06-19 18:12 Анатолий Дубинский
2020-06-22 8:42 ` Wei Hu (Xavier)
2020-07-14 15:14 ` Анатолий Дубинский
0 siblings, 2 replies; 3+ messages in thread
From: Анатолий Дубинский @ 2020-06-19 18:12 UTC (permalink / raw)
To: dev
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.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [dpdk-dev] small error in rte_eth_bond_api.c
2020-06-19 18:12 [dpdk-dev] small error in rte_eth_bond_api.c Анатолий Дубинский
@ 2020-06-22 8:42 ` Wei Hu (Xavier)
2020-07-14 15:14 ` Анатолий Дубинский
1 sibling, 0 replies; 3+ messages in thread
From: Wei Hu (Xavier) @ 2020-06-22 8:42 UTC (permalink / raw)
To: Анатолий
Дубинский
Cc: dev, Chas Williams
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.
>
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [dpdk-dev] small error in rte_eth_bond_api.c
2020-06-19 18:12 [dpdk-dev] small error in rte_eth_bond_api.c Анатолий Дубинский
2020-06-22 8:42 ` Wei Hu (Xavier)
@ 2020-07-14 15:14 ` Анатолий Дубинский
1 sibling, 0 replies; 3+ messages in thread
From: Анатолий Дубинский @ 2020-07-14 15:14 UTC (permalink / raw)
To: dev; +Cc: xavier.huwei
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.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-07-17 10:01 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-19 18:12 [dpdk-dev] small error in rte_eth_bond_api.c Анатолий Дубинский
2020-06-22 8:42 ` Wei Hu (Xavier)
2020-07-14 15:14 ` Анатолий Дубинский
DPDK patches and discussions
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://inbox.dpdk.org/dev/0 dev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 dev dev/ https://inbox.dpdk.org/dev \
dev@dpdk.org
public-inbox-index dev
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://inbox.dpdk.org/inbox.dpdk.dev
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git