From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx140.postini.com [74.125.245.140]) by kanga.kvack.org (Postfix) with SMTP id 7387A6B0005 for ; Mon, 8 Apr 2013 15:34:36 -0400 (EDT) Date: Mon, 8 Apr 2013 12:34:34 -0700 From: Andrew Morton Subject: Re: [PATCH v8 1/3] mm: limit growth of 3% hardcoded other user reserve Message-Id: <20130408123434.1a6dd5fb7341b0243e9dcdb3@linux-foundation.org> In-Reply-To: <20130408190402.GA2321@localhost.localdomain> References: <20130408190402.GA2321@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Andrew Shewmaker Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, alan@lxorguk.ukuu.org.uk, simon.jeons@gmail.com, ric.masonn@gmail.com On Mon, 8 Apr 2013 15:04:02 -0400 Andrew Shewmaker wrote: > v8: > * Rebased onto v3.9-rc4-mmotm-2013-03-26-15-09 > > * Clarified reasoning between different calculations for > overcommit 'guess' and 'never modes in FAQ entry > "How do you calculate a minimum useful reserve?" > in response to Simon Jeons. > > * Added third patch in series to handle hot-added or hot-swapped > memory. Well here's the v7-plus-my-fixes to v8 delta: --- a/.gitignore~mm-limit-growth-of-3%-hardcoded-other-user-reserve-v8 +++ a/.gitignore @@ -30,6 +30,7 @@ modules.builtin *.lzma *.xz *.lzo +*.out *.patch *.gcno diff -puN include/linux/mm.h~mm-limit-growth-of-3%-hardcoded-other-user-reserve-v8 include/linux/mm.h --- a/include/linux/mm.h~mm-limit-growth-of-3%-hardcoded-other-user-reserve-v8 +++ a/include/linux/mm.h @@ -44,8 +44,6 @@ extern int sysctl_legacy_va_layout; #include #include -extern unsigned long sysctl_user_reserve_kbytes; - #define nth_page(page,n) pfn_to_page(page_to_pfn((page)) + (n)) /* to align the pointer to the (next) page boundary */ diff -puN kernel/sysctl.c~mm-limit-growth-of-3%-hardcoded-other-user-reserve-v8 kernel/sysctl.c --- a/kernel/sysctl.c~mm-limit-growth-of-3%-hardcoded-other-user-reserve-v8 +++ a/kernel/sysctl.c @@ -97,6 +97,7 @@ /* External variables not in a header file. */ extern int sysctl_overcommit_memory; extern int sysctl_overcommit_ratio; +extern unsigned long sysctl_user_reserve_kbytes; extern int max_threads; extern int suid_dumpable; #ifdef CONFIG_COREDUMP ie, it reverts my cleanup of the extern declaration and adds a random unchangelogged line to .gitignore. I'll grab the new changelog then ignore this update ;) -- 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