Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965764AbbKDUzU (ORCPT ); Wed, 4 Nov 2015 15:55:20 -0500 Received: from gum.cmpxchg.org ([85.214.110.215]:41558 "EHLO gum.cmpxchg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965730AbbKDUzS (ORCPT ); Wed, 4 Nov 2015 15:55:18 -0500 Date: Wed, 4 Nov 2015 15:55:04 -0500 From: Johannes Weiner To: Minchan Kim Cc: Andrew Morton , linux-kernel@vger.kernel.org, linux-mm@kvack.org, Michael Kerrisk , linux-api@vger.kernel.org, Hugh Dickins , zhangyanfei@cn.fujitsu.com, Rik van Riel , Mel Gorman , KOSAKI Motohiro , Jason Evans , Daniel Micay , "Kirill A. Shutemov" , Michal Hocko , yalin.wang2010@gmail.com, Shaohua Li , "Wang, Yalin" Subject: Re: [PATCH 5/8] mm: move lazily freed pages to inactive list Message-ID: <20151104205504.GA9927@cmpxchg.org> References: <1446188504-28023-1-git-send-email-minchan@kernel.org> <1446188504-28023-6-git-send-email-minchan@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1446188504-28023-6-git-send-email-minchan@kernel.org> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2119 Lines: 41 On Fri, Oct 30, 2015 at 04:01:41PM +0900, Minchan Kim wrote: > MADV_FREE is a hint that it's okay to discard pages if there is memory > pressure and we use reclaimers(ie, kswapd and direct reclaim) to free them > so there is no value keeping them in the active anonymous LRU so this > patch moves them to inactive LRU list's head. > > This means that MADV_FREE-ed pages which were living on the inactive list > are reclaimed first because they are more likely to be cold rather than > recently active pages. > > An arguable issue for the approach would be whether we should put the page > to the head or tail of the inactive list. I chose head because the kernel > cannot make sure it's really cold or warm for every MADV_FREE usecase but > at least we know it's not *hot*, so landing of inactive head would be a > comprimise for various usecases. Even if we're wrong about the aging of those MADV_FREE pages, their contents are invalidated; they can be discarded freely, and restoring them is a mere GFP_ZERO allocation. All other anonymous pages have to be written to disk, and potentially be read back. [ Arguably, MADV_FREE pages should even be reclaimed before inactive page cache. It's the same cost to discard both types of pages, but restoring page cache involves IO. ] It probably makes sense to stop thinking about them as anonymous pages entirely at this point when it comes to aging. They're really not. The LRU lists are split to differentiate access patterns and cost of page stealing (and restoring). From that angle, MADV_FREE pages really have nothing in common with in-use anonymous pages, and so they shouldn't be on the same LRU list. That would also fix the very unfortunate and unexpected consequence of tying the lazy free optimization to the availability of swap space. I would prefer to see this addressed before the code goes upstream. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/