On 19-Mar-2020, at 6:53 PM, Vlastimil Babka [off-list ref] wrote:

On 3/19/20 9:52 AM, Sachin Sant wrote:

quoted

quoted

OK how about this version? It's somewhat ugly, but important is that the fast
path case (c->page exists) is unaffected and another common case (c->page is
NULL, but node is NUMA_NO_NODE) is just one extra check - impossible to avoid at
some point anyway.

I attempted the suggested tests.

Test 1: March 18 linux-next + Patch 1 [1] + Patch  2 [2]

Machine boots fine. numactl o/p after boot:

Great, thanks! Can I add your Tested-by: then?

Sure.
Tested-by: Sachin Sant <redacted>

Thank you for the fix.

Thanks
-Sachin