From mboxrd@z Thu Jan 1 00:00:00 1970 From: Glauber Costa Subject: Re: linux-next: Tree for Jan 19 (mm/memcontrol.c) Date: Fri, 20 Jan 2012 15:09:17 +0400 Message-ID: <4F194B5D.6080701@parallels.com> References: <20120119125932.a4c67005cf6a0938558e8b36@canb.auug.org.au> <4F189A97.5080007@xenotime.net> <20120120090037.e32a119f.kamezawa.hiroyu@jp.fujitsu.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from mx2.parallels.com ([64.131.90.16]:49578 "EHLO mx2.parallels.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752141Ab2ATLKJ (ORCPT ); Fri, 20 Jan 2012 06:10:09 -0500 In-Reply-To: <20120120090037.e32a119f.kamezawa.hiroyu@jp.fujitsu.com> Sender: linux-next-owner@vger.kernel.org List-ID: To: KAMEZAWA Hiroyuki Cc: Randy Dunlap , Stephen Rothwell , linux-next@vger.kernel.org, LKML , "linux-mm@kvack.org" On 01/20/2012 04:00 AM, KAMEZAWA Hiroyuki wrote: > On Thu, 19 Jan 2012 14:35:03 -0800 > Randy Dunlap wrote: > >> On 01/18/2012 05:59 PM, Stephen Rothwell wrote: >>> Hi all, >>> >>> Changes since 20120118: >> >> >> on i386: >> >> mm/built-in.o:(__jump_table+0x8): undefined reference to `memcg_socket_limit_enabled' >> mm/built-in.o:(__jump_table+0x14): undefined reference to `memcg_socket_limit_enabled' >> >> >> Full randconfig file is attached. >> > > Thank you. Forwarding this to Costa. > > Thanks, > -Kame > Oh dear lord... So what happened here, is that I moved this code out of CONFIG_INET to fix another problem, and forgot that it needed to be wrapped under CONFIG_NET instead. It is not an excuse, but I did compiled it over at least 6 random configs, and thought it was okay. I guess so many things select CONFIG_NET that it ends up being hard to generate a config without it. I think the fix for this needs to go through dave's tree, since it is where the original fix went through. I will send a fix shortly.