Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935854AbYBCMjY (ORCPT ); Sun, 3 Feb 2008 07:39:24 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1761800AbYBCMjP (ORCPT ); Sun, 3 Feb 2008 07:39:15 -0500 Received: from monty.telenet-ops.be ([195.130.132.56]:60473 "EHLO monty.telenet-ops.be" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1760925AbYBCMjO (ORCPT ); Sun, 3 Feb 2008 07:39:14 -0500 Date: Sun, 3 Feb 2008 13:39:02 +0100 (CET) From: Geert Uytterhoeven To: Heiko Carstens , aneesh.kumar@linux.vnet.ibm.com cc: Bastian Blank , Andrew Morton , linux-s390@vger.kernel.org, Linux Kernel Development , Linux/PPC Development , linux-ext4@vger.kernel.org Subject: Re: [PATCH] Fix ext4 bitops In-Reply-To: <20080203121238.GD18211@osiris.ibm.com> Message-ID: References: <20080201200208.GA28274@wavehammer.waldi.eu.org> <20080201122257.9524c2bb.akpm@linux-foundation.org> <20080201210404.GA31271@wavehammer.waldi.eu.org> <20080203121238.GD18211@osiris.ibm.com> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2032 Lines: 50 On Sun, 3 Feb 2008, Heiko Carstens wrote: > On Fri, Feb 01, 2008 at 10:04:04PM +0100, Bastian Blank wrote: > > On Fri, Feb 01, 2008 at 12:22:57PM -0800, Andrew Morton wrote: > > > On Fri, 1 Feb 2008 21:02:08 +0100 > > > Bastian Blank wrote: > > > > > > > Fix ext4 bitops. > > > > > > This is incomplete. Please tell us what was "fixed". > > > > > > If it was a build error then please quote the compile error output in the > > > changelog, as well as the usual description of what the problem is, and how > > > it was fixed. > > > > | fs/ext4/mballoc.c: In function 'ext4_mb_generate_buddy': > > | fs/ext4/mballoc.c:954: error: implicit declaration of function 'generic_find_next_le_bit' > > > > The s390 specific bitops uses parts of the generic implementation. > > Include the correct header. > > That doesn't work: > > fs/built-in.o: In function `ext4_mb_release_inode_pa': > mballoc.c:(.text+0x95a8a): undefined reference to `generic_find_next_le_bit' > fs/built-in.o: In function `ext4_mb_init_cache': > mballoc.c:(.text+0x967ea): undefined reference to `generic_find_next_le_bit' > > This still needs generic_find_next_le_bit which comes > from lib/find_next_bit.c. That one doesn't get built on s390 since we > don't set GENERIC_FIND_NEXT_BIT. > Currently we have the lengthly patch below queued. Similar issue on m68k. As Bastian also saw it on powerpc, I'm getting the impression the ext4 people don't (compile) test on big endian machines? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- 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/