[lkml]   [2000]   [Sep]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: The INN/mmap bug

On Sun, 17 Sep 2000, Alexander Viro wrote:
> Ugh. OK, first of all, patch is _not_ correct. Doesn't zero out the end of
> partial block.
> <note to myself: no patches before the first cup of coffee>

Yeah. See my other mail - I don't think the patch is needed at all,
although I'm convinced that it would actually hide the problem for the
particular case of INN (zeroing out the partial block is not an issue
there, as truncate will have done it - it's really only protection against
badly behaved software and shouldn't matter to any "regular" use).

> We know that ->writepage() is called and we know that data doesn't get
> to the disk. How can it happen? If we can... Oh, fsck. Linus, we could
> very well lose the buffers since the moment when page had been read.
> See what happens? We recreate the buffer ring for the page and it's
> nowhere near "everything uptodate" state.

Yeah. See my suggested fix in the other mail. We just must not read in
buffers for pages that are already up-to-date.


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
Please read the FAQ at

 \ /
  Last update: 2005-03-22 12:45    [from the cache]
©2003-2014 Jasper Spaans. hosted at Digital Ocean