>On Fri, Oct 25, 2024 at 03: 20: 40PM +0530, Shijith Thotton wrote: > The value >of RTE_IOVA_IN_MBUF has always been disabled on CNXK platforms, > as >IOVA in the mbuf is unnecessary. This update changes that behavior to > >respect the value > >On Fri, Oct 25, 2024 at 03:20:40PM +0530, Shijith Thotton wrote: >> The value of RTE_IOVA_IN_MBUF has always been disabled on CNXK >platforms, >> as IOVA in the mbuf is unnecessary. This update changes that behavior to >> respect the value set by the user. A warning message will be printed if >> the build is configured to enable IOVA on the CNXK platform. >> >> Signed-off-by: Shijith Thotton <sthot...@marvell.com> --- v2: - Don't >> override the default value of RTE_IOVA_IN_MBUF in CNXK platform. - Print >> a warning message if IOVA in mbuf is enabled on CNXK platform. >> >> config/arm/meson.build | 8 ++------ >> drivers/common/cnxk/meson.build | 6 ++++++ 2 files changed, 8 >> insertions(+), 6 deletions(-) >> >I think you probably need some doc updates for this. The current doc (cnxk >platform guide) says "Meson build option enable_iova_as_pa is disabled on >CNXK platforms." This is no longer true. Instead, you probably need to put >- in more than one place in the docs, I suggest - a note telling the user >that the option should be disabled. >
Thanks, will update the documentation.
