xfs
[Top] [All Lists]

Re: [PATCH] xfsprogs: skip over "rootfs" entry if mtab links to /proc/mo

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: [PATCH] xfsprogs: skip over "rootfs" entry if mtab links to /proc/mounts
From: Eric Sandeen <sandeen@xxxxxxxxxx>
Date: Wed, 03 Aug 2011 14:12:43 -0500
Cc: xfs-oss <xfs@xxxxxxxxxxx>
In-reply-to: <20110803190216.GA31421@xxxxxxxxxxxxx>
References: <4E399677.1010508@xxxxxxxxxx> <20110803190216.GA31421@xxxxxxxxxxxxx>
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:5.0) Gecko/20110624 Thunderbird/5.0
On 8/3/11 2:02 PM, Christoph Hellwig wrote:
> On Wed, Aug 03, 2011 at 01:41:59PM -0500, Eric Sandeen wrote:
>> This is for RH bug 727938, xfs_fsr regression for root file system
>>
>> Fedora has made /etc/mtab a symlink to /proc/mounts, but when
>> we issue "xfs_fsr /" and fsr's getmntany() goes looking for
>> the "/" entry, the first one it finds is
>>
>> rootfs / rootfs rw 0 0
>>
>> it says no way, that's a rootfs filesystem type, not xfs!
>> And it never finds this later:
>>
>> /dev/sda2 / xfs rw,relatime,attr2,noquota 0 0
>>
>> This patch to skip over the rootfs entry seems to fix it.
> 
> I don't like this.  rootfs is the symptom, but the underlying problem
> is that in Linux we're perfectly fine to have multiple filesystems
> mounted on a single mountpoint, and the getmntany can't deal with it.

But in this case, xfs_fsr has been told to "defrag mountpoint /bar"
and if /dev/X is mounted on /foo, /bar, and /baz, it'll do the
right thing, won't it? 

oh wait I'm thinking about multiple mountpoints for the same device,
not multiple devices on the same mountpoint.

Ok, duh, you're right of course.

Well, this patch certainly makes the current situation no worse,
and fixes the immediate problem with /proc/mounts... and we really
never do want to try to do something with "rootfs" do we?

Could the other behavioral change be done as a separate patch?

Well, maybe there's no point, what do you think?

> I think the right fix is to simply remove the break from the loop,
> and thus let a second match for our fs override the first.  This relies
> on getmntent returning entries in the order they were mounted, but
> without that I can't think of a reliable way for getmntent to work
> with the multiple mounts in Linux.

fair enough.

> It would also be nice to have a testcase for this behaviour in xfstests,
> creating two loop devices with different backing files and mounting them
> at the same mount point.

easy enough :)

-Eric


<Prev in Thread] Current Thread [Next in Thread>