xfs
[Top] [All Lists]

Re: TAKE 981875 - xfs_dm_rdwr() needs to pass a vfsmount to dentry_open(

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: TAKE 981875 - xfs_dm_rdwr() needs to pass a vfsmount to dentry_open()
From: Niv Sardi <xaiki@xxxxxxx>
Date: Mon, 26 May 2008 16:02:29 +1000
Cc: Lachlan McIlroy <lachlan@xxxxxxx>, sgi.bugs.xfs@xxxxxxxxxxxx, xfs@xxxxxxxxxxx
In-reply-to: <20080520064356.GA8711@xxxxxxxxxxxxx> (Christoph Hellwig's message of "Tue, 20 May 2008 02:43:56 -0400")
References: <20080516045732.F12DC58C4C29@xxxxxxxxxxxxxxxxxxxxxxx> <20080516051048.GB5035@xxxxxxxxxxxxx> <20080520064356.GA8711@xxxxxxxxxxxxx>
Sender: xfs-bounce@xxxxxxxxxxx
User-agent: Gnus/5.110009 (No Gnus v0.9) Emacs/23.0.60 (i486-pc-linux-gnu)
Christoph Hellwig <hch@xxxxxxxxxxxxx> writes:

> On Fri, May 16, 2008 at 01:10:48AM -0400, Christoph Hellwig wrote:
>> On Fri, May 16, 2008 at 02:57:32PM +1000, Lachlan McIlroy wrote:
>> > xfs_dm_rdwr() needs to pass a vfsmount to dentry_open()
>> > 
>> > We need a vfsmount pointer in xfs_dm_rdwr() but we are not
>> > provided with one and there's no way to get to it.  So add
>> > a m_vfsmount field to the xfs_mount structure and set it
>> > up at mount time.  We can then access it through the xfs
>> > inode.
>> 
>> Stong NACK for this crap, and double NACK for pushing this towards
>> mainline.
>
> No reply yet.  Folks, you realize this is an instant oops if you perform
> dmapi operations after one fs instance has been unmounted?

This is not going into mainline, but we need it in -dev to make sure we
can still QA DMAPI. We (I should be but flooded with other stuff) are
looking at other solutions to fix the issue the right way.

Cheers,
-- 
Niv Sardi


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