| 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> |
|---|---|---|
| ||
| Previous by Date: | Re: Crash on 2.6.21.7 Vanilla + DRBD 0.7, David Chinner |
|---|---|
| Next by Date: | Re: review: use correct buffer flags when reading superblock, David Chinner |
| Previous by Thread: | Re: review: use correct buffer flags when reading superblock, David Chinner |
| Next by Thread: | Re: review: use correct buffer flags when reading superblock, David Chinner |
| Indexes: | [Date] [Thread] [Top] [All Lists] |