mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Sohil Mehta <sohil.mehta@intel.com>
To: "Chang S. Bae" <chang.seok.bae@intel.com>,
	Dave Hansen <dave.hansen@intel.com>,
	"Maciej W. Rozycki" <macro@orcam.me.uk>
Cc: <linux-kernel@vger.kernel.org>, <x86@kernel.org>,
	<tglx@kernel.org>, "Ingo Molnar" <mingo@redhat.com>,
	<bp@alien8.de>, <dave.hansen@linux.intel.com>, <hpa@zytor.com>
Subject: Re: [PATCH] x86/microcode/intel: Panic on partial microcode update
Date: Mon, 27 Jul 2026 14:59:44 -0700	[thread overview]
Message-ID: <e97add0e-33f1-4837-9642-91e1d9773288@intel.com> (raw)
In-Reply-To: <a9262015-e403-4d36-a3a9-3e29f6f49318@intel.com>

On 7/27/2026 1:36 PM, Chang S. Bae wrote:
> I think the first two points have played in this thread already. There 
> seems to be some caution, specifically around the authentication-failure 
> as per spec. 

What does an authentication failure mean in this context?

Does that mean unauthenticated microcode components somehow got loaded?

	or that

Someone attempted to load bogus microcode which was rejected by the
processor?

If it was rejected, then there is no need to panic, right? Just warning
about the attempt should be enough.


> 
> In summary, I think options are:
> 
>   1. Panic on any partial update is the most conservative and secure
>      option, but it may unnecessarily disrupt systems and users

My understanding of a panic is the OS saying "something catastrophic has
happened and I don't know how to make forward progress". So, it's better
to crash at this point than spew out incoherent information.

>   2. Warn admins that the system has entered a taint state, assuming the
>      processor handles truly unrecoverable cases itself.

If the processor chooses not to terminate itself after a partial update
maybe the kernel shouldn't either and give the admin the opportunity to
reconcile at their convenience. I think a strong warning makes the most
sense here.

In addition to a taint, should there also be a WARN()? That gives any
paranoid admin the option to automatically convert WARN => PANIC via
panic_on_warn. They can set panic_on_warn at runtime just before the
microcode update and then clear it once the update is done.

There seems to be a kernel command line option called panic_on_taint
which can be configured to panic only for TAINT_CPU_OUT_OF_SPEC. But
that may not be very useful because there isn't any runtime sysctl option.

  reply	other threads:[~2026-07-27 21:59 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-30 19:13 Chang S. Bae
2026-06-30 21:38 ` Dave Hansen
2026-06-30 21:47   ` Dave Hansen
2026-06-30 23:21     ` Borislav Petkov
2026-07-01 19:02     ` Chang S. Bae
2026-07-18 22:54     ` Maciej W. Rozycki
2026-07-19 15:52       ` Dave Hansen
2026-07-23 22:42         ` Maciej W. Rozycki
2026-07-27 12:35           ` Nikolay Borisov
2026-07-28 21:53             ` Maciej W. Rozycki
2026-07-27 20:36         ` Chang S. Bae
2026-07-27 21:59           ` Sohil Mehta [this message]
2026-07-28  2:58             ` Chang S. Bae
2026-07-28 14:12               ` Dave Hansen
2026-07-28 18:56                 ` David Laight
2026-07-28 20:48                   ` Dave Hansen
2026-07-28 21:47                     ` Maciej W. Rozycki
2026-07-28 21:53                       ` Dave Hansen
2026-07-28 22:09                     ` David Laight
2026-07-08 21:18 ` [PATCH v2] x86/microcode/intel: Taint kernel on partial update Chang S. Bae
2026-07-27 11:27   ` Nikolay Borisov

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=e97add0e-33f1-4837-9642-91e1d9773288@intel.com \
    --to=sohil.mehta@intel.com \
    --cc=bp@alien8.de \
    --cc=chang.seok.bae@intel.com \
    --cc=dave.hansen@intel.com \
    --cc=dave.hansen@linux.intel.com \
    --cc=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=macro@orcam.me.uk \
    --cc=mingo@redhat.com \
    --cc=tglx@kernel.org \
    --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®