]> git.karo-electronics.de Git - karo-tx-linux.git/commit
vmscan.c: fix invalid strict_strtoul() check in write_scan_unevictable_node()
authorKautuk Consul <consul.kautuk@gmail.com>
Wed, 5 Oct 2011 00:43:13 +0000 (11:43 +1100)
committerStephen Rothwell <sfr@canb.auug.org.au>
Thu, 6 Oct 2011 05:59:49 +0000 (16:59 +1100)
commitf02fb72ca3dc4e88ce926974fe62e67b3ce1f7d6
tree6943e5b8160faf1fc4d56338d77708b16b0a1f21
parent4b711abd7ff8e805786ebcdff84afbfb69cc0c7e
vmscan.c: fix invalid strict_strtoul() check in write_scan_unevictable_node()

write_scan_unevictable_node() checks the value req returned by
strict_strtoul() and returns 1 if req is 0.

However, when strict_strtoul() returns 0, it means successful conversion
of buf to unsigned long.

Due to this, the function was not proceeding to scan the zones for
unevictable pages even though we write a valid value to the
scan_unevictable_pages sys file.

Change this check slightly to check for invalid value in buf as well as 0
value stored in res after successful conversion via strict_strtoul.  In
both cases, we do not perform the scanning of this node's zones.

Signed-off-by: Kautuk Consul <consul.kautuk@gmail.com>
Reviewed-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Cc: Johannes Weiner <jweiner@redhat.com>
Cc: Lee Schermerhorn <lee.schermerhorn@hp.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
mm/vmscan.c