Q: Why do we need to bother at all? A: If a node is highly loaded and its memory is significantly fragmented (unfortunately almost any node with serious load has highly fragmented memory) then any high order memory allocation can trigger massive memory shrink and result in quite a big allocation latency. And the node becomes less responsive and users don't like it. The ultimate solution here is to get rid of large allocations, but we need an instrument to detect them. Q: Why warning? Use tracepoints! A: Well, this is a matter of magic defaults. Yes, you can use tracepoints to catch large allocations, but you need to do this on purpose and regularly and this is to be done by every developer which is quite unreal. On the other hand if you develop something and get a warning, you'll have to think about the reason and either succeed with reworking the code to use smaller allocation sizes (and thus decrease allocation latency!) or just use kvmalloc() if you don't really need physically continuos chunk or come to the conclusion you definitely need physically continuos memory and shut up the warning. Q: Why compile time config option? A: In order not to decrease the performance even a bit in case someone does not want to hunt for large allocations. In an ideal life i'd prefer this check/warning is enabled by default and may be even without a config option so it works on every node. Once we find and rework or mark all large allocations that would be good by default. Until that though it will be noisy. Another option is to rework the patch via static keys (having the warning disabled by default surely). That makes it possible to turn on the feature without recompiling the kernel - during testing period for example. If you prefer this way, i would be happy to rework the patch via static keys. Konstantin Khorenko (1): mm/page_alloc: add warning about high order allocations kernel/sysctl.c | 15 +++++++++++++++ mm/Kconfig | 18 ++++++++++++++++++ mm/page_alloc.c | 25 +++++++++++++++++++++++++ 3 files changed, 58 insertions(+) -- 2.15.1
