From: Ted Ts'o Subject: Re: [next-20101038] Call trace in ext4 Date: Thu, 28 Oct 2010 16:05:48 -0400 Message-ID: <20101028200548.GC28126@thunk.org> References: <20101028175221.GA1578@arch.trippelsdorf.de> <20101028180118.GC6814@thunk.org> <20101028193211.GA28126@thunk.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Markus Trippelsdorf , LKML , linux-ext4@vger.kernel.org, sfr@canb.auug.org.au, Arnd Bergmann , Avinash Kurup To: sedat.dilek@gmail.com Return-path: Received: from thunk.org ([69.25.196.29]:43415 "EHLO thunker.thunk.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757707Ab0J1UFw (ORCPT ); Thu, 28 Oct 2010 16:05:52 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-ext4-owner@vger.kernel.org List-ID: On Thu, Oct 28, 2010 at 09:54:23PM +0200, Sedat Dilek wrote: > > Hm, unfortunately NO (see logs). > > I have compiled via M=fs/ext4 in an already compiled build-tree with > these 3 patches. Ok, stupid question. You did make sure the new ext4 module was loaded, right? > [ 100.884524] ------------[ cut here ]------------ > [ 100.884718] kernel BUG at /home/sd/src/linux-2.6/linux-2.6.36/debian/build/source_i386_none/fs/ext4/inode.c:2721! OK, so after the patch, line 2721: changed from page_buffers() to: if (!page_has_buffers(page)) page_has_buffers() expands to: #define page_has_buffers(page) PagePrivate(page) which expands to test_bit() call to see if PG_private1 is set in page->flags. There is no BUG_ON anywhere there as far as I can tell. Line 2721 in the older kernel was page_has_buffers() which does have a BUG_ON check. - Ted