On Fri, 19 Feb 2021, Christian Eggers wrote:

On Friday, 19 February 2021, 17:21:21 CET, John Kacur wrote:

quoted

On Fri, 19 Feb 2021, Daniel Wagner wrote:

quoted

On Fri, Feb 19, 2021 at 04:21:46PM +0100, Christian Eggers wrote:

quoted

CONFIG_NUMA is not available on ARM32:

libnuma is not happy if the kernel doesn't have the config
option. Well, in this case we just revert the patch I suppose.

Yeah, I'm not happy with this.
To be sure Daniel did a lot of good work cleaning-up some of the numa 
calls and cleaning up the somewhat artificial smp / numa divide.

That work needs to continue, but I want the ability to build cyclictest
without NUMA, and that also means we need the small helpers.

Just for clarification: Does this mean that cyclictest shall link against
libnuma (libnuma can be built on ARM without problems), but not call it
functions at runtime?

From the numa.h:

/* NUMA support available. If this returns a negative value all other function
   in this library are undefined. */
int numa_available(void);

On the other hand, this is what was working for you before the changes 
right? We required libnuma for building but not for runtime?

That would be easier to maintain.

regards,
Christian