All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michal Hocko <mhocko@suse.com>
To: "zhen.ni" <zhen.ni@easystack.cn>
Cc: akpm@linux-foundation.org, vbabka@kernel.org, surenb@google.com,
	jackmanb@google.com, hannes@cmpxchg.org, ziy@nvidia.com,
	linux-mm@kvack.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v6 0/3] mm/page_owner: add filter infrastructure for print_mode and NUMA filtering
Date: Mon, 11 May 2026 14:54:30 +0200	[thread overview]
Message-ID: <agHRhlfIYbaqHHMK@tiehlicka> (raw)
In-Reply-To: <d19c927c-4af7-4c2d-8c62-82b8fc841089@easystack.cn>

On Mon 11-05-26 20:40:07, zhen.ni wrote:
> 
> 
> 在 2026/5/11 20:23, Michal Hocko 写道:
> > On Mon 11-05-26 11:30:14, Zhen Ni wrote:
> > > Solution
> > > ========
> > > 
> > > This patch series introduces a flexible filter infrastructure with
> > > two initial filters:
> > > 
> > > 1. **Print Mode Filter**: Outputs only stack handles instead of
> > >     full stack traces. The handle-to-stack mapping can be retrieved
> > >     from the existing show_stacks_handles interface. This dramatically
> > >     reduces output size while preserving all allocation metadata.
> > > 
> > > 2. **NUMA Node Filter**: Allows filtering pages by specific NUMA node(s)
> > >     using flexible nodelist format, enabling targeted analysis of memory
> > >     issues in NUMA-aware deployments.
> > 
> > How does this work when there are multiple consumers of the interface?
> > E.g per numa tool to watch node lock page_owner information?
> > 
> I understand your concern about concurrent access. Are you asking
> about this scenario?
> 
> Scenario: Multiple tools monitoring different NUMA nodes
>   Tool 1: echo "0" > nid && cat page_owner > node0.log
>   Tool 2: echo "1" > nid && cat page_owner > node1.log
> 
> The current global filter implementation would have race conditions
> in this case.

That makes the interface rather broken in my eyes TBH. Is there any way
to make the filter local to the fd?
-- 
Michal Hocko
SUSE Labs

  reply	other threads:[~2026-05-11 12:54 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-05-11  3:30 [PATCH v6 0/3] mm/page_owner: add filter infrastructure for print_mode and NUMA filtering Zhen Ni
2026-05-11  3:30 ` [PATCH v6 1/3] mm/page_owner: add print_mode filter Zhen Ni
2026-05-11  8:29   ` Oscar Salvador
2026-05-11 11:54     ` zhen.ni
2026-05-11  3:30 ` [PATCH v6 2/3] mm/page_owner: add NUMA node filter with nodelist support Zhen Ni
2026-05-11  8:54   ` Oscar Salvador
2026-05-11 12:24     ` zhen.ni
2026-05-11  3:30 ` [PATCH v6 3/3] mm/page_owner: document page_owner filter features Zhen Ni
2026-05-11  8:33   ` Oscar Salvador
2026-05-11 12:23 ` [PATCH v6 0/3] mm/page_owner: add filter infrastructure for print_mode and NUMA filtering Michal Hocko
2026-05-11 12:40   ` zhen.ni
2026-05-11 12:54     ` Michal Hocko [this message]
2026-05-12  3:11       ` zhen.ni
2026-05-12  7:26         ` Michal Hocko
2026-05-12  8:16           ` zhen.ni
2026-05-12  8:54             ` Michal Hocko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=agHRhlfIYbaqHHMK@tiehlicka \
    --to=mhocko@suse.com \
    --cc=akpm@linux-foundation.org \
    --cc=hannes@cmpxchg.org \
    --cc=jackmanb@google.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=surenb@google.com \
    --cc=vbabka@kernel.org \
    --cc=zhen.ni@easystack.cn \
    --cc=ziy@nvidia.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.