Re: [PATCH] Re: fs corruption with invalidate_buffers()

From: Jan Niehusmann (jan@gondor.com)
Date: Thu Dec 21 2000 - 19:48:10 EST


On Thu, Dec 21, 2000 at 04:37:30PM -0800, Linus Torvalds wrote:
> This looks bogus.

It may be - I just did what Al told me without really understanding it ;-)

The test I did initially was the following:

if(!atomic_read(&bh->b_count) &&
        (destroy_dirty_buffers || !buffer_dirty(bh))
        && ! (bh->b_page && bh->b_page->mapping)
        )

That is, I was explicitely checking for a mapped page. It worked well, too.
Is this more reasonable?

Jan

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



This archive was generated by hypermail 2b29 : Sat Dec 23 2000 - 21:00:30 EST