From: Barry Song <21cnbao@gmail.com>
To: "Huang, Ying" <ying.huang@intel.com>
Cc: akpm@linux-foundation.org, linux-mm@kvack.org,
linux-kernel@vger.kernel.org, Barry Song <v-songbaohua@oppo.com>,
Baolin Wang <baolin.wang@linux.alibaba.com>,
David Hildenbrand <david@redhat.com>,
Chris Li <chrisl@kernel.org>,
Yosry Ahmed <yosryahmed@google.com>,
Kairui Song <kasong@tencent.com>,
Ryan Roberts <ryan.roberts@arm.com>,
Kanchana P Sridhar <kanchana.p.sridhar@intel.com>,
Usama Arif <usamaarif642@gmail.com>
Subject: Re: [PATCH v3] mm: add per-order mTHP swpin counters
Date: Thu, 31 Oct 2024 21:21:24 +1300 [thread overview]
Message-ID: <CAGsJ_4yoH2qVpiVmYhYH6WTN4yueQ4yYw+P-CG4Q-dKjc9VVRQ@mail.gmail.com> (raw)
In-Reply-To: <874j4sae8y.fsf@yhuang6-desk2.ccr.corp.intel.com>
On Thu, Oct 31, 2024 at 9:09 PM Huang, Ying <ying.huang@intel.com> wrote:
>
> Barry Song <21cnbao@gmail.com> writes:
>
> > From: Barry Song <v-songbaohua@oppo.com>
> >
> > This helps profile the sizes of folios being swapped in. Currently,
> > only mTHP swap-out is being counted.
> > The new interface can be found at:
> > /sys/kernel/mm/transparent_hugepage/hugepages-<size>/stats/swpin
> > For example,
> > $ cat /sys/kernel/mm/transparent_hugepage/hugepages-64kB/stats/swpin
> > 12809
> > $ cat /sys/kernel/mm/transparent_hugepage/hugepages-32kB/stats/swpin
> > 4763
> >
> > Signed-off-by: Barry Song <v-songbaohua@oppo.com>
> > Reviewed-by: Baolin Wang <baolin.wang@linux.alibaba.com>
> > Acked-by: David Hildenbrand <david@redhat.com>
> > Cc: Chris Li <chrisl@kernel.org>
> > Cc: Yosry Ahmed <yosryahmed@google.com>
> > Cc: "Huang, Ying" <ying.huang@intel.com>
> > Cc: Kairui Song <kasong@tencent.com>
> > Cc: Ryan Roberts <ryan.roberts@arm.com>
> > Cc: Kanchana P Sridhar <kanchana.p.sridhar@intel.com>
> > Cc: Usama Arif <usamaarif642@gmail.com>
> > ---
> > -v3: minor refine changelog per Huang, Ying. thanks!
> >
> > Documentation/admin-guide/mm/transhuge.rst | 4 ++++
> > include/linux/huge_mm.h | 1 +
> > mm/huge_memory.c | 3 +++
> > mm/page_io.c | 3 +++
> > 4 files changed, 11 insertions(+)
> >
> > diff --git a/Documentation/admin-guide/mm/transhuge.rst b/Documentation/admin-guide/mm/transhuge.rst
> > index 2a171ed5206e..5caa3fb2feb1 100644
> > --- a/Documentation/admin-guide/mm/transhuge.rst
> > +++ b/Documentation/admin-guide/mm/transhuge.rst
> > @@ -534,6 +534,10 @@ zswpout
> > is incremented every time a huge page is swapped out to zswap in one
> > piece without splitting.
> >
> > +swpin
> > + is incremented every time a huge page is swapped in from a non-zswap
> > + swap device in one piece.
> > +
> > swpout
> > is incremented every time a huge page is swapped out to a non-zswap
> > swap device in one piece without splitting.
>
> In Documentation/ABI/testing/sysfs-kernel-mm-transparent-hugepage, I
> found
>
> "
> What: /sys/kernel/mm/transparent_hugepage/
> Date: April 2024
> Contact: Linux memory management mailing list <linux-mm@kvack.org>
> Description:
> /sys/kernel/mm/transparent_hugepage/ contains a number of files and
> subdirectories,
>
> - defrag
> - enabled
> - hpage_pmd_size
> - khugepaged
> - shmem_enabled
> - use_zero_page
> - subdirectories of the form hugepages-<size>kB, where <size>
> is the page size of the hugepages supported by the kernel/CPU
> combination.
>
> See Documentation/admin-guide/mm/transhuge.rst for details.
> "
>
> So, Documentation/admin-guide/mm/transhuge.rst will be used as ABI
> document?
Yes. this follows:
sysfs-kernel-mm-hugepages: See
Documentation/admin-guide/mm/hugetlbpage.rst for details.
sysfs-kernel-mm-ksm: See Documentation/mm/ksm.rst for more
information.
sysfs-kernel-mm-transparent-hugepage: See
Documentation/admin-guide/mm/transhuge.rst for details.
>
> [snip]
>
> --
> Best Regards,
> Huang, Ying
Thanks
barry
next prev parent reply other threads:[~2024-10-31 8:21 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-31 7:53 Barry Song
2024-10-31 8:05 ` Huang, Ying
2024-10-31 8:21 ` Barry Song [this message]
2024-10-31 8:25 ` Huang, Ying
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=CAGsJ_4yoH2qVpiVmYhYH6WTN4yueQ4yYw+P-CG4Q-dKjc9VVRQ@mail.gmail.com \
--to=21cnbao@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=baolin.wang@linux.alibaba.com \
--cc=chrisl@kernel.org \
--cc=david@redhat.com \
--cc=kanchana.p.sridhar@intel.com \
--cc=kasong@tencent.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=ryan.roberts@arm.com \
--cc=usamaarif642@gmail.com \
--cc=v-songbaohua@oppo.com \
--cc=ying.huang@intel.com \
--cc=yosryahmed@google.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox