On Sat, 2007-12-29 at 13:51 +0800, Fengguang Wu wrote: > On Fri, Dec 28, 2007 at 10:50:59PM -0600, Dave Kleikamp wrote: > > > > On Sat, 2007-12-29 at 10:21 +0800, Fengguang Wu wrote:
> > > This line should be moved below: > > > > + set_page_writeback(page); > > > > No. set_page_writeback() needs to be called before submit_bio() is > > called. > > Ah yes. > > > I don't think there is any harm in calling set_page_writeback(), > > redirty_page_for_writeback() and end_page_writeback() in the case where > > there is no I/O to submit, and some dirty data cannot be written. It is > > consistent with what happens in __block_write_full_page(). > > > > It's also possible that some part of the page was written, and another > > part cannot be, causing the page to be redirtied. > > You are right. I revisited the code and there's nothing wrong with > your patch :-) Fengguang, Thank you for all your help with this bug. I've been on vacation and hadn't had much time to look at it. We'd probably still be scratching our heads wondering where the problem was without your hard work. Shaggy > > Regards, > Fengguang > -- David Kleikamp IBM Linux Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
