On Sun, Jun 24, 2007 at 05:47:55AM +0200, Nick Piggin wrote: > On Sat, Jun 23, 2007 at 11:07:54PM -0400, Jeff Garzik wrote: > > > >- Large block support. I can mount and run an 8K block size minix3 fs on > > > my 4K page system and it didn't require anything special in the fs. We > > > can go up to about 32MB blocks now, and gigabyte+ blocks would only > > > require one more bit in the fsblock flags. fsblock_superpage blocks > > > are > PAGE_CACHE_SIZE, midpage ==, and subpage <. > > > > definitely useful, especially if I rewrite my ibu filesystem for 2.6.x, > > like I've been planning. > > Yeah, it wasn't the primary motivation for the rewrite, but it would > be negligent to not even consider large blocks in such a rewrite, I > think.
I'll join the cheering here, thanks for starting on this. > > > My gut feeling is that there are several problem areas you haven't hit > > yet, with the new code. > > I would agree with your gut :) > Without having read the code yet (light reading for monday morning ;), ext3 and reiserfs use buffers heads for data=ordered to help them do deadlock free writeback. Basically they need to be able to write out the pending data=ordered pages, potentially with the transaction lock held (or if not held, while blocking new transactions from starting). But, writepage, prepare_write and commit_write all need to start a transaction with the page lock already held. So, if the page lock were used for data=ordered writeback, there would be a lock inversion between the transaction lock and the page lock. Using buffer heads instead allows the FS to send file data down inside the transaction code, without taking the page lock. So, locking wrt data=ordered is definitely going to be tricky. The best long term option may be making the locking order transaction -> page lock, and change writepage to punt to some other queue when it needs to start a transaction. -chris - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html