Re: try_to_swap_out() return value problem?

From: Rik van Riel (riel@conectiva.com.br)
Date: Thu Jan 04 2001 - 11:14:41 EST


On Wed, 3 Jan 2001, Linus Torvalds wrote:
> On Thu, 4 Jan 2001, Marcelo Tosatti wrote:

> I agree that the return value of swap_out() is fairly meaningless. It's
> been fairly meaningless for a long time now, and it's entirely possible
> that the "while (swap_out())" loop should be just something like
>
> /* Scan the VM space, try to clean up the page tables a bit */
> for (i = 0 ; i <= nr_threads >> priority; i++)
> swap_out(gfp_mask);

The problem with this is that it means that page aging of
the mapped active pages is no longer balanced against the
aging of the unmapped active pages.

Though I guess this is not an issue in practice since the
downward aging for all active pages is done in
refill_inactive_scan() anyway and only the mapped pages
which aren't on any list have the potential to be balanced
badly.

regards,

Rik

--
Hollywood goes for world dumbination,
	Trailer at 11.

http://www.surriel.com/ http://www.conectiva.com/ http://distro.conectiva.com.br/

- 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 Jan 07 2001 - 21:00:18 EST