From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx188.postini.com [74.125.245.188]) by kanga.kvack.org (Postfix) with SMTP id 02E686B005A for ; Mon, 5 Nov 2012 16:44:57 -0500 (EST) Date: Mon, 5 Nov 2012 13:44:56 -0800 From: Andrew Morton Subject: Re: [PATCH 1/2] mm: Export vm_committed_as Message-Id: <20121105134456.f655b85a.akpm@linux-foundation.org> In-Reply-To: <426367E2313C2449837CD2DE46E7EAF930DF7FBB@SN2PRD0310MB382.namprd03.prod.outlook.com> References: <1349654347-18337-1-git-send-email-kys@microsoft.com> <1349654386-18378-1-git-send-email-kys@microsoft.com> <20121008004358.GA12342@kroah.com> <426367E2313C2449837CD2DE46E7EAF930A1FB31@SN2PRD0310MB382.namprd03.prod.outlook.com> <20121008133539.GA15490@kroah.com> <20121009124755.ce1087b4.akpm@linux-foundation.org> <426367E2313C2449837CD2DE46E7EAF930DF7FBB@SN2PRD0310MB382.namprd03.prod.outlook.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: KY Srinivasan Cc: Greg KH , "olaf@aepfle.de" , "linux-kernel@vger.kernel.org" , "andi@firstfloor.org" , "apw@canonical.com" , "devel@linuxdriverproject.org" , linux-mm@kvack.org, Hiroyuki Kamezawa , Michal Hocko , Johannes Weiner , Ying Han On Sat, 3 Nov 2012 14:09:38 +0000 KY Srinivasan wrote: > > > > > > > > Ok, but you're going to have to get the -mm developers to agree that > > > this is ok before I can accept it. > > > > Well I guess it won't kill us. > > Andrew, > > I presumed this was an Ack from you with regards to exporting the > symbol. Looks like Greg is waiting to hear from you before he can check > these patches in. Could you provide an explicit Ack. > Well, I do have some qualms about exporting vm_committed_as to modules. vm_committed_as is a global thing and only really makes sense in a non-containerised system. If the application is running within a memory cgroup then vm_enough_memory() and the global overcommit policy are at best irrelevant and misleading. If use of vm_committed_as is indeed a bad thing, then exporting it to modules might increase the amount of badness in the kernel. I don't think these qualms are serious enough to stand in the way of this patch, but I'd be interested in hearing the memcg developers' thoughts on the matter? Perhaps you could provide a detailed description of why your module actually needs this? Precisely what information is it looking for and why? If we know that then perhaps a more comfortable alternative can be found. -- 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/ . Don't email: email@kvack.org