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