| To: | ProfiHost - Stefan Priebe <s.priebe@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: XFS / Quota Bug in 2.6.17.x and 2.6.18x |
| From: | Nathan Scott <nathans@xxxxxxx> |
| Date: | Fri, 28 Jul 2006 09:42:54 +1000 |
| Cc: | linux-kernel@xxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx |
| In-reply-to: | <44C8A5F1.7060604@xxxxxxxxxxxxx>; from s.priebe@xxxxxxxxxxxxx on Thu, Jul 27, 2006 at 01:39:29PM +0200 |
| References: | <44C8A5F1.7060604@xxxxxxxxxxxxx> |
| Sender: | xfs-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.2.5i |
Hi Stefan, On Thu, Jul 27, 2006 at 01:39:29PM +0200, ProfiHost - Stefan Priebe wrote: > Hello! > > The crash only occurs if you use quota and IDE without barrier support. > > The Problem is, that on a new mount of a root filesystem - the flag > VFS_RDONLY is set - and so no barrier check is done before checking > quota. With this patch barrier check is done always. The partition > should not be mounted at that moment. For mount -o remount, rw or > something like this it uses another function where VFS_RDONLY is checked. Ah, I see. The patch isn't quite right, I think we will now need to also add a test to xfs_mountfs_check_barriers() to ensure the device beneath us is not bdev_read_only(). I'll add that and get the fix merged, thanks. cheers. -- Nathan |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH] [xfs] Add lock annotations to xfs_trans_update_ail and xfs_trans_delete_ail, Nathan Scott |
|---|---|
| Next by Date: | Review: xfs_repair fixes for dir2 corruption, Barry Naujok |
| Previous by Thread: | XFS / Quota Bug in 2.6.17.x and 2.6.18x, ProfiHost - Stefan Priebe |
| Next by Thread: | Re: [PATCH] [xfs] Add lock annotations to xfs_trans_update_ail and xfs_trans_delete_ail, Nathan Scott |
| Indexes: | [Date] [Thread] [Top] [All Lists] |