From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail6.bemta12.messagelabs.com (mail6.bemta12.messagelabs.com [216.82.250.247]) by kanga.kvack.org (Postfix) with ESMTP id 92F186B002D for ; Thu, 10 Nov 2011 21:31:15 -0500 (EST) Received: by ywp17 with SMTP id 17so342254ywp.14 for ; Thu, 10 Nov 2011 18:31:13 -0800 (PST) Date: Thu, 10 Nov 2011 18:31:05 -0800 (PST) From: Hugh Dickins Subject: Re: [PATCH v2] mm: add free_hot_cold_page_list helper In-Reply-To: <20111101074502.32668.93131.stgit@zurg> Message-ID: References: <20110729075837.12274.58405.stgit@localhost6> <20111101074502.32668.93131.stgit@zurg> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-linux-mm@kvack.org List-ID: To: Konstantin Khlebnikov Cc: linux-mm@kvack.org, akpm@linux-foundation.org, linux-kernel@vger.kernel.org, minchan.kim@gmail.com On Tue, 1 Nov 2011, Konstantin Khlebnikov wrote: > This patch adds helper free_hot_cold_page_list() to free list of 0-order pages. > It frees pages directly from the list without temporary page-vector. > It also calls trace_mm_pagevec_free() to simulate pagevec_free() behaviour. Sorry for not speaking up sooner, but I do like this patch very much (and I'm content with your trace compatibility choice - whatever). Not so much in itself, but because it then allows a further patch (mainly to mm/vmscan.c) to remove two levels of pagevec, reducing its deepest stack by around 240 bytes. I have that patch, but keep putting off sending it in, because I want to show a reclaim stack overflow that it prevents, but the new avoidance of writeback in direct reclaim makes that harder to demonstrate. Damn! One question on your patch: where you have release_pages() doing > + list_add_tail(&page->lru, &pages_to_free); That seems reasonable, but given that __pagevec_free() proceeds by while (--i >= 0) { , starting from the far end of the pagevec (the most recently added struct page, the most likely to be hot), wouldn't you reproduce existing behaviour more accurately by a simple list_add()? Or have I got that back to front? If so, a comment on the list_add_tail() would help me to remember why - thanks. Hugh -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: email@kvack.org