Re: [PATCH v2] mm/memcg: mem_cgroup_resize_xxx_limit can guaranteememcg->res.limit <= memcg->memsw.limit

From: Johannes Weiner
Date: Fri Jul 06 2012 - 05:16:57 EST


On Fri, Jul 06, 2012 at 11:24:15AM +0800, Wanpeng Li wrote:
> From: Wanpeng Li <liwp@xxxxxxxxxxxxxxxxxx>
>
> Changlog:
>
> V2:
> * correct title

Would it be possible to collect all these comment fixes you send out
every other day into a single patch?

> Signed-off-by: Wanpeng Li <liwp.linux@xxxxxxxxx>
> ---
> mm/memcontrol.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/mm/memcontrol.c b/mm/memcontrol.c
> index 4b64fe0..a501660 100644
> --- a/mm/memcontrol.c
> +++ b/mm/memcontrol.c
> @@ -3418,7 +3418,7 @@ static int mem_cgroup_resize_limit(struct mem_cgroup *memcg,
> /*
> * Rather than hide all in some function, I do this in
> * open coded manner. You see what this really does.
> - * We have to guarantee memcg->res.limit < memcg->memsw.limit.
> + * We have to guarantee memcg->res.limit <= memcg->memsw.limit.
> */

It would probably make sense to also remove the first two sentences,
they add nothing of value.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/