Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 16 May 2023 14:12:45 -0700
From:   Andrew Morton <akpm@linux-foundation.org>
To:     Oscar Salvador <osalvador@suse.de>
Cc:     linux-kernel@vger.kernel.org, linux-mm@kvack.org,
        Michal Hocko <mhocko@suse.com>,
        Vlastimil Babka <vbabka@suse.cz>,
        Waiman Long <longman@redhat.com>,
        Suren Baghdasaryan <surenb@google.com>,
        Marco Elver <elver@google.com>,
        Andrey Konovalov <andreyknvl@gmail.com>,
        Eric Dumazet <edumazet@google.com>,
        Alexander Potapenko <glider@google.com>
Subject: Re: [PATCH v5 0/3] page_owner: print stacks and their counter
Message-Id: <20230516141245.356200756ff3750ef4f12094@linux-foundation.org>
In-Reply-To: <20230516182537.3139-1-osalvador@suse.de>
References: <20230516182537.3139-1-osalvador@suse.de>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Precedence: bulk

On Tue, 16 May 2023 20:25:34 +0200 Oscar Salvador <osalvador@suse.de> wrote:

> 
> ...
>
> page_owner is a great debug functionality tool that gets us to know
> about all pages that have been allocated/freed and their stacktrace.
> This comes very handy when e.g: debugging leaks, as with some scripting
> we might be able to see those stacktraces that are allocating pages
> but not freeing theme.
> 
> ...
> 
>  include/linux/stackdepot.h |   8 +++
>  lib/stackdepot.c           | 128 ++++++++++++++++++++++++++++++++++---
>  mm/page_owner.c            |  54 ++++++++++++++++
>  3 files changed, 180 insertions(+), 10 deletions(-)

Documentation/mm/page_owner.rst is feeling very sad.

I'll toss it in there for some testing for now.  The changelogs would
benefit from a bit of proofreading, please - quite a few typos and
grammaros.