Re: [PATCH] mm: Implement Swap Prefetching v23

From: Andrew Morton
Date: Thu Feb 09 2006 - 23:54:34 EST


Nick Piggin <nickpiggin@xxxxxxxxxxxx> wrote:
>
> Andrew Morton wrote:
> > Con Kolivas <kernel@xxxxxxxxxxx> wrote:
> >
> >> Ok I see. We don't have a way to add to the tail of that list though?
> >
> >
> > del_page_from_lru() + (new) add_page_to_inactive_list_tail().
> >
> >
> >>Is that
> >> a worthwhile addition to this (ever growing) project? That would definitely
> >> have an impact on the other code if not all done within swap_prefetch.c..
> >> which would also be quite a large open coded something.
> >
> >
> > Do both of the above in a new function in swap.c.
> >
>
> That'll require the caller to do lru locking.
>
> I'd add an lru_cache_add_tail, use it instead of the current lru_cache_add
> that Con's got now, and just implement it in a simple manner, without
> pagevecs.

umm, that's what I said ;)
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/