diff options
author | Han Pingtian <hanpt@linux.vnet.ibm.com> | 2014-01-24 00:53:17 +0100 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-01-24 01:36:52 +0100 |
commit | da8c757b080ee84f219fa2368cb5dd23ac304fc0 (patch) | |
tree | c6a351683151f272a488e0fc379653a1b259e64f /Documentation/trace | |
parent | mm: new_vma_page() cannot see NULL vma for hugetlb pages (diff) | |
download | linux-da8c757b080ee84f219fa2368cb5dd23ac304fc0.tar.xz linux-da8c757b080ee84f219fa2368cb5dd23ac304fc0.zip |
mm: prevent setting of a value less than 0 to min_free_kbytes
If echo -1 > /proc/vm/sys/min_free_kbytes, the system will hang. Changing
proc_dointvec() to proc_dointvec_minmax() in the
min_free_kbytes_sysctl_handler() can prevent this to happen.
mhocko said:
: You can still do echo $BIG_VALUE > /proc/vm/sys/min_free_kbytes and make
: your machine unusable but I agree that proc_dointvec_minmax is more
: suitable here as we already have:
:
: .proc_handler = min_free_kbytes_sysctl_handler,
: .extra1 = &zero,
:
: It used to work properly but then 6fce56ec91b5 ("sysctl: Remove references
: to ctl_name and strategy from the generic sysctl table") has removed
: sysctl_intvec strategy and so extra1 is ignored.
Signed-off-by: Han Pingtian <hanpt@linux.vnet.ibm.com>
Acked-by: Michal Hocko <mhocko@suse.cz>
Acked-by: David Rientjes <rientjes@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'Documentation/trace')
0 files changed, 0 insertions, 0 deletions