Re: [PATCH v5 0/2] Balanced allocation of hugepages

On 6/14/2017 11:41 AM, Ilya Maximets wrote:

On 08.06.2017 18:44, Sergio Gonzalez Monroy wrote:

quoted

On 08/06/2017 13:14, Bruce Richardson wrote:

quoted

On Thu, Jun 08, 2017 at 02:21:58PM +0300, Ilya Maximets wrote:

quoted

Hi everyone,

I just want to clarify current status of these patches.
As I understand, moving to the new build system (for example,
meson+ninja as proposed[1] by Bruce) is a very long process.
But we have issues with imbalanced memory allocation now, and
IMHO it's better to fix them in a near future.

Latest version (v5) of balanced allocation patches adds linbuma
as general unconditional dependency which conflicts with the
current DPDK policies.

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.

I have no preferences about above options. I'm asking your opinions.

Bruce, Sergio, Thomas, what do you think?

[1] http://dpdk.org/ml/archives/dev/2017-June/067428.html

Best regards, Ilya Maximets.

I would be ok with having libnuma as a dependency, so I think I'd prefer
option 2 to 1, assuming libnuma is available in all major Linux distros.

/Bruce

+1 on option 2 (current patch and libnuma as DPDK dependency).

Sergio

Ok. In this case I'm waiting for review.

And someone need to install libnuma development package in automatic
build test environment. Otherwise there will be constant compilation
test failures like this:
	http://dpdk.org/ml/archives/test-report/2017-June/021437.html

Best regards, Ilya Maximets.

+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.