On Tue, 23 Jan 2007 16:49:55 -0800 (PST)
Christoph Lameter <[EMAIL PROTECTED]> wrote:

> If we enter reclaim and the number of page cache pages
> is too high then we switch off swapping during reclaim
> to avoid touching anonymous pages.

In general, I like this (kind of) feature.

> +     /*
> +      * If the page cache is too big then focus on page cache
> +      * and ignore anonymous pages
> +      */
> +     if (sc->may_swap && zone_page_state(zone, NR_FILE_PAGES)
> +                     > zone->max_pagecache_pages)
> +             sc->may_swap = 0;
> +


How about adding this (kind of) check ?

if (sc->may_swap &&
    zone_page_state(zone, NR_FILE_PAGES) &&
    !(curreht->flags & PF_MEMALLOC))
        sc->may_swap = 0;

-Kame

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to