Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755439Ab0K2XaB (ORCPT ); Mon, 29 Nov 2010 18:30:01 -0500 Received: from mail-ww0-f44.google.com ([74.125.82.44]:46843 "EHLO mail-ww0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751555Ab0K2XaA (ORCPT ); Mon, 29 Nov 2010 18:30:00 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-type; b=rnRUcOR6hyjUmaX8gOMK2Xaao7lJRr/UveA9WWrhJBTM5dLYFtdGs0I6aDenDAR6Of o7HmDqjAIHKhg4I33mRJ3ydOynNOSe6eLd1S6IGQ+0w6aPR6gh4slljXlC4QvPFm35DX YeRLuwTLjsiPDriebpqGv0G9Cl3BWg/SgOMj0= MIME-Version: 1.0 In-Reply-To: <20101129143850.dbc533da.akpm@linux-foundation.org> References: <20100831083844.043619334@linux.vnet.ibm.com> <20101018103729.GA4122@linux.vnet.ibm.com> <20101019110801.GA4128@linux.vnet.ibm.com> <20101019213112.GA11432@merkur.ravnborg.org> <4CC8ABC7.5030003@suse.cz> <20101028203542.GA30879@merkur.ravnborg.org> <4CC9E8C7.60306@suse.cz> <20101129143850.dbc533da.akpm@linux-foundation.org> From: Mike Frysinger Date: Mon, 29 Nov 2010 18:29:37 -0500 Message-ID: Subject: Re: [PATCH] initramfs: Fix build break on symbol-prefixed archs To: Andrew Morton Cc: Michal Marek , Sam Ravnborg , Hendrik Brueckner , Michael Holzheu , tabbott@ksplice.com, vda.linux@googlemail.com, hpa@linux.intel.com, linux-kernel@vger.kernel.org, heiko.carstens@de.ibm.com, schwidefsky@de.ibm.com Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2543 Lines: 58 On Mon, Nov 29, 2010 at 17:38, Andrew Morton wrote: > On Wed, 24 Nov 2010 03:40:51 -0500 Mike Frysinger wrote: >> On Sat, Nov 13, 2010 at 18:17, Mike Frysinger wrote: >> > On Thu, Oct 28, 2010 at 17:19, Michal Marek wrote: >> >> On 28.10.2010 22:35, Sam Ravnborg wrote: >> >>> On Thu, Oct 28, 2010 at 12:46:31AM +0200, Michal Marek wrote: >> >>>> On 27.10.2010 20:23, Mike Frysinger wrote: >> >>>>> can we get this change dropped from linux-next until someone gets a >> >>>>> chance to fix it ? >> >>>> >> >>>> Sorry I missed this thread. So your last patch >> >>>> (http://lkml.org/lkml/2010/10/19/488) did fix the Blackfin build >> >>>> failure? Can you submit it with a proper Signed-off-by and changelog? >> >>> >> >>> Did you see the patch I posted in: http://lkml.org/lkml/2010/10/19/534 >> >>> >> >>> The one that fix up the cppflags usage. >> >>> >> >>> I think this will solve the issue with blackfin nicely - and >> >>> make use of cppflags more as expected. >> >> >> >> As Mike noted - your patch does not strip the double quotes. And Mike's >> >> patch is really minimal, it just adds the SYMBOL_PREFIX definition to >> >> _a_flags. I'll look at your patch after this merge window. >> > >> > why am i seeing __initramfs_size failures in mainline now ? >> >> ping ... it looks like no one merged the VMLINUX_SYMBOL() changes in >> usr/initramfs_data.S proposed by Hendrik > > Well I found and forward-ported the below, from October. > > Problem is, the changelog didn't describe the alleged "build break" his e-mail from October you found contains the context showing the build failure: i'm pretty sure this breaks on every symbol-prefixed arch. at least, it broke every Blackfin board: init/built-in.o: In function `populate_rootfs': init/initramfs.c:(.init.text+0x2548): undefined reference to `__initramfs_size' init/initramfs.c:(.init.text+0x254e): undefined reference to `__initramfs_size' make: *** [.tmp_vmlinux1] Error 1 > it's unclear whether the fix is needed in mainline, in 2.6.36 or is > just fixing some bug which was introduced by some other > perhaps-not-applied patch. it was caused by changes Michal added post 2.6.36, and then pushed to Linus without bothering to address the reports sent to him. so it needs to be in before 2.6.37 is released. -mike -- 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/