Messages in this thread | | | Date | Wed, 06 Dec 2000 07:35:47 +1100 | From | Andrew Morton <> | Subject | Re: [PATCH] inode dirty blocks Re: test12-pre4 |
| |
Alexander Viro wrote: > > On Sun, 3 Dec 2000, Linus Torvalds wrote: > > > > > Synching up with Alan and various other stuff. The most important one > > being the fix to the inode dirty block list. > > It doesn't solve the problem. If you unlink a file with dirty metadata > you have a nice chance to hit the BUG() in inode.c:83. I hope that patch > below closes all remaining holes. See analysis in previous posting > (basically, bforget() is not enough when we free the block; bh should > be removed from the inode's list regardless of the ->b_count). > Cheers, > Al > > diff -urN rc12-pre4/fs/buffer.c rc12-pre4-dirty_blocks/fs/buffer.c > --- rc12-pre4/fs/buffer.c Mon Dec 4 01:01:43 2000 > +++ rc12-pre4-dirty_blocks/fs/buffer.c Mon Dec 4 01:11:42 2000
That bforget-inode patch ran fine on two machines for ten hours. One was SMP. The other was running the ATA guy's latest set of patches including taskfile support.
The proposed FS changes are solid.
The third machine died horribly twice - recursive pagefaults. Without IDE patch. This could be anything, including hardware. Will investigate. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org Please read the FAQ at http://www.tux.org/lkml/
| |