Re: [PATCH v6 4/9] x86/nmi: Assign and register NMI-source vectors

[Xin Li]

On 5/13/2025 1:37 PM, Sohil Mehta wrote:
> Prior to NMI-source support, the vector information was ignored by the
> hardware while delivering NMIs. With NMI-source, the architecture
> currently supports a 16-bit source bitmap to identify the source of the
> NMI. Upon receiving an NMI, this bitmap is delivered as part of the FRED
> event delivery mechanism to the kernel.
>
> Assign a vector space of 0-15 that is specific to NMI-source and
> independent of the IDT vector space of 0-255. Being a bitmap, the
> NMI-source vectors do not have any inherent priority associated with
> them. The order of executing the NMI handlers is up to the kernel.

I'm thinking should we mention that the bitmap could be extended more
than 16 bits in future?  Or we just don't emphasize 16-bit or 0~15?


> Existing NMI handling already has a priority mechanism for the NMI
> handlers, with CPU-specific (NMI_LOCAL) handlers executed first,
> followed by platform NMI handlers and unknown NMI (NMI_UNKNOWN) handlers
> being last. Within each of these NMI types, the handlers registered with
> NMI_FLAG_FIRST are given priority.
>
> NMI-source follows the same priority scheme to avoid unnecessary
> complexity. Therefore, the NMI-source vectors are assigned arbitrarily,
> except for vectors 0 and 2.
>
> Vector 0 is set by the hardware whenever a source vector was not used
> while generating an NMI or the originator could not be reliably
> identified. Do not assign it to any handler.
>
> Vector 2 is reserved for external NMIs corresponding to Local APIC -
> LINT1. Some third-party chipsets may send NMI messages with a hardcoded
> vector of 2, which would result in vector 2 being set in the NMI-source
> bitmap. To avoid confusion, do not assign vector 2 to any handler.
>
> NMI-source vectors are only assigned for NMI_LOCAL type handlers.
> Platform NMI handlers have a single handler registered per type. They
> don't need additional source information to differentiate among them.
>
> Use the assigned vectors to register the respective NMI handlers. Warn
> if the vector values are unexpected.
>
> A couple of NMI handlers, such as the microcode rendezvous and the crash
> reboot, do not use the typical NMI registration interface. Leave them
> as-is for now.
>
> Originally-by: Jacob Pan <jacob.jun.pan@xxxxxxxxxxxxxxx>
> Signed-off-by: Sohil Mehta <sohil.mehta@xxxxxxxxx>

Reviewed-by: Xin Li (Intel) <xin@xxxxxxxxx>