From: Naveen N Rao <naveen@kernel.org>
To: Borislav Petkov <bp@alien8.de>
Cc: Dave Hansen <dave.hansen@linux.intel.com>,
linux-kernel@vger.kernel.org, x86@kernel.org,
Thomas Gleixner <tglx@linutronix.de>,
Ingo Molnar <mingo@redhat.com>, Bharata B Rao <bharata@amd.com>,
Manali Shukla <manali.shukla@amd.com>,
Nikunj A Dadhania <nikunj@amd.com>,
"H. Peter Anvin" <hpa@zytor.com>,
Robert Richter <rrichter@amd.com>,
Christian Ludloff <ludloff@gmail.com>
Subject: Re: [PATCH v5] x86/apic: Use EILVT register count from APIC_EFEAT
Date: Thu, 8 Oct 2026 14:25:35 +0530 [thread overview]
Message-ID: <asdZHtWamB6VewVB@blrnaveerao1> (raw)
In-Reply-To: <20260930031309.GEarx-RcbEf8tHWUmg@fat_crate.local>
[Restore CC list]
I messed up and dropped the Cc list in my reply to Boris, who noticed
and promptly pointed it out. Restoring the CC list so that the below
discussion is public.
On Tue, Sep 29, 2026 at 08:13:09PM -0700, Borislav Petkov wrote:
> On Tue, Sep 29, 2026 at 09:49:56PM +0530, Naveen N Rao wrote:
> > No, I think that defeats the purpose of this patch. Part of the
> > motivation for this patch is that we don't have to care if the EILVT
> > register count changes again.
>
> If you really think that for a new CPU, you won't have to touch the kernel,
> then you need to wake up and smell the coffee...
>
> :-)
One can hope ;)
>
> > Note that the issue Sashiko is referring to requires three things:
> > - that HW advertise an inflated EILVT Register count in APIC_EFEAT,
> > - that some other _new_ HW unit also advertises an offset > 175 for use
> > (the two existing units: IBS and MCE, use a 4-bit field to encode the
> > EILVT register number, so those can't trip this)
> > - and, that Linux stay in the old xAPIC mode (rather than x2APIC).
> >
> > If the first two are true, then that is hardware broken beyond help and
> > it is hard to imagine Linux additionally being used in xAPIC mode.
> >
> > This is certainly easier to trigger with virtualization, but there are
> > far worse ways for the hypervisor to mess with the guest.
> >
> > So, I don't think this alone is reason enough to clamp the value.
>
> ... but ok, you feel way stronger about this than me actually cares so sure,
> let's not do anything here. And yes, I kinda agree with some of your points
> too.
Thanks,
Naveen
next parent reply other threads:[~2026-10-08 8:59 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20260930031309.GEarx-RcbEf8tHWUmg@fat_crate.local>
2026-10-08 8:55 ` Naveen N Rao [this message]
2026-09-25 16:17 Naveen N Rao (AMD)
2026-09-26 0:38 ` Borislav Petkov
2026-09-28 6:50 ` Naveen N Rao
2026-09-29 3:27 ` Borislav Petkov
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=asdZHtWamB6VewVB@blrnaveerao1 \
--to=naveen@kernel.org \
--cc=bharata@amd.com \
--cc=bp@alien8.de \
--cc=dave.hansen@linux.intel.com \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=ludloff@gmail.com \
--cc=manali.shukla@amd.com \
--cc=mingo@redhat.com \
--cc=nikunj@amd.com \
--cc=rrichter@amd.com \
--cc=tglx@linutronix.de \
--cc=x86@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®