From: Hemant Agrawal <[email protected]> Every DMA map logged its virtual, IOVA and physical addresses at NOTICE. An application that maps many memory segments prints one such line per segment during initialisation, which drowns out messages the operator actually needs; the neighbouring mapping paths already use debug level. Lower these to DEBUG in fslmc_vfio.c and clean up a redundant variable assignment while there.
Also add a missing rte_bus_device_is_ignored() check in rte_dpaa2_create_dprc_device() so that devices on the denylist or outside the allowlist are skipped during DPRC population, consistent with the per-device check performed in the main probe loop. Signed-off-by: Hemant Agrawal <[email protected]> --- drivers/bus/fslmc/fslmc_vfio.c | 3 +-- drivers/bus/fslmc/portal/dpaa2_hw_dprc.c | 4 ++++ 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/bus/fslmc/fslmc_vfio.c b/drivers/bus/fslmc/fslmc_vfio.c index 412b70e5ae..adde629ce6 100644 --- a/drivers/bus/fslmc/fslmc_vfio.c +++ b/drivers/bus/fslmc/fslmc_vfio.c @@ -900,8 +900,7 @@ fslmc_map_dma(uint64_t vaddr, rte_iova_t iovaddr, size_t len) fslmc_mem_va2iova = RTE_BAD_IOVA; TAILQ_INSERT_TAIL(&fslmc_memsegs, dmaseg, next); } - DPAA2_BUS_LOG(NOTICE, - "%s(%zx): VA(%" PRIx64 "):IOVA(%" PRIx64 "):PHY(%" PRIx64 ")", + DPAA2_BUS_DEBUG("%s(%zx): VA(%" PRIx64 "):IOVA(%" PRIx64 "):PHY(%" PRIx64 ")", is_io ? "DMA I/O map size" : "DMA MEM map size", len, vaddr, iovaddr, phy); diff --git a/drivers/bus/fslmc/portal/dpaa2_hw_dprc.c b/drivers/bus/fslmc/portal/dpaa2_hw_dprc.c index 868ed646af..02b69d404d 100644 --- a/drivers/bus/fslmc/portal/dpaa2_hw_dprc.c +++ b/drivers/bus/fslmc/portal/dpaa2_hw_dprc.c @@ -52,6 +52,10 @@ rte_dpaa2_create_dprc_device(int vdev_fd __rte_unused, RTE_BUS_FOREACH_DEV(dev, &rte_fslmc_bus) { /** DPRC is always created before it's children are created.*/ dev->container = dprc_node; + + if (rte_bus_device_is_ignored(&rte_fslmc_bus, dev->device.name)) + continue; + if (dev->dev_type == DPAA2_ETH) { int link_state;
