On Tue, Jul 21, 2020 at 4:09 PM Ferruh Yigit <ferruh.yi...@intel.com> wrote:
> On 7/21/2020 6:47 PM, Ajit Khaparde wrote: > > Add an option in meson build to indicate the memory management scheme. > > bnxt_eem_system is set to false by default. > > > > Signed-off-by: Ajit Khaparde <ajit.khapa...@broadcom.com> > > <...> > > > --- a/meson_options.txt > > +++ b/meson_options.txt > > @@ -2,6 +2,8 @@ > > > > option('armv8_crypto_dir', type: 'string', value: '', > > description: 'path to the armv8_crypto library installation > directory') > > +option('bnxt_eem_system', type: 'boolean', value: false, > > + description: 'EEM system memory support') > > option('disable_drivers', type: 'string', value: '', > > description: 'Comma-separated list of drivers to explicitly > disable.') > > option('drivers_install_subdir', type: 'string', value: > 'dpdk/pmds-<VERSION>', > > > > I think it was a design decision to not have all Make config options to > meson > build, and reduce compile time configuration as much as possible. > > For a while we are pushing back new compile time config options unless > they are > really must, the 'RTE_LIBRTE_BNXT_PMD_SYSTEM' has been merged mostly > because it > has been missed. > > Instead of introducing this PMD level compile time option to meson, can it > be > possible to convert the config to runtime config. In the past we have used > the > device arguments many times for same reason, can something similar be used > here? > Sure. I can take a look at that.