Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753491AbbLSIvz (ORCPT ); Sat, 19 Dec 2015 03:51:55 -0500 Received: from relay.parallels.com ([195.214.232.42]:51091 "EHLO relay.parallels.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751896AbbLSIvy (ORCPT ); Sat, 19 Dec 2015 03:51:54 -0500 Date: Sat, 19 Dec 2015 11:51:38 +0300 From: Vladimir Davydov To: Andrew Morton CC: Johannes Weiner , , Michal Hocko , , Subject: Re: [PATCH v2] mm: memcontrol: fix possible memcg leak due to interrupted reclaim Message-ID: <20151219085137.GV28521@esperanza> References: <1450182697-11049-1-git-send-email-vdavydov@virtuozzo.com> <20151217150217.a02c264ce9b5335b02bae888@linux-foundation.org> <20151218153202.GS28521@esperanza> <20151218160041.GA4201@cmpxchg.org> <20151218162405.GU28521@esperanza> <20151218144004.6ec6189817b64e04d9405001@linux-foundation.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20151218144004.6ec6189817b64e04d9405001@linux-foundation.org> X-ClientProxiedBy: US-EXCH2.sw.swsoft.com (10.255.249.46) To MSK-EXCH1.sw.swsoft.com (10.67.48.55) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1583 Lines: 40 On Fri, Dec 18, 2015 at 02:40:04PM -0800, Andrew Morton wrote: > On Fri, 18 Dec 2015 19:24:05 +0300 Vladimir Davydov wrote: > > > > > OK, got it, thanks. Here goes the incremental patch (it should also fix > > the warning regarding unused cmpxchg returned value): > > --- > > diff --git a/mm/memcontrol.c b/mm/memcontrol.c > > index fc25dc211eaf..908c075e04eb 100644 > > --- a/mm/memcontrol.c > > +++ b/mm/memcontrol.c > > @@ -864,7 +864,7 @@ struct mem_cgroup *mem_cgroup_iter(struct mem_cgroup *root, > > * might block it. So we clear iter->position right > > * away. > > */ > > - cmpxchg(&iter->position, pos, NULL); > > + (void)cmpxchg(&iter->position, pos, NULL); > > No, this doesn't actually squish the __must_check warning. The warning was caused not by a __must_check annotation - using it for cmpxchg would be just wrong - it was caused by type conversion done in expansion of cmpxchg macro: arch/m68k/include/asm/cmpxchg.h:121:3: warning: value computed is not used [-Wunused-value] ((__typeof__(*(ptr)))__cmpxchg((ptr), (unsigned long)(o), \ ^ (see http://www.spinics.net/lists/linux-mm/msg99133.html) Type conversion to (void) helps suppressing this warning, and it seems this is what is done commonly (e.g. see kernel/rcu/tree_plugin.h) Thanks, Vladimir -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/