From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751800AbdBOO3V (ORCPT ); Wed, 15 Feb 2017 09:29:21 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:59326 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751749AbdBOO3U (ORCPT ); Wed, 15 Feb 2017 09:29:20 -0500 Date: Wed, 15 Feb 2017 06:29:14 -0800 From: Greg KH To: peter enderborg Cc: devel@driverdev.osuosl.org, riandrews@android.com, linux-kernel@vger.kernel.org, linux-mm@kvack.org, arve@android.com, torvalds@linux-foundation.org Subject: Re: [PATCH 1/3 staging-next] android: Collect statistics from lowmemorykiller Message-ID: <20170215142914.GB11454@kroah.com> References: <20170214160932.4988-1-peter.enderborg@sonymobile.com> <20170214165015.GD17335@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.7.2 (2016-11-26) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Feb 15, 2017 at 09:22:10AM +0100, peter enderborg wrote: > On 02/14/2017 05:50 PM, Greg KH wrote: > > On Tue, Feb 14, 2017 at 05:09:30PM +0100, peter.enderborg@sonymobile.com wrote: > >> From: Peter Enderborg > >> > >> This collects stats for shrinker calls and how much > >> waste work we do within the lowmemorykiller. > >> > >> Signed-off-by: Peter Enderborg > >> --- > >> drivers/staging/android/Kconfig | 11 ++++ > >> drivers/staging/android/Makefile | 1 + > >> drivers/staging/android/lowmemorykiller.c | 9 ++- > >> drivers/staging/android/lowmemorykiller_stats.c | 85 +++++++++++++++++++++++++ > >> drivers/staging/android/lowmemorykiller_stats.h | 29 +++++++++ > >> 5 files changed, 134 insertions(+), 1 deletion(-) > >> create mode 100644 drivers/staging/android/lowmemorykiller_stats.c > >> create mode 100644 drivers/staging/android/lowmemorykiller_stats.h > >> > >> diff --git a/drivers/staging/android/Kconfig b/drivers/staging/android/Kconfig > >> index 6c00d6f..96e86c7 100644 > >> --- a/drivers/staging/android/Kconfig > >> +++ b/drivers/staging/android/Kconfig > >> @@ -24,6 +24,17 @@ config ANDROID_LOW_MEMORY_KILLER > >> scripts (/init.rc), and it defines priority values with minimum free memory size > >> for each priority. > >> > >> +config ANDROID_LOW_MEMORY_KILLER_STATS > >> + bool "Android Low Memory Killer: collect statistics" > >> + depends on ANDROID_LOW_MEMORY_KILLER > >> + default n > >> + help > >> + Create a file in /proc/lmkstats that includes > >> + collected statistics about kills, scans and counts > >> + and interaction with the shrinker. Its content > >> + will be different depeding on lmk implementation used. > > Ick, no new /proc files please, this isn't a "process" value. What's > > wrong with debugfs? > This is intended for android. Android users are very limited in their access > to linux part of the system on commercial models and lmk activity has a bad impact on the performance > of the device. Even the application developers has not much access so it seems to be fair to give > the users the information about why there is a problem. Why would you want to give "all users" this information at all? This is a debugging tool, your debugging userspace framework can use whatever interface you create to access it (i.e. debugfs). Again, do not add debugging stuff to /proc/ that's not ok, sorry. thanks, greg k-h