From patchwork Tue Jun 28 05:16:14 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Commit 94968e74 breaks f_lotsbad Date: Mon, 27 Jun 2011 19:16:14 -0000 From: Andreas Dilger X-Patchwork-Id: 102322 Message-Id: <574FA1C6-62AF-4AF0-B43C-415923D63BB1@whamcloud.com> To: Ted Ts'o Cc: ext4 development On 2011-06-19, at 7:58 PM, Ted Ts'o wrote: > On Fri, Jun 17, 2011 at 12:28:01AM -0600, Andreas Dilger wrote: >> >> I was going to send some more fixes, but it looks like your commit >> 94968e74 "libext2fs: teach bitmap functions about bigalloc/cluster" >> breaks the "f_lotsbad" test, which I found via git bisect and verified >> is the offending commit: > > The problem was a bug in lib/ext2fs/Makefile.in; the $(SRCS) macro > included "$(srcdir)/blkmap64_ba.o" (note ".o" instead of ".o"; Oops.) > > As a result we didn't have a dependency for blkmap64_ba.c in the > Makefile, and so blkmap64_ba.o wasn't getting regenerated when > bmap64.h changed. You can fix it by rm'ing blkmap64_ba.o and then > rebuilding. I'll fixed the Makefile.in, rerun "make depend", and then > check in a fix. I retested after pulling your commit 1ca87790b914a1958f3cbd8b5e1e7037f2cf30ed "libext2fs: fix makefile dependency problem", but even when I do "make clean" before each test I still get a failure on the "f_lotsbad" test: I verified that no "*.o" files remained before rebuilding, but it didn't fix the problem. I also tested on both the "maint" and "next" branches in case they were different, but it failed on both. Cheers, Andreas --- Andreas Dilger Principal Engineer Whamcloud, Inc. -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html --- ./f_lotsbad/expect.1 2011-06-01 20:25:00.871615457 +0000 +++ f_lotsbad.1.log 2011-06-28 05:09:16.484198659 +0000 @@ -26,6 +26,7 @@ Restarting e2fsck from the beginning... Pass 1: Checking inodes, blocks, and sizes +Illegal block number passed to ext2fs_test_block_bitmap #0 for in-use block map Pass 2: Checking directory structure Entry 'termcap' in / (2) has deleted/unused inode 12. Clear? yes