mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
To: K Prateek Nayak <kprateek.nayak@amd.com>
Cc: "Thomas Gleixner" <tglx@kernel.org>,
	"Ingo Molnar" <mingo@redhat.com>,
	linux-kernel@vger.kernel.org,
	"Peter Zijlstra" <peterz@infradead.org>,
	"Darren Hart" <dvhart@infradead.org>,
	"Davidlohr Bueso" <dave@stgolabs.net>,
	"André Almeida" <andrealmeid@igalia.com>
Subject: Re: [RFC PATCH] futex: Dynamically allocate futex_queues depending on nr_node_ids
Date: Wed, 25 Feb 2026 10:22:13 +0100	[thread overview]
Message-ID: <20260225092213.oRHJyLG5@linutronix.de> (raw)
In-Reply-To: <99a81417-5dbf-4c29-8dac-a9ed17ee24d6@amd.com>

On 2026-02-25 14:21:33 [+0530], K Prateek Nayak wrote:
Hi Prateek,

> I would have thought a quarter of that would be plenty but looking at
> the footnote in [1] that says "16 socket GNR system" and the fact that
> GNR can feature up to 256 threads per socket - that could theoretically
> put such systems at that NR_CPUS_DEFAULT limit - I don't know if it is
> practically possible.
> 
> [1] https://lore.kernel.org/lkml/aYPjOgiO_XsFWnWu@hpe.com/
> 
> Still, I doubt such setup would practically cross more than 64 nodes.

I am still trying to figure out if this is practical or some drunk guys
saying "you know what would be fun?"

> Why was this selected as the default for MAXSMP? It came from [2] but
> I'm not really able to understand why other than this line in Mike's
> response:
> 
>     "MAXSMP" represents what's really usable
> 
> so we just set it to the max of range to test for scalability? Seems
> little impractical for real-world cases but on the flip side if we
> don't sit it, some bits might not get enough testing?

Sounds like it. What would be sane default upper limit then? Something
like 1024 CPUs? 2048? Or even more than that?

I would try to use this and convince Debian to drop MAXSMP and then
lower NODES_SHIFT to default 6. I would need a default for
NR_CPUS_DEFAULT without having people complaining about missing CPUs.
Maybe we could get a sane default setting in kernel without testing
limits.

Also probably will compile two kernels to see how much memory this safes
in total since there should be other data structures depending on max
CPUs/ NODEs.

Sebastian

  reply	other threads:[~2026-02-25  9:22 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-28 10:13 K Prateek Nayak
2026-02-24 11:13 ` Sebastian Andrzej Siewior
2026-02-25  3:36   ` K Prateek Nayak
2026-02-25  7:39     ` Sebastian Andrzej Siewior
2026-02-25  8:51       ` K Prateek Nayak
2026-02-25  9:22         ` Sebastian Andrzej Siewior [this message]
2026-02-27  8:47           ` K Prateek Nayak
2026-02-27 15:15             ` Sebastian Andrzej Siewior
2026-02-27 16:04               ` K Prateek Nayak
2026-02-27 14:42 ` Peter Zijlstra
2026-02-27 14:59   ` K Prateek Nayak
2026-02-27 16:18     ` Peter Zijlstra
2026-02-27 17:12       ` Sebastian Andrzej Siewior
2026-02-27 17:18         ` Peter Zijlstra
2026-03-02  4:59       ` K Prateek Nayak

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=20260225092213.oRHJyLG5@linutronix.de \
    --to=bigeasy@linutronix.de \
    --cc=andrealmeid@igalia.com \
    --cc=dave@stgolabs.net \
    --cc=dvhart@infradead.org \
    --cc=kprateek.nayak@amd.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@redhat.com \
    --cc=peterz@infradead.org \
    --cc=tglx@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®