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.
fslmc_vfio_get_bus_info() also issued mc_get_soc_version() twice in a row into the same structure, so the second command only overwrote the first result with an identical one. Keep a single query. Signed-off-by: Hemant Agrawal <[email protected]> --- drivers/bus/fslmc/fslmc_vfio.c | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/drivers/bus/fslmc/fslmc_vfio.c b/drivers/bus/fslmc/fslmc_vfio.c index 4eb8552696..5ccb457a75 100644 --- a/drivers/bus/fslmc/fslmc_vfio.c +++ b/drivers/bus/fslmc/fslmc_vfio.c @@ -281,13 +281,6 @@ fslmc_vfio_get_bus_info(void *regs, struct rte_fslmc_bus_info *bus_info) return ret; } - memset(&mc_plat_info, 0, sizeof(struct mc_soc_version)); - ret = mc_get_soc_version(&mc_io, CMD_PRI_LOW, &mc_plat_info); - if (ret) { - DPAA2_BUS_ERR("Unable to get SoC version err:%d", ret); - return ret; - } - for (i = 0; i < RTE_DIM(s_soc_type); i++) { if ((mc_plat_info.svr & DPAA2_SVR_MASK) == s_soc_type[i].svr) { dpaa2_svr_family = s_soc_type[i].svr; @@ -1041,8 +1034,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); -- 2.43.0
