Re: mm-more-likely-reclaim-madv_sequential-mappings.patch

[Rik van Riel <riel@redhat.com>]

Nick Piggin wrote:
>
> That's just handwaving. The patch still clears PG_referenced, which
> is a shared resource, and it is wrong, conceptually. You can't argue
> with that.
>
>   
I don't see an easy way around that.  If the PG_referenced bit is
set and the page is mapped, the code in vmscan.c will move the
page to the active list.

Even if the one pte mapping the page is in an MADV_SEQUENTIAL
VMA, in which case we definately do not want to activate the page.

Of course, if the PG_referenced came from a different access, things
would be a different matter. 

Fixing the page fault code so that it does not set the PG_referenced bit
would take care of that.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>