All of lore.kernel.org
 help / color / mirror / Atom feed
From: Vlastimil Babka <vbabka@suse.cz>
To: Hyeonggon Yoo <42.hyeyoo@gmail.com>
Cc: linux-mm@kvack.org, Christoph Lameter <cl@linux.com>,
	Pekka Enberg <penberg@kernel.org>,
	David Rientjes <rientjes@google.com>,
	Joonsoo Kim <iamjoonsoo.kim@lge.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	linux-kernel@vger.kernel.org
Subject: Re: [QUESTION] is SLAB considered legacy and deprecated?
Date: Fri, 1 Oct 2021 16:07:56 +0200	[thread overview]
Message-ID: <b3c5bd9a-5fbc-8388-a4ab-6c20864dc7c6@suse.cz> (raw)
In-Reply-To: <20210928111231.GA2596@linux.asia-northeast3-a.c.our-ratio-313919.internal>

On 9/28/21 13:12, Hyeonggon Yoo wrote:
>>> So there is a fundamental question coming into my mind:
>>> 	'is SLAB considered legacy and deprecated?'
>>
>> To some extend, but not yet in a sense where we would have a deadline to get
>> rid of it. 
> 
> What makes you to say 'To some extent'?
> That's what I'm curious about :)

"To some extent" because SLUB is default and if some new stuff was added
that only worked with SLUB and not SLAB, there wouldn't be major
objections expected.

>> In some contexts it's still being preferred, AFAIK.
> 
> In what context is SLAB or SLUB is preferred?

I don't know the full answer. With our distro we have used SLAB, and
switched to SLUB after verifying that there are no major regressions.
Better debugging features were perhaps the major reason.

> And what is the core reason that SLUB is used by default?

The usual reason in MM, nobody objected :)
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=a0acd82080768

>>> It seems there are little development on SLAB and people think that
>>> SLAB is legacy and deprecated, so CONFIG_SLUB is used by default.
>>>
>>> But I think both has pros and cons for their own:
>>> 	SLAB: more temporal locality (cache friendly)
>>> 	but high usage of memory, and less spatial locality (TLB misses) than SLUB.
>>>
>>> 	SLUB: less temporal locality (less cache friendly) than SLAB
>>> 	but more spatial locality (TLB hit), and low usage of memory
>>> 	and good debugging feature.
>>
>> I'm not so sure about the usage of memory, SLUB can easily use more I
>> believe. Instead of caching some arrays of objects it will have one or more
>> private slabs per cpu, and the slabs are larger-order pages.
>>
> 
> SLAB uses much memory than slab, when there are *lots* of NUMA nodes.
> 	because kmem_cache_node exist per node and it keeps alien cache
> 	(used for caching freeing object from remote node),
> 	so it keeps MAX_NUMANODES ^ 2 of array_cache per slab.
> 	that is why I said SLAB uses much memory than SLUB.

Maybe, theoretically. But SLUB has per-cpu partial lists caching whole
slab pages, not just arrays of objects. Number of cpus will generally
grow with number of NUMA nodes.

> to quote Christoph Lameter's (who made SLUB and current maintainer) presentation in 2014:
> 	See link [6] for this
> 	p21: "SLAB queuing can get intensive memory usage going grows
> 	exponentially by NUMA node"
> 
> But I'm not sure if SLAB or SLUB is good on UMA architecture.
> 
>>> Why do people say SLAB is deprecated/legacy?
>>
>> Do they?
>>
> 
> Yes I think they do.
> 
> One example is presentation of David Rientjes (one of maintainers) in 2011:
> 	See link [7]
> 	p5: "SLAB is deprecated, very little development"
> 	p5: "Many distributions still ship with CONFIG_SLAB even
> 		though it is not the kernel default"
> 
> Anyway, I wanted to discuss the current status of SLAB.
> 
> To me, it seems that people do not work on SLAB lately
> and it seems some maintainers (David, Christoph) think
> it's better to move from SLAB to SLUB. one of them is Christoph himself.
> As he made SLUB because he don't agree on design of SLAB. [8]

Right.
Here's the discussion from the last time SLAB removal was proposed and
rejected, feel free to evaluate if the points there are still valid today:

https://lore.kernel.org/all/20190410024714.26607-1-tobin@kernel.org/

> Thanks,
> Hyeonggon.
> 
> Searching his name and sort by date
> [1] https://lore.kernel.org/lkml/?q=Christoph+Lameter&o=10800
> 
> [GIT PULL] SLAB updates for 2.6.37-rc1 by Pekka Enberg
> [2] https://lore.kernel.org/lkml/alpine.DEB.2.00.1010242005280.4447@tiger/#r
> 
> His work on lockless fastpath of slub (2011-02),
> [3] https://lore.kernel.org/lkml/20110225173850.486326452@linux.com/
> 
> His work on lockless fastpath of slub (2011-06)
> [4] https://lore.kernel.org/lkml/20110601172543.437240675@linux.com/
> 
> His work on per cpu partial lists
> [5] https://lore.kernel.org/lkml/20110809211221.831975979@linux.com/
> 
> Christoph's presentation on slab
> [6] - https://events.static.linuxfound.org/sites/events/files/slides/slaballocators.pdf
> 
> David's presentation on slab
> [7] - https://www.socallinuxexpo.org/scale9x-media/scalemedia/scale/scale9x-media/simple_cfp/presentations/16_30-DavidRientjes-Status_of_the_Linux_Slab_Allocators.pdf
> 
> [8] SLUB: The unqueued slab allocator V6
> https://lwn.net/Articles/229096/
> 


  parent reply	other threads:[~2021-10-01 14:08 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-27  9:03 [QUESTION] is SLAB considered legacy and deprecated? Hyeonggon Yoo
2021-09-27 17:03 ` Vlastimil Babka
2021-09-28 11:12   ` Hyeonggon Yoo
2021-09-29 10:50     ` Hyeonggon Yoo
2021-10-01 14:07     ` Vlastimil Babka [this message]
2021-10-03  5:59       ` Hyeonggon Yoo
2021-10-04  1:25         ` David Rientjes
2021-10-04  1:25           ` David Rientjes
2021-10-04  6:01           ` Hyeonggon Yoo
2021-10-04 11:39             ` Vlastimil Babka
2021-10-05 13:31               ` Hyeonggon Yoo
2021-10-04 11:34         ` Vlastimil Babka
2021-10-04 14:22           ` Hyeonggon Yoo
2021-10-01  0:39   ` Queueing is outside of SLUB nowdays Hyeonggon Yoo
2021-10-04 14:56     ` Christoph Lameter
2021-10-04 14:56       ` Christoph Lameter
2021-10-05  8:19       ` Vlastimil Babka
2021-10-05 14:16         ` Hyeonggon 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=b3c5bd9a-5fbc-8388-a4ab-6c20864dc7c6@suse.cz \
    --to=vbabka@suse.cz \
    --cc=42.hyeyoo@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=cl@linux.com \
    --cc=iamjoonsoo.kim@lge.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=penberg@kernel.org \
    --cc=rientjes@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 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.