Re: [PATCH v8 2/9] foundations of per-cgroup memory pressure controlling.

From: Glauber Costa
Date: Fri Dec 09 2011 - 07:41:39 EST


On 12/08/2011 11:24 PM, KAMEZAWA Hiroyuki wrote:
On Mon, 5 Dec 2011 19:34:56 -0200
Glauber Costa<glommer@xxxxxxxxxxxxx> wrote:

This patch replaces all uses of struct sock fields' memory_pressure,
memory_allocated, sockets_allocated, and sysctl_mem to acessor
macros. Those macros can either receive a socket argument, or a mem_cgroup
argument, depending on the context they live in.

Since we're only doing a macro wrapping here, no performance impact at all is
expected in the case where we don't have cgroups disabled.

Signed-off-by: Glauber Costa<glommer@xxxxxxxxxxxxx>
CC: David S. Miller<davem@xxxxxxxxxxxxx>
CC: Hiroyouki Kamezawa<kamezawa.hiroyu@xxxxxxxxxxxxxx>
CC: Eric W. Biederman<ebiederm@xxxxxxxxxxxx>
CC: Eric Dumazet<eric.dumazet@xxxxxxxxx>

please get ack from network guys.
from me.
Reviewed-by: KAMEZAWA Hiroyuki<kamezawa.hiroyu@xxxxxxxxxxxxxx>

Ok.

I think that now with all the Reviewed-by:'s I will resend it as a Request for Inclusion for Dave (plus fixing the problem you noted in patch 1)
--
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/