On Tue, 24 Mar 2026 at 15:15, Kevin Traynor <[email protected]> wrote: > > diff --git a/drivers/dma/idxd/idxd_bus.c b/drivers/dma/idxd/idxd_bus.c > > index 136ac511ef..00e7e7315c 100644 > > --- a/drivers/dma/idxd/idxd_bus.c > > +++ b/drivers/dma/idxd/idxd_bus.c > > @@ -263,12 +263,8 @@ is_for_this_process_use(struct rte_dsa_device *dev, > > const char *name) > > if (strncmp(name, prefix, prefixlen) == 0 && name[prefixlen] == '_') > > retval = 1; > > > > - if (retval && dsa_bus.bus.conf.scan_mode != RTE_BUS_SCAN_UNDEFINED) { > > - if (dsa_bus.bus.conf.scan_mode == RTE_BUS_SCAN_ALLOWLIST) > > - retval = rte_bus_find_devargs(&dsa_bus.bus, > > dev->device.name) != NULL; > > - else > > - retval = rte_bus_find_devargs(&dsa_bus.bus, > > dev->device.name) == NULL; > > - } > > + if (retval && !rte_bus_is_ignored_device(&dsa_bus.bus, > > dev->device.name)) > > + retval = 1; > > hmm, should this be > > if (retval && rte_bus_is_ignored_device(&dsa_bus.bus, dev->device.name)) > retval = 0; > > > > > return retval; > > }
Indeed, something is wrong with my change. I will go with a if (retval) retval = !rte_bus_is_ignored_device(&dsa_bus.bus, dev->device.name); -- David Marchand
