Thread (4 messages) flat view 4 messages, 2 authors, 2017-10-17

Re: [PATCH v4] mm, sysctl: make NUMA stats configurable

From: Michal Hocko <mhocko@kernel.org>
Date: 2017-10-17 08:14:02
Also in: linux-fsdevel, linux-mm, lkml

On Tue 17-10-17 16:03:44, kemi wrote:
On 2017年10月17日 15:54, Michal Hocko wrote:
[...]
quoted
So basically any value will enable numa stats. This means that we would
never be able to extend this interface to e.g. auto mode (say value 2).
I guess you meant to check sysctl_vm_numa_stat == ENABLE_NUMA_STAT?
I meant to make it more general other than ENABLE_NUMA_STAT(non 0 is enough), 
but it will make it hard to scale, as you said.
So, it would be like this:
0 -- disable
1 -- enable
other value is invalid.

May add option 2 later for auto if necessary:)
But if you allow to set 2 without EINVAL now then you cannot change it
in future.

-- 
Michal Hocko
SUSE Labs

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help