19/06/2017 13:10, Hemant Agrawal: > >>> On Thu, Jun 08, 2017 at 02:21:58PM +0300, Ilya Maximets wrote: > >>>> So, there are 2 option: > >>>> > >>>> 1. Return back config option RTE_LIBRTE_EAL_NUMA_AWARE_HUGEPAGES > >>>> from the first version of the patch and disable it by default. > >>>> > >>>> 2. Keep patch as it is now and make everyone install libnuma > >>>> for successful build. > > > +1 for option 1 > It will be a issue and undesired dependency for SoCs, not supporting > NUMA architecture. > > It can be added to the config, who desired to use it by default.
Yes I agree, it cannot be a dependency for architectures which do not support NUMA. Please can we rework the patch so that only one node is assumed if NUMA is disabled for the architecture?
