eal: simplify parameters of hotplug functions
authorThomas Monjalon <thomas@monjalon.net>
Fri, 7 Sep 2018 22:27:27 +0000 (00:27 +0200)
committerThomas Monjalon <thomas@monjalon.net>
Thu, 11 Oct 2018 12:09:24 +0000 (14:09 +0200)
commit911462eb4a5fc38b4e3581d167a1541207a69d5c
tree4a50445c513930762a1dce3aa278cfeb9b7b73d7
parent6878cd397d20d65991e6fcd3a3bcc39323a692cf
eal: simplify parameters of hotplug functions

All information about a device to probe can be grouped
in a common string, which is what we usually call devargs.
An application should not have to parse this string before
calling the EAL probe function.
And the syntax could evolve to be more complex and support
matching multiple devices in one string.
That's why the bus name and device name should be removed from
rte_eal_hotplug_add().
Instead of changing this function, a simpler one is added
and used in the old one, which may be deprecated later.

When removing a device, we already know its rte_device handle
which can be directly passed as parameter of rte_eal_hotplug_remove().
If the rte_device is not known, it can be retrieved with the devargs,
by iterating in the device list (future RTE_DEV_FOREACH()).
Similarly to the probing case, a new function is added
and used in the old one, which may be deprecated later.
The new function is used in failsafe, because the replacement is easy.

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
Reviewed-by: Andrew Rybchenko <arybchenko@solarflare.com>
Acked-by: Gaetan Rivet <gaetan.rivet@6wind.com>
Reviewed-by: Stephen Hemminger <stephen@networkplumber.org>
drivers/net/failsafe/failsafe_eal.c
drivers/net/failsafe/failsafe_ether.c
lib/librte_eal/common/eal_common_dev.c
lib/librte_eal/common/include/rte_dev.h
lib/librte_eal/rte_eal_version.map