On 7/14/2017 10:12 PM, Jan Blunck wrote: > Let the rte_eal_devargs_parse() function explicitly take a "busname" > argument that is validated. > > Now that the busname is known and validated at parse time the validity of > the device name is checked for all device types when they get probed. > > Signed-off-by: Jan Blunck <jblu...@infradead.org>
<...> > --- a/test/test/test_devargs.c > +++ b/test/test/test_devargs.c > @@ -110,24 +110,17 @@ test_devargs(void) > goto fail; > free_devargs_list(); > > - /* test error case: bad PCI address */ > - if (rte_eal_devargs_add(RTE_DEVTYPE_WHITELISTED_PCI, "08:1") == 0) > - goto fail; > - if (rte_eal_devargs_add(RTE_DEVTYPE_WHITELISTED_PCI, "00.1") == 0) > - goto fail; > - if (rte_eal_devargs_add(RTE_DEVTYPE_WHITELISTED_PCI, "foo") == 0) > - goto fail; > - if (rte_eal_devargs_add(RTE_DEVTYPE_WHITELISTED_PCI, ",") == 0) > - goto fail; > - if (rte_eal_devargs_add(RTE_DEVTYPE_WHITELISTED_PCI, "000f:0:0") == 0) > - goto fail; Why removed these cases? <...>
