From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: ext4 still broken on arm (at least) Date: Tue, 27 Nov 2007 21:43:08 -0800 Message-ID: <20071127214308.5215a6a2.akpm@linux-foundation.org> References: <20071127211444.703741a9.akpm@linux-foundation.org> <20071128053020.GA6977@skywalker> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: "linux-ext4@vger.kernel.org" , linux-arch@vger.kernel.org To: "Aneesh Kumar K.V" Return-path: Received: from smtp2.linux-foundation.org ([207.189.120.14]:54545 "EHLO smtp2.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750876AbXK1FoK (ORCPT ); Wed, 28 Nov 2007 00:44:10 -0500 In-Reply-To: <20071128053020.GA6977@skywalker> Sender: linux-ext4-owner@vger.kernel.org List-Id: linux-ext4.vger.kernel.org On Wed, 28 Nov 2007 11:00:20 +0530 "Aneesh Kumar K.V" wrote: > On Tue, Nov 27, 2007 at 09:14:44PM -0800, Andrew Morton wrote: > > fs/ext4/mballoc.c: In function `ext4_mb_generate_buddy': > > fs/ext4/mballoc.c:836: error: implicit declaration of function `ext2_find_next_bit' > > > I actually sent in a patch which changes asking for review to > linux-arch. I haven't got the response yet. Don't expect one... > Attaching the patch > below > Introduce ext4_find_next_bit > > From: Aneesh Kumar K.V > > This gets used by the ext4 multi block allocator patches. > > Also add generic_find_next_le_bit > > Signed-off-by: Aneesh Kumar K.V > Cc: > Signed-off-by: Andrew Morton > --- > > include/asm-arm/bitops.h | 2 + > include/asm-generic/bitops/ext2-non-atomic.h | 2 + > include/asm-generic/bitops/le.h | 4 ++ > include/asm-m68k/bitops.h | 2 + > include/asm-m68knommu/bitops.h | 2 + > include/asm-powerpc/bitops.h | 4 ++ > include/asm-s390/bitops.h | 2 + > include/linux/ext4_fs.h | 1 + > lib/find_next_bit.c | 43 ++++++++++++++++++++++++++ May as well merge it I guess. I'll do that. Possibly it should be merged into the ext4 tree instead, so that people can build the ext4 devel tree on other architectures but obviously there's no call for that.