Re: [patch] mm, memcg: store memcg name for oom kill log consistency

2013-09-09 Thread Michal Hocko
On Mon 09-09-13 02:00:26, David Rientjes wrote: > On Thu, 5 Sep 2013, Michal Hocko wrote: [...] > > Reported-by: David Rientjes > > Remove this. OK. Is there any other way how to give you a credit for discovering/reporting this issue? -- Michal Hocko SUSE Labs -- To unsubscribe from this list:

Re: [patch] mm, memcg: store memcg name for oom kill log consistency

2013-09-09 Thread David Rientjes
On Thu, 5 Sep 2013, Michal Hocko wrote: > From 4cee36f56100f5689fe1ae22f468016ce5a0cbae Mon Sep 17 00:00:00 2001 > From: Michal Hocko > Date: Thu, 5 Sep 2013 15:39:20 +0200 > Subject: [PATCH] memcg, oom: lock mem_cgroup_print_oom_info > > mem_cgroup_print_oom_info uses a static buffer (memcg_nam

Re: [patch] mm, memcg: store memcg name for oom kill log consistency

2013-09-05 Thread Michal Hocko
On Thu 29-08-13 15:30:32, Michal Hocko wrote: > On Wed 28-08-13 23:03:54, David Rientjes wrote: > > A shared buffer is currently used for the name of the oom memcg and the > > memcg of the killed process. There is no serialization of memcg oom > > kills, so this buffer can easily be overwritten if

Re: [patch] mm, memcg: store memcg name for oom kill log consistency

2013-08-29 Thread Michal Hocko
On Wed 28-08-13 23:03:54, David Rientjes wrote: > A shared buffer is currently used for the name of the oom memcg and the > memcg of the killed process. There is no serialization of memcg oom > kills, so this buffer can easily be overwritten if there is a concurrent > oom kill in another memcg. R

[patch] mm, memcg: store memcg name for oom kill log consistency

2013-08-28 Thread David Rientjes
A shared buffer is currently used for the name of the oom memcg and the memcg of the killed process. There is no serialization of memcg oom kills, so this buffer can easily be overwritten if there is a concurrent oom kill in another memcg. This patch stores the names of the memcgs directly in str