Messages in this thread | | | Date | Fri, 9 Dec 2011 10:41:02 -0200 | From | Glauber Costa <> | Subject | Re: [PATCH v8 2/9] foundations of per-cgroup memory pressure controlling. |
| |
On 12/08/2011 11:24 PM, KAMEZAWA Hiroyuki wrote: > On Mon, 5 Dec 2011 19:34:56 -0200 > Glauber Costa<glommer@parallels.com> 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@parallels.com> >> CC: David S. Miller<davem@davemloft.net> >> CC: Hiroyouki Kamezawa<kamezawa.hiroyu@jp.fujitsu.com> >> CC: Eric W. Biederman<ebiederm@xmission.com> >> CC: Eric Dumazet<eric.dumazet@gmail.com> > > please get ack from network guys. > from me. > Reviewed-by: KAMEZAWA Hiroyuki<kamezawa.hiroyu@jp.fujitsu.com> > 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)
| |