[dpdk-dev] [dpdk-stable] [PATCH] eal: fix positive error codes from probe/remove

Ilya Maximets i.maximets at samsung.com
Thu Jun 6 10:39:09 CEST 2019


On 03.06.2019 19:13, David Marchand wrote:
> 
> 
> On Mon, Jun 3, 2019 at 5:37 PM Ilya Maximets <i.maximets at samsung.com <mailto:i.maximets at samsung.com>> wrote:
> 
>     On 03.06.2019 11:50, David Marchand wrote:
>     >
>     >
>     > On Thu, May 30, 2019 at 3:26 PM Ilya Maximets <i.maximets at samsung.com <mailto:i.maximets at samsung.com> <mailto:i.maximets at samsung.com <mailto:i.maximets at samsung.com>>> wrote:
>     >
>     >     According to API, 'rte_dev_probe()' and 'rte_dev_remove()' and their
>     >     'hotplug' equivalents must return 0 or negative error code. Bus code
>     >
>     >
>     > About this first part, existing callers in dpdk are not consistent with the api which might explain why this was not seen earlier.
>     > How about fixing the existing callers?
> 
>     Do you mean replacing all the 'rte_dev_probe() != 0' with 'rte_dev_probe() < 0'
>     around the codebase?
> 
> 
> Yes.
> It is not necessary to this patch so I can handle it if you don't have time.
> But dpdk should show a good example by respecting its own apis description.

I agree. I'll send v2 with fixed users.

> The problem is in local_dev_probe() (resp. local_dev_remove()) itself, since
> this internal api announces it should return < 0 on error.

Hmm. I missed that private internal API defined for local_* functions.
I'll move the check from rte_dev_probe() to local_dev_probe().

Best regards, Ilya Maximets.


More information about the dev mailing list