On Wed, 9 Dec 1998 18:43:25 +0100 (CET), Andrea Arcangeli
<andrea@e-mind.com> said:
> I think that my state = 0 in do_try_to_free_page() helped a lot to handle
> the better kernel performance.
Have you done any benchmarking on it? The VM is now looking pretty
good, and I'd be very reluctant to keep tweaking it now without solid
evidence as to how that will affect performance: we need to draw a line
somewhere for 2.2. I think we're now beyond the point where it makes
sense to say "here, try THIS patch to see what happens" without at least
making some attempt to test it first.
> And why not to use GFP_USER in the userspace swaping code?
Good point.
> Index: linux/mm/swap_state.c
> diff -u linux/mm/swap_state.c:1.1.3.2 linux/mm/swap_state.c:1.1.1.1.2.4
> --- linux/mm/swap_state.c:1.1.3.2 Wed Dec 9 16:11:46 1998
> +++ linux/mm/swap_state.c Wed Dec 9 18:39:03 1998
> @@ -308,7 +336,7 @@
> if (found_page)
> goto out;
>
> - new_page_addr = __get_free_page(GFP_KERNEL);
> + new_page_addr = __get_free_page(GFP_USER);
> if (!new_page_addr)
> goto out; /* Out of memory */
> new_page = mem_map + MAP_NR(new_page_addr);
--Stephen
-
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/