From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oleg Nesterov Subject: Re: [PATCH 3/7] memcg: immigrate charges only when a threadgroup leader is moved Date: Fri, 22 May 2015 20:30:42 +0200 Message-ID: <20150522183042.GF26770@redhat.com> References: <20150519121321.GB6203@dhcp22.suse.cz> <20150519212754.GO24861@htj.duckdns.org> <20150520131044.GA28678@dhcp22.suse.cz> <20150520132158.GB28678@dhcp22.suse.cz> <20150520175302.GA7287@redhat.com> <20150520202221.GD14256@dhcp22.suse.cz> <20150521192716.GA21304@redhat.com> <20150522093639.GE5109@dhcp22.suse.cz> <20150522162900.GA8955@redhat.com> <20150522165734.GH5109@dhcp22.suse.cz> Mime-Version: 1.0 Return-path: Content-Disposition: inline In-Reply-To: <20150522165734.GH5109@dhcp22.suse.cz> Sender: owner-linux-mm@kvack.org List-ID: Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Michal Hocko Cc: Tejun Heo , lizefan@huawei.com, cgroups@vger.kernel.org, hannes@cmpxchg.org, linux-mm@kvack.org On 05/22, Michal Hocko wrote: > > On Fri 22-05-15 18:29:00, Oleg Nesterov wrote: > > > > In the likely case (if CLONE_VM without CLONE_THREAD was not used) the > > last for_each_process() in mm_update_next_owner() will find another thread > > from the same group. > > My understanding was that for_each_process will iterate only over > processes (represented by the thread group leaders). Yes. But note the inner for_each_thread() loop. And note that we we need this loop exactly because the leader can be zombie. > How would we get > !group_leader from p->{real_parent->}sibling As for children/siblings we can't get !group_leader, yes. And this is actually not right ;) See the (self-nacked) 2/3 I just sent. > > Oh. I think mm_update_next_owner() needs some cleanups. Perhaps I'll send > > the patch today. > > Please hold on, I have a patch to get rid of the owner altogether. I > will post it sometimes next week. Let's see whether this is a viable > option. If not then we can clean this up. Great. Please ignore 1-3 I already sent. Oleg. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org