From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932851Ab2CZQFj (ORCPT ); Mon, 26 Mar 2012 12:05:39 -0400 Received: from mail-bk0-f46.google.com ([209.85.214.46]:38949 "EHLO mail-bk0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932540Ab2CZQFi (ORCPT ); Mon, 26 Mar 2012 12:05:38 -0400 Message-ID: <4F7093CD.4060008@openvz.org> Date: Mon, 26 Mar 2012 20:05:33 +0400 From: Konstantin Khlebnikov User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0.2) Gecko/20120217 Firefox/10.0.2 Iceape/2.7.2 MIME-Version: 1.0 To: Michal Hocko CC: "linux-mm@kvack.org" , Andrew Morton , "linux-kernel@vger.kernel.org" , Hugh Dickins , KAMEZAWA Hiroyuki Subject: Re: [PATCH v6 6/7] mm/memcg: kill mem_cgroup_lru_del() References: <20120322214944.27814.42039.stgit@zurg> <20120322215639.27814.4996.stgit@zurg> <20120326152315.GC22754@tiehlicka.suse.cz> In-Reply-To: <20120326152315.GC22754@tiehlicka.suse.cz> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Michal Hocko wrote: > On Fri 23-03-12 01:56:39, Konstantin Khlebnikov wrote: >> This patch kills mem_cgroup_lru_del(), we can use mem_cgroup_lru_del_list() >> instead. On 0-order isolation we already have right lru list id. >> >> Signed-off-by: Konstantin Khlebnikov >> Cc: KAMEZAWA Hiroyuki >> Cc: Hugh Dickins > > Yes, looks good > Acked-by: Michal Hocko > > Just a small nit.. > [...] >> diff --git a/mm/vmscan.c b/mm/vmscan.c >> index 5f6ed98..9de66be 100644 >> --- a/mm/vmscan.c >> +++ b/mm/vmscan.c > [...] >> @@ -1205,8 +1205,11 @@ static unsigned long isolate_lru_pages(unsigned long nr_to_scan, >> >> if (__isolate_lru_page(cursor_page, mode) == 0) { >> unsigned int isolated_pages; >> + enum lru_list cursor_lru; >> >> - mem_cgroup_lru_del(cursor_page); >> + cursor_lru = page_lru(cursor_page); >> + mem_cgroup_lru_del_list(cursor_page, >> + cursor_lru); > > Why not mem_cgroup_lru_del_list(cursor_page, > page_lru(cursor_page)); > The patch would be smaller and it doesn't make checkpatch unhappy as well. Lumpy-reclaim supposed to be removed soon, so... > >> list_move(&cursor_page->lru, dst); >> isolated_pages = hpage_nr_pages(cursor_page); >> nr_taken += isolated_pages; >> >> -- >> 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/ >