On Sun, 25 Sep 2011 16:29:40 +0530 Kautuk Consul <consul.kautuk@xxxxxxxxx> wrote: > write_scan_unavictable_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. > > Changing this if 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@xxxxxxxxx> Reviewed-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx> -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>