On Wed, Apr 20, 2011 at 8:50 PM, KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com> wrote:

Ying, I noticed this at test. please fix the code in your set.
==
if low_wmark_distance = 0, mem_cgroup_watermark_ok() returns
false when usage hits limit.

Signed-off-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
---
 mm/memcontrol.c |    3 +++
 1 file changed, 3 insertions(+)

Index: mmotm-Apr14/mm/memcontrol.c
===================================================================
--- mmotm-Apr14.orig/mm/memcontrol.c
+++ mmotm-Apr14/mm/memcontrol.c
@@ -5062,6 +5062,9 @@ int mem_cgroup_watermark_ok(struct mem_c
       long ret = 0;
       int flags = CHARGE_WMARK_LOW | CHARGE_WMARK_HIGH;

+       if (!mem->low_wmark_distance)
+               return 1;
+
       VM_BUG_ON((charge_flags & flags) == flags);

       if (charge_flags & CHARGE_WMARK_LOW)

Thanks. Will add this in the next post.

--Ying