xfs
[Top] [All Lists]

Re: Fix xfs_ichgtime mainline breakage.

To: David Chinner <dgc@xxxxxxx>
Subject: Re: Fix xfs_ichgtime mainline breakage.
From: Lachlan McIlroy <lachlan@xxxxxxx>
Date: Wed, 05 Dec 2007 13:32:04 +1100
Cc: xfs-dev <xfs-dev@xxxxxxx>, xfs-oss <xfs@xxxxxxxxxxx>
In-reply-to: <20071203074345.GW119954183@xxxxxxx>
References: <20071203074345.GW119954183@xxxxxxx>
Reply-to: lachlan@xxxxxxx
Sender: xfs-bounce@xxxxxxxxxxx
User-agent: Thunderbird 2.0.0.9 (X11/20071031)
Looks good Dave.  Thanks for catching that.

David Chinner wrote:
Folks,

Just noticed with the mainline merge (2.6.24-rc3) to xfs-dev that
xfs_ichgtime + xfs_ichgtime_fast are broken.

        if (!(inode->i_state & I_SYNC))
                mark_inode_dirty_sync(inode);

that check used to be against I_LOCK, which was arguably broken,
but this one will mean newly created files are not put on the dirty
list and hence not written back by pdflush.

Patch to fix this attached.

Cheers,

Dave.


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