[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <50B27AD1.6010703@redhat.com>
Date: Sun, 25 Nov 2012 15:08:49 -0500
From: Rik van Riel <riel@...hat.com>
To: Fengguang Wu <fengguang.wu@...el.com>
CC: Metin Döşlü <metindoslu@...il.com>,
Jaegeuk Hanse <jaegeuk.hanse@...il.com>,
Jan Kara <jack@...e.cz>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-mm@...ck.org" <linux-mm@...ck.org>,
Johannes Weiner <jweiner@...hat.com>
Subject: Re: Problem in Page Cache Replacement
On 11/22/2012 10:53 AM, Fengguang Wu wrote:
> Ah it's more likely caused by this logic:
>
> if (is_active_lru(lru)) {
> if (inactive_list_is_low(mz, file))
> shrink_active_list(nr_to_scan, mz, sc, priority, file);
>
> The active file list won't be scanned at all if it's smaller than the
> active list. In this case, it's inactive=33586MB > active=25719MB. So
> the data-1 pages in the active list will never be scanned and reclaimed.
That's it, indeed.
The reason we have that code is that otherwise one large streaming
IO could easily end up evicting the entire page cache working set.
Usually it works well, because the new page cache working set tends
to get touched twice while on the inactive list, and the old working
set gets demoted from the active list.
Only in a few very specific cases, where the inter-reference distance
of the new working set is larger than the size of the inactive list,
does it fail.
Something like Johannes's patches should solve the problem.
--
All rights reversed
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists