From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752229Ab2CIHAm (ORCPT ); Fri, 9 Mar 2012 02:00:42 -0500 Received: from 1wt.eu ([62.212.114.60]:62979 "EHLO 1wt.eu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751760Ab2CIHAl (ORCPT ); Fri, 9 Mar 2012 02:00:41 -0500 Date: Fri, 9 Mar 2012 07:57:03 +0100 From: Willy Tarreau To: Ben Hutchings Cc: Jiri Slaby , Heiko Carstens , Greg KH , linux-kernel@vger.kernel.org, Andrew Morton , torvalds@linux-foundation.org, stable@vger.kernel.org, lwn@lwn.net, Tony Luck , Jiri Slaby , raven@themaw.net Subject: Re: Build broken on s390 and ia64 [was: Linux 2.6.32.58] Message-ID: <20120309065703.GE25153@1wt.eu> References: <20120304180911.GA7336@kroah.com> <4F54A46A.6000305@suse.cz> <20120305120241.GA3627@osiris.boeblingen.de.ibm.com> <4F54B618.1090301@suse.cz> <20120305223338.GE2930@1wt.eu> <1331270953.3022.89.camel@deadeye> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1331270953.3022.89.camel@deadeye> User-Agent: Mutt/1.4.2.3i Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Ben, On Fri, Mar 09, 2012 at 05:29:13AM +0000, Ben Hutchings wrote: > On Mon, 2012-03-05 at 23:33 +0100, Willy Tarreau wrote: > > Hi, > > > > On Mon, Mar 05, 2012 at 01:48:24PM +0100, Jiri Slaby wrote: > > > >>> Linus Torvalds (2): Fix autofs compile without CONFIG_COMPAT > > > >> > > > >> But who defines is_compat_task *with* CONFIG_COMPAT on ia64? > > > >> > > > >> fs/autofs4/inode.c: In function 'autofs4_fill_super': > > > >> fs/autofs4/inode.c:345: error: implicit declaration of function > > > >> 'is_compat_task' > > > > > > > > The ia64 compat code got entirely removed, since it was broken: > > > > > > > > 32974ad4907cdde6c9de612cd1b2ee0568fb9409 "[IA64] Remove COMPAT_IA32 support" > > > > > > Yes, but that is even in 2.6.34. So the fix for autofs is incomplete in > > > .32 as it breaks build on configs which used to work. > > > > So in the end, does anybody have an idea what is missing from this patch ? > > I'm not sure that reverting the autofs fix is a right solution either :-/ > > Either cherry-pick commit 32974ad4907cdde6c9de612cd1b2ee0568fb9409 or > use this reduced version. Thanks but now I have a doubt, maybe Jiri and Tony can help. In the commit above, Tony says that CONFIG_COMPAT has been broken for a long time (2008) and nobody apparently uses it. Still, Jiri got a build failure because he used CONFIG_COMPAT. So either there are some valid uses and we should not remove the feature that late in a stable branch, or Jiri only encountered it upon a make allyesconfig then it might be safe to remove it. You see, I wouldn't want to have to issue .60 with a revert of this patch because someone complains about CONFIG_COMPAT disappearing on IA64. I'm fine with either patch (#define is_compat_task 0 or remove COMPAT) but I want to ensure we're not breaking working setups which are currently stuck to .57 due to the recent breakage. Regards, Willy