All of lore.kernel.org
 help / color / mirror / Atom feed
From: Harry Yoo <harry@kernel.org>
To: Seongjun Hong <hsj0512@snu.ac.kr>,
	Vlastimil Babka <vbabka@kernel.org>,
	Andrew Morton <akpm@linux-foundation.org>
Cc: linux-mm@kvack.org, linux-doc@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH] docs: ABI: sysfs-kernel-slab: mark cpu_partial attributes deprecated
Date: Thu, 2 Jul 2026 14:18:49 +0900	[thread overview]
Message-ID: <54e75619-3dbc-448c-81fa-122edcdad1f0@kernel.org> (raw)
In-Reply-To: <20260701141755.85119-1-hsj0512@snu.ac.kr>


[-- Attachment #1.1: Type: text/plain, Size: 1396 bytes --]



On 7/1/26 11:17 PM, Seongjun Hong wrote:
> The per-cpu slab and per-cpu partial slab mechanisms were removed when
> SLUB was converted to per-cpu sheaves in Linux 7.0. The cpu_slabs,

small nit: sheaves were introduced in v6.18,
but the cpu (partial) slabs layer were removed later in v7.0.

> slabs_cpu_partial and cpu_partial sysfs attributes were kept as stubs
> that always return 0 for backwards compatibility, but their
> documentation still described them as if they were functional.

Right, for backward compatibility, because, unlike files that
might not exist w/o SLUB_STATS, userspace tools might assume that
they always exist.

> Update the three descriptions to state that the attributes are
> deprecated and always read 0, and note that they are retained only for
> compatibility. While here, fix a "partialli" typo in the
> slabs_cpu_partial description.
> 
> Signed-off-by: Seongjun Hong <hsj0512@snu.ac.kr>
> ---

Overall looks good to me, so:
Acked-by: Harry Yoo (Oracle) <harry@kernel.org>

By the way.

There's still some gap between the documentation
and the implementation :)

We've dropped a bunch of files that exist only on
SLUB_STATS and new files introduced with by
sheaves were not documented.

Also, tools/mm/slabinfo.c needs some attention as
it hasn't been updated for a while.

Thanks.

-- 
Cheers,
Harry / Hyeonggon

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

  reply	other threads:[~2026-07-02  5:18 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-01 14:17 [PATCH] docs: ABI: sysfs-kernel-slab: mark cpu_partial attributes deprecated Seongjun Hong
2026-07-02  5:18 ` Harry Yoo [this message]
2026-07-02  8:28   ` Vlastimil Babka (SUSE)
2026-07-03  5:12     ` Harry Yoo

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=54e75619-3dbc-448c-81fa-122edcdad1f0@kernel.org \
    --to=harry@kernel.org \
    --cc=akpm@linux-foundation.org \
    --cc=hsj0512@snu.ac.kr \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=vbabka@kernel.org \
    /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.