Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Mime-Version: 1.0
Subject: Re: [PATCH 1/1] stackdepot: interface to check entries and size of
 stackdepot.
Reply-To: maninder1.s@samsung.com
From:   Maninder Singh <maninder1.s@samsung.com>
To:     Michal Hocko <mhocko@kernel.org>
CC:     "kstewart@linuxfoundation.org" <kstewart@linuxfoundation.org>,
        "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
        "jkosina@suse.cz" <jkosina@suse.cz>,
        "pombredanne@nexb.com" <pombredanne@nexb.com>,
        "jpoimboe@redhat.com" <jpoimboe@redhat.com>,
        "akpm@linux-foundation.org" <akpm@linux-foundation.org>,
        "vbabka@suse.cz" <vbabka@suse.cz>,
        "guptap@codeaurora.org" <guptap@codeaurora.org>,
        "vinmenon@codeaurora.org" <vinmenon@codeaurora.org>,
        AMIT SAHRAWAT <a.sahrawat@samsung.com>,
        PANKAJ MISHRA <pankaj.m@samsung.com>,
        Lalit Mohan Tripathi <lalit.mohan@samsung.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-mm@kvack.org" <linux-mm@kvack.org>,
        Vaneet Narang <v.narang@samsung.com>
In-Reply-To: <20171123162835.6prpgrz3qkdexx56@dhcp22.suse.cz>
Message-ID: <20171124094108epcms5p396558828a365a876d61205b0fdb501fd@epcms5p3>
Date:   Fri, 24 Nov 2017 09:41:08 +0000
Content-Transfer-Encoding: 7bit
Content-Type: text/plain; charset="utf-8"
CMS-TYPE: 105P
DLP-Filter: Pass
References: <20171123162835.6prpgrz3qkdexx56@dhcp22.suse.cz>
        <1511347661-38083-1-git-send-email-maninder1.s@samsung.com>
        <CGME20171122105142epcas5p173b7205da12e1fc72e16ec74c49db665@epcms5p3>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Hi Michal,
  
> On Wed 22-11-17 16:17:41, Maninder Singh wrote:
> > This patch provides interface to check all the stack enteries
> > saved in stackdepot so far as well as memory consumed by stackdepot.
> > 
> > 1) Take current depot_index and offset to calculate end address for one
> >         iteration of (/sys/kernel/debug/depot_stack/depot_entries).
> > 
> > 2) Fill end marker in every slab to point its end, and then use it while
> >         traversing all the slabs of stackdepot.
> > 
> > "debugfs code inspired from page_onwer's way of printing BT"
> > 
> > checked on ARM and x86_64.
> > $cat /sys/kernel/debug/depot_stack/depot_size
> > Memory consumed by Stackdepot:208 KB
> > 
> > $ cat /sys/kernel/debug/depot_stack/depot_entries
> > stack count 1 backtrace
> >  init_page_owner+0x1e/0x210
> >  start_kernel+0x310/0x3cd
> >  secondary_startup_64+0xa5/0xb0
> >  0xffffffffffffffff
>  
> Why do we need this? Who is goging to use this information and what for?
> I haven't looked at the code but just the diffstat looks like this
> should better have a _very_ good justification to be considered for
> merging. To be honest with you I have hard time imagine how this can be
> useful other than debugging stack depot...

This interface can be used for multiple reasons as:

1) For debugging stackdepot for sure.
2) For checking all the unique allocation paths in system.
3) To check if any invalid stack is coming which is increasing 
stackdepot memory.
(https://lkml.org/lkml/2017/10/11/353)

Althoutgh this needs to be taken care in ARM as replied by maintainer, but with help
of this interface it was quite easy to check and we added workaround for saving memory.

4) At some point of time to check current memory consumed by stackdepot.
5) To check number of entries in stackdepot to decide stackdepot hash size for different systems. 
   For fewer entries hash table size can be reduced from 4MB. 

Thanks
Maninder Singh

From 1584874989254914530@xxx Thu Nov 23 16:29:31 +0000 2017
X-GM-THRID: 1584763196629983480
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread