diff options
author | Han Pingtian <hanpt@linux.vnet.ibm.com> | 2014-01-23 23:53:17 (GMT) |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-01-24 00:36:52 (GMT) |
commit | da8c757b080ee84f219fa2368cb5dd23ac304fc0 (patch) | |
tree | c6a351683151f272a488e0fc379653a1b259e64f /Documentation/hwmon/ad7314 | |
parent | cc81717ed3bc6d4f3738d13a1e097437caada0e9 (diff) | |
download | linux-da8c757b080ee84f219fa2368cb5dd23ac304fc0.tar.xz |
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/hwmon/ad7314')
0 files changed, 0 insertions, 0 deletions