Re: [RFC][PATCH] split file and anonymous page queues #3

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

Nikita Danilov wrote:
> Rik van Riel writes:
>  > [ OK, I suck.  I edited yesterday's email with the new info, but forgot
>  >    to change the attachment to today's patch.  Here is today's patch. ]
>  > 
>  > Split the anonymous and file backed pages out onto their own pageout
>  > queues.  This we do not unnecessarily churn through lots of anonymous
>  > pages when we do not want to swap them out anyway.
> 
> Won't this re-introduce problems similar to ones due to split
> inactive_clean/inactive_dirty queues we had in the past?
> 
> For example, by rotating anon queues faster than file queues, kernel
> would end up reclaiming anon pages that are hotter (in "absolute" LRU
> order) than some file pages.

That is why we check the fraction of referenced pages in each
queue.  Please look at the get_scan_ratio() and shrink_zone()
code in my patch.

-- 
Politics is the struggle between those who want to make their country
the best in the world, and those who believe it already is.  Each group
calls the other unpatriotic.