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>
