From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-we0-f173.google.com (mail-we0-f173.google.com [74.125.82.173]) by kanga.kvack.org (Postfix) with ESMTP id DF4D46B0035 for ; Mon, 23 Jun 2014 05:42:19 -0400 (EDT) Received: by mail-we0-f173.google.com with SMTP id t60so6635635wes.32 for ; Mon, 23 Jun 2014 02:42:19 -0700 (PDT) Received: from metis.ext.pengutronix.de (metis.ext.pengutronix.de. [2001:6f8:1178:4:290:27ff:fe1d:cc33]) by mx.google.com with ESMTPS id o11si22423644wjw.72.2014.06.23.02.42.17 for (version=TLSv1 cipher=RC4-SHA bits=128/128); Mon, 23 Jun 2014 02:42:18 -0700 (PDT) Date: Mon, 23 Jun 2014 11:42:03 +0200 From: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= Subject: Re: [patch 12/13] mm: memcontrol: rewrite charge API Message-ID: <20140623094203.GG14781@pengutronix.de> References: <1403124045-24361-1-git-send-email-hannes@cmpxchg.org> <1403124045-24361-13-git-send-email-hannes@cmpxchg.org> <20140623061526.GH13440@pengutronix.de> <20140623093052.GG9743@dhcp22.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20140623093052.GG9743@dhcp22.suse.cz> Sender: owner-linux-mm@kvack.org List-ID: To: Michal Hocko Cc: Johannes Weiner , Andrew Morton , Hugh Dickins , Tejun Heo , Vladimir Davydov , linux-mm@kvack.org, cgroups@vger.kernel.org, linux-kernel@vger.kernel.org, kernel@pengutronix.de On Mon, Jun 23, 2014 at 11:30:52AM +0200, Michal Hocko wrote: > On Mon 23-06-14 08:15:26, Uwe Kleine-Konig wrote: > > Hello, > > > > On Wed, Jun 18, 2014 at 04:40:44PM -0400, Johannes Weiner wrote: > > > The memcg charge API charges pages before they are rmapped - i.e. have > > > an actual "type" - and so every callsite needs its own set of charge > > > and uncharge functions to know what type is being operated on. Worse, > > > uncharge has to happen from a context that is still type-specific, > > > rather than at the end of the page's lifetime with exclusive access, > > > and so requires a lot of synchronization. > > > ... > > > > this patch made it into next-20140623 as 5e49555277df (mm: memcontrol: rewrite > > charge API) and it makes efm32_defconfig (ARCH=arm) fail with: > > > > CC mm/swap.o > > mm/swap.c: In function 'lru_cache_add_active_or_unevictable': > > mm/swap.c:719:2: error: implicit declaration of function 'TestSetPageMlocked' [-Werror=implicit-function-declaration] > > if (!TestSetPageMlocked(page)) { > > ^ > > cc1: some warnings being treated as errors > > scripts/Makefile.build:257: recipe for target 'mm/swap.o' failed > > make[3]: *** [mm/swap.o] Error 1 > > Makefile:1471: recipe for target 'mm/swap.o' failed > > > > imx_v4_v5_defconfig works, so probably the thing that makes > > efm32_defconfig fail is CONFIG_MMU=n. > > Fix is here: > http://marc.info/?l=linux-mm&m=140330132521104 Thanks for the link. I have another problem that makes my machine fail to boot but at least this patch makes next compilable again for me. Thanks Uwe -- Pengutronix e.K. | Uwe Kleine-Konig | Industrial Linux Solutions | http://www.pengutronix.de/ | -- 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