On Fri, 2005-03-04 at 16:45, Andrew Morton wrote:
> Badari Pulavarty <[EMAIL PROTECTED]> wrote:
> >
> > > What's all this doing?  (It needs comments please - it's very unobvious).
> > 
> > All its trying to do is - to make sure the page is uptodate so that
> > it can zero out the portion thats needed. 
> 
> OK.
> 
> > I can do getblock() and ll_rw_block(READ) instead. I was
> > trying to re-use the code and mpage_readpage() drops the lock.
> > What do you think ?
> 
> Can you just call ->prepare_write, as nobh_truncate_page() does?  That'll
> cause a nested transaction, but that's legal.
> 

Thats what I tried earlier, but never got it working. Lots of journal
asserts :( 

I guess I really need to learn journaling code someday.

Thanks,
Badari

-
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/

Reply via email to