From: Valerie Aurora Subject: Re: [PATCH 14/38] fallthru: ext2 fallthru support Date: Thu, 5 Aug 2010 19:30:19 -0400 Message-ID: <20100805233019.GA30761@shell> References: <1276627208-17242-1-git-send-email-vaurora@redhat.com> <1276627208-17242-15-git-send-email-vaurora@redhat.com> <20100804224831.GB29353@shell> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: viro@zeniv.linux.org.uk, jblunck@suse.de, hch@infradead.org, linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, tytso@mit.edu, linux-ext4@vger.kernel.org To: Miklos Szeredi Return-path: Received: from mx1.redhat.com ([209.132.183.28]:1025 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754007Ab0HEXav (ORCPT ); Thu, 5 Aug 2010 19:30:51 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-ext4-owner@vger.kernel.org List-ID: On Thu, Aug 05, 2010 at 12:36:55PM +0200, Miklos Szeredi wrote: > On Wed, 4 Aug 2010, Valerie Aurora wrote: > > --zYM0uCDKw75PZbzx > > Content-Type: text/plain; charset=us-ascii > > Content-Disposition: inline > > > > On Wed, Aug 04, 2010 at 04:44:10PM +0200, Miklos Szeredi wrote: > > > On Tue, 15 Jun 2010, Valerie Aurora wrote: > > > > Add support for fallthru directory entries to ext2. > > > > > > If a previously used ext2 filesystem with is mounted again then > > > fallthroughs don't appear to work as expected. Stat returns ENOENT > > > for these entries. > > > > > > Can't see anything obviously wrong with the code. > > > > Hm, I wrote one test case for this that worked (attached). Can you > > give me more details on your test case? Thanks, Cool, thanks. Yes, I suppose the fallthrus should be ignored if they don't fall through to anything. If I do a proper lookup for d_ino, I can kill two birds with one stone, since that will tell us whether there is anything below the fallthru and thus whether to return this directory entry. -VAL > uml:~# mount -oloop -r ext3-2.img /mnt/img/ > uml:~# mount -oloop -r ext3.img /mnt/img/ > uml:~# losetup -f ovl.img > uml:~# mmount -b 8 -t ext2 /dev/loop2 /mnt/img/ > uml:~# "ls" /mnt/img > bunion lost+found union > uml:~# "ls" /mnt/img/union > 1 2 3 > uml:~# "ls" /mnt/img/union/1 > a x > uml:~# umount /mnt/img/ > uml:~# mmount -b 8 -t ext2 /dev/loop2 /mnt/img/ > uml:~# ls -l /mnt/img/ > total 14 > drwxr-xr-x 2 root root 1024 Aug 5 09:56 bunion > drwx------ 2 root root 12288 Aug 5 09:41 lost+found > drwxr-xr-x 3 root root 1024 Aug 5 09:56 union > uml:~# ls -l /mnt/img/union/ > ls: cannot access /mnt/img/union/3: No such file or directory > ls: cannot access /mnt/img/union/2: No such file or directory > total 1 > drwxr-xr-x 2 root root 1024 Aug 5 09:56 1 > ?????????? ? ? ? ? ? 2 > ?????????? ? ? ? ? ? 3 > uml:~# ls -l /mnt/img/union/1 > ls: cannot access /mnt/img/union/1/a: No such file or directory > ls: cannot access /mnt/img/union/1/x: No such file or directory > total 0 > ?????????? ? ? ? ? ? a > ?????????? ? ? ? ? ? x > uml:~# > > Thanks, > Miklos