linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] sparc64: Make SLUB the default allocator
@ 2016-10-26 22:27 Nitin Gupta
  2016-10-27 14:55 ` David Miller
  0 siblings, 1 reply; 2+ messages in thread
From: Nitin Gupta @ 2016-10-26 22:27 UTC (permalink / raw)
  To: David S. Miller
  Cc: Nitin Gupta, David S. Miller, Sowmini Varadhan, Aaron Young,
	Borislav Petkov, sparclinux, linux-kernel

SLUB has better debugging support.

Signed-off-by: Nitin Gupta <nitin.m.gupta@oracle.com>
---
 arch/sparc/configs/sparc64_defconfig | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/arch/sparc/configs/sparc64_defconfig b/arch/sparc/configs/sparc64_defconfig
index 3583d67..0a615b0 100644
--- a/arch/sparc/configs/sparc64_defconfig
+++ b/arch/sparc/configs/sparc64_defconfig
@@ -7,7 +7,9 @@ CONFIG_LOG_BUF_SHIFT=18
 CONFIG_BLK_DEV_INITRD=y
 CONFIG_PERF_EVENTS=y
 # CONFIG_COMPAT_BRK is not set
-CONFIG_SLAB=y
+CONFIG_SLUB_DEBUG=y
+CONFIG_SLUB=y
+CONFIG_SLUB_CPU_PARTIAL=y
 CONFIG_PROFILING=y
 CONFIG_OPROFILE=m
 CONFIG_KPROBES=y
-- 
2.9.2

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] sparc64: Make SLUB the default allocator
  2016-10-26 22:27 [PATCH] sparc64: Make SLUB the default allocator Nitin Gupta
@ 2016-10-27 14:55 ` David Miller
  0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2016-10-27 14:55 UTC (permalink / raw)
  To: nitin.m.gupta; +Cc: sowmini.varadhan, aaron.young, bp, sparclinux, linux-kernel

From: Nitin Gupta <nitin.m.gupta@oracle.com>
Date: Wed, 26 Oct 2016 15:27:56 -0700

> SLUB has better debugging support.
> 
> Signed-off-by: Nitin Gupta <nitin.m.gupta@oracle.com>
 ...
> @@ -7,7 +7,9 @@ CONFIG_LOG_BUF_SHIFT=18
>  CONFIG_BLK_DEV_INITRD=y
>  CONFIG_PERF_EVENTS=y
>  # CONFIG_COMPAT_BRK is not set
> -CONFIG_SLAB=y
> +CONFIG_SLUB_DEBUG=y
> +CONFIG_SLUB=y
> +CONFIG_SLUB_CPU_PARTIAL=y
>  CONFIG_PROFILING=y
>  CONFIG_OPROFILE=m
>  CONFIG_KPROBES=y

This patch made me try to find guidance on other architectures as to what
they put in their defconfig files.

And it seems to me that they leave things like this to the user, and the
defconfig only really contains things that really are required by the
architecture or are architecture specific.

So maybe instead of making arbitrary choices for these values which
one set of users want and another set of users do not (I prefer SLAB,
it is faster, and I know what the downsides wrt. NUMA balancing
compared to SLAB are and am willing to accept that), we should do
what other platforms do and not specify things like this at all.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-10-27 14:55 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-10-26 22:27 [PATCH] sparc64: Make SLUB the default allocator Nitin Gupta
2016-10-27 14:55 ` David Miller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).