Re: [patch] 2.2.0-pre5 FreeAfter() check missing

Linus Torvalds (torvalds@transmeta.com)
Fri, 8 Jan 1999 11:08:18 -0800 (PST)

On Fri, 8 Jan 1999, Andrea Arcangeli wrote:
>
> 2.2.0-pre5 doesn't check if the page is under async swapout before
> do a COW. I think it was not harming because shrink_mmap() should be able
> to shrink the orphaned swap cache page. But here the fix:

I hate the is_shared() function, and part of that is exactly because of
this. I looked at the code, and for the life of me I could not _prove_
that it was always safe to just ignore the fact that we're busy using the
page for IO. So the 2.2.0-pre5 behaviour is actually on purpose.

Linus

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/