From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752964AbaIYNy6 (ORCPT ); Thu, 25 Sep 2014 09:54:58 -0400 Received: from gum.cmpxchg.org ([85.214.110.215]:35697 "EHLO gum.cmpxchg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751883AbaIYNy5 (ORCPT ); Thu, 25 Sep 2014 09:54:57 -0400 Date: Thu, 25 Sep 2014 09:54:50 -0400 From: Johannes Weiner To: Michal Hocko Cc: Andrew Morton , Hugh Dickins , Tejun Heo , linux-mm@kvack.org, cgroups@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [patch v2] mm: memcontrol: do not iterate uninitialized memcgs Message-ID: <20140925135450.GA1822@cmpxchg.org> References: <1411612278-4707-1-git-send-email-hannes@cmpxchg.org> <20140925024054.GA4888@cmpxchg.org> <20140925114339.GD12090@dhcp22.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20140925114339.GD12090@dhcp22.suse.cz> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Sep 25, 2014 at 01:43:39PM +0200, Michal Hocko wrote: > On Wed 24-09-14 22:40:55, Johannes Weiner wrote: > > Argh, buggy css_put() against the root. Hand grenades, everywhere. > > Update: > > > > --- > > From 9b0b4d72d71cd8acd7aaa58d2006c751decc8739 Mon Sep 17 00:00:00 2001 > > From: Johannes Weiner > > Date: Wed, 24 Sep 2014 22:00:20 -0400 > > Subject: [patch] mm: memcontrol: do not iterate uninitialized memcgs > > > > The cgroup iterators yield css objects that have not yet gone through > > css_online(), but they are not complete memcgs at this point and so > > the memcg iterators should not return them. d8ad30559715 ("mm/memcg: > > iteration skip memcgs not yet fully initialized") set out to implement > > exactly this, but it uses CSS_ONLINE, a cgroup-internal flag that does > > not meet the ordering requirements for memcg, and so we still may see > > partially initialized memcgs from the iterators. > > I do not see how would this happen. CSS_ONLINE is set after css_online > callback returns and mem_cgroup_css_online ends the core initialization > with mutex_unlock which should provide sufficient memory ordering > requirements But the iterators do not use the mutex? We are missing the matching acquire for the proper ordering.