19/02/2023 11:04, Thomas Monjalon:
> 29/01/2023 13:35, Zhang, Qi Z:
> > From: Zhang, Qi Z <qi.z.zh...@intel.com>
> > >
> > > PMD not announce pmd_supports_disable_iova_as_pa will not be build
> > > when RTE_IOVA_AS_PA is not defined, but some AVX library for vector path
> > > is not skipped by the build system which cause compile error.
> > >
> > > The patch modify i40e, iavf, ice's meson file to skip AVX library build
> > > when
> > > RTE_IOVA_AS_PA is not defined.
> > >
> > > Cc: sta...@dpdk.org
> > >
> > > Signed-off-by: Qi Zhang <qi.z.zh...@intel.com>
> >
> > Move this for next-net review.
> >
> > Acked-by: Qi Zhang <qi.z.zh...@intel.com>
>
> You ack your own patch?
>
> > Applied to dpdk-next-net-intel.
>
> Sorry I don't pull this one, as I think there is a better fix:
> we should not handle this option in each driver.
> Instead the file drivers/meson.build must be fixed.
No sorry, it cannot be handled in drivers/meson.build.
So I suggest disabling the whole driver:
+if not get_option('enable_iova_as_pa')
+ subdir_done()
+endif
Note: no need to disable ice, as it is enabled in another commit.
