xfs
[Top] [All Lists]

Re: review: use correct buffer flags when reading superblock

To: David Chinner <dgc@xxxxxxx>
Subject: Re: review: use correct buffer flags when reading superblock
From: Lachlan McIlroy <lachlan@xxxxxxx>
Date: Thu, 11 Oct 2007 12:42:25 +1000
Cc: xfs-dev <xfs-dev@xxxxxxx>, xfs-oss <xfs@xxxxxxxxxxx>
In-reply-to: <20071010112821.GI23367404@xxxxxxx>
References: <470C8F5B.90705@xxxxxxx> <20071010112821.GI23367404@xxxxxxx>
Reply-to: lachlan@xxxxxxx
Sender: xfs-bounce@xxxxxxxxxxx
User-agent: Thunderbird 2.0.0.4 (X11/20070604)
David Chinner wrote:
On Wed, Oct 10, 2007 at 06:37:47PM +1000, Lachlan McIlroy wrote:
When reading the superblock during log recovery we are not setting
the correct buffer flags.  Specifically we are not turning off flags
we do not need such as XBF_ASYNC that is causing the synchronous
xfs_iowait() to hang.  We should also turn off XBF_WRITE and remove
the buffer from the delay write queue just to be safe.

We really don't need the removal of the write flags - the XFS_bflush()
call above the xfs_getsb() call guarantees that they won't be set....


It's not obvious though.  It wasn't obvious that ASYNC was still set
and look where that got us.


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