On Tue, Jul 05, 2011 at 05:36:18PM -0500, Alex Elder wrote:
> > A large part of the issue is that XFS writes data out itself two times
> > in the ->sync_fs method, overriding the lifelock protection in the core
> > writeback code, and another issue is the lock-less xfs_ioend_wait call,
> > which doesn't prevent new ioend from beeing queue up while waiting for
> > the count to reach zero.
>
> The change affects only the first thing you mention here, not
> the second.
It does. We're also removing the xfs_ioend_wait done from
xfs_sync_data/xfs_sync_inode_data. We still have another one in
->write_inode, though.
> The i_iocount wait is not affected by your patch.
We're only removing one of the two we're doing per inode now.
> I'm OK with the change, but really prefer to have
> the description not include stuff that just isn't
> there. If you want me to commit this as-is, just
> say so and I will. Otherwise, post an update and
> I'll use that. In any case, you can consider this
> reviewed by me.
If you have an idea how to reword the description send it my way.
|