xfs
[Top] [All Lists]
<prev [Date] next>
[Advanced]
 <prev [Thread] next>

Re: Fix xfs_ichgtime mainline breakage.

from [Lachlan McIlroy] [Permanent Link][Original]
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.
[More with this subject...]
<Prev in Thread] Current Thread [Next in Thread>
Previous by Date:  Re: [Patch] xfs_lowbit64 broken on ia32, Lachlan McIlroy
Next by Date:  Re: [Patch] xfs_lowbit64 broken on ia32, David Chinner
Previous by Thread:  Fix xfs_ichgtime mainline breakage., David Chinner
Next by Thread:  [Patch] xfs_lowbit64 broken on ia32, David Chinner
Indexes:  [Date] [Thread] [Top] [All Lists]