From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail190.messagelabs.com (mail190.messagelabs.com [216.82.249.51]) by kanga.kvack.org (Postfix) with ESMTP id 9C57C8D0040 for ; Wed, 23 Mar 2011 00:42:46 -0400 (EDT) Received: from m3.gw.fujitsu.co.jp (unknown [10.0.50.73]) by fgwmail6.fujitsu.co.jp (Postfix) with ESMTP id 88A913EE0B5 for ; Wed, 23 Mar 2011 13:42:42 +0900 (JST) Received: from smail (m3 [127.0.0.1]) by outgoing.m3.gw.fujitsu.co.jp (Postfix) with ESMTP id 6D36D45DE55 for ; Wed, 23 Mar 2011 13:42:42 +0900 (JST) Received: from s3.gw.fujitsu.co.jp (s3.gw.fujitsu.co.jp [10.0.50.93]) by m3.gw.fujitsu.co.jp (Postfix) with ESMTP id 460F945DE56 for ; Wed, 23 Mar 2011 13:42:42 +0900 (JST) Received: from s3.gw.fujitsu.co.jp (localhost.localdomain [127.0.0.1]) by s3.gw.fujitsu.co.jp (Postfix) with ESMTP id 314E1E18003 for ; Wed, 23 Mar 2011 13:42:42 +0900 (JST) Received: from ml14.s.css.fujitsu.com (ml14.s.css.fujitsu.com [10.240.81.134]) by s3.gw.fujitsu.co.jp (Postfix) with ESMTP id E9432E08003 for ; Wed, 23 Mar 2011 13:42:41 +0900 (JST) Date: Wed, 23 Mar 2011 13:36:14 +0900 From: KAMEZAWA Hiroyuki Subject: Re: [PATCH 3/3] memcg: move page-freeing code outside of lock Message-Id: <20110323133614.95553de8.kamezawa.hiroyu@jp.fujitsu.com> In-Reply-To: <1300788417.1492.2.camel@leonhard> References: <1300452855-10194-1-git-send-email-namhyung@gmail.com> <1300452855-10194-3-git-send-email-namhyung@gmail.com> <20110322085938.0691f7f4.kamezawa.hiroyu@jp.fujitsu.com> <1300763079.1483.21.camel@leonhard> <20110322135619.90593f5d.kamezawa.hiroyu@jp.fujitsu.com> <1300788417.1492.2.camel@leonhard> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: owner-linux-mm@kvack.org List-ID: To: Namhyung Kim Cc: Paul Menage , Li Zefan , containers@lists.linux-foundation.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org On Tue, 22 Mar 2011 19:06:57 +0900 Namhyung Kim wrote: > 2011-03-22 (i??), 13:56 +0900, KAMEZAWA Hiroyuki: > > On Tue, 22 Mar 2011 12:04:39 +0900 > > Namhyung Kim wrote: > > > > > 2011-03-22 (i??), 08:59 +0900, KAMEZAWA Hiroyuki: > > > > On Fri, 18 Mar 2011 21:54:15 +0900 > > > > Namhyung Kim wrote: > > > > > > > > > Signed-off-by: Namhyung Kim > > > > > Cc: KAMEZAWA Hiroyuki > > > > > > > > What is the benefit of this patch ? > > > > > > > > -Kame > > > > > > > > > > Oh, I just thought generally it'd better call such a (potentially) > > > costly function outside of locks and it could reduce few of theoretical > > > contentions between swapons and/or offs. If it doesn't help any > > > realistic cases I don't mind discarding it. > > > > > > > My point is, please write patch description which shows for what this patc is. > > All cleanup are okay to me if it reasonable. But without patch description as > > "this is just a cleanup, no functional change, and the reason is...." > > we cannot maintain patches. > > > > Thanks, > > -Kame > > > > OK, I will do that in the future. Anyway, do you want me to resend the > patch with new description? > please. I'll never ack a patch without description. Thanks, -Kame -- 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