On Sat, 30 Dec 2000, Daniel Phillips wrote:
> Linus Torvalds wrote:
> >
> > Ok, there's a test13-pre6 out there now, which does a partial sync with
> > Alan, in addition to hopefully fixing the innd shared mapping writeback
> > problem for good. Thanks to Marcelo Tosatti and others..
>
> After the page_cache_release at line 574 of vmscan.c the page is
> unlocked and only owned by the page cache - anything could happen. How
> do you know the set_page_dirty at line 581 is still hitting a valid
> page?
Good question.
It should be safe because of the magic return value from "writepage()".
If "writepage()" returns 1, then that implies that the page is locked down
somehow. But you're right, this is ugly, if not outright buggy (maybe the
locked down state could change after the writepage, who knows?).
Moving the test is probably a good idea.
Linus
-
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 : Sun Dec 31 2000 - 21:00:13 EST