From: Thomas Gleixner <tglx@linutronix.de>
To: Michael Ellerman <mpe@ellerman.id.au>,
"Nysal Jan K.A." <nysal@linux.ibm.com>
Cc: Tyrel Datwyler <tyreld@linux.ibm.com>,
Laurent Dufour <ldufour@linux.ibm.com>,
Peter Zijlstra <peterz@infradead.org>,
Nicholas Piggin <npiggin@gmail.com>,
linux-kernel@vger.kernel.org,
Christophe Leroy <christophe.leroy@csgroup.eu>,
"Nysal Jan K.A" <nysal@linux.ibm.com>,
"Naveen N. Rao" <naveen.n.rao@linux.ibm.com>,
Michal Suchanek <msuchanek@suse.de>,
linuxppc-dev@lists.ozlabs.org
Subject: Re: [PATCH 0/2] Skip offline cores when enabling SMT on PowerPC
Date: Sun, 23 Jun 2024 22:14:51 +0200 [thread overview]
Message-ID: <87ikxza01w.ffs@tglx> (raw)
In-Reply-To: <875xudoz4d.fsf@mail.lhotse>
Michael!
On Thu, Jun 13 2024 at 21:34, Michael Ellerman wrote:
> IIUIC the regression was in the ppc64_cpu userspace tool, which switched
> to using the new kernel interface without taking into account the way it
> behaves.
>
> Or are you saying the kernel behaviour changed on x86 after the powerpc
> HOTPLUG_SMT was added?
No. The mechanism was always this way. Only offline nodes have been
skipped. x86 never checked for the core being online.
> It's only x86 and powerpc right?
>
> Having different behaviour on the only two arches that support the
> interface does not seem like a good result.
>
>> What is the expected behaviour on x86 when enabling SMT and certain cores
>> are offline?
>
> AFAIK no one really touches SMT on x86 other than to turn it off for
> security reasons.
Right. So changing it not to online a thread when the full core is
offline should not really break stuff.
OTH, the mechanism to figure that out on x86 is definitely different and
more complicated than on power because the sibling threads are not
having consecutive CPU numbers.
So I'm not sure whether it's worth to make this consistent and I
definitely can live with the proposed patches.
Thanks,
tglx
next prev parent reply other threads:[~2024-06-23 20:21 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-12 18:50 [PATCH 0/2] Skip offline cores when enabling SMT on PowerPC Nysal Jan K.A.
2024-06-12 18:50 ` [PATCH 1/2] cpu/SMT: Enable SMT only if a core is online Nysal Jan K.A.
2024-06-12 18:50 ` [PATCH 2/2] powerpc/topology: Check " Nysal Jan K.A.
2024-06-24 19:06 ` Shrikanth Hegde
2024-06-25 6:13 ` Nysal Jan K.A.
2024-06-13 11:34 ` [PATCH 0/2] Skip offline cores when enabling SMT on PowerPC Michael Ellerman
2024-06-13 12:07 ` Michal Suchánek
2024-06-14 3:52 ` Nysal Jan K.A.
2024-06-23 20:14 ` Thomas Gleixner [this message]
2024-06-24 19:11 ` Shrikanth Hegde
2024-06-24 21:24 ` Thomas Gleixner
2024-07-01 4:54 ` Shrikanth Hegde
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=87ikxza01w.ffs@tglx \
--to=tglx@linutronix.de \
--cc=christophe.leroy@csgroup.eu \
--cc=ldufour@linux.ibm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=mpe@ellerman.id.au \
--cc=msuchanek@suse.de \
--cc=naveen.n.rao@linux.ibm.com \
--cc=npiggin@gmail.com \
--cc=nysal@linux.ibm.com \
--cc=peterz@infradead.org \
--cc=tyreld@linux.ibm.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;
as well as URLs for NNTP newsgroup(s).