Re: [patch V2 03/17] x86/traps: Remove pointless irq enable from do_spurious_interrupt_bug()

From: Josh Poimboeuf
Date: Wed Oct 23 2019 - 17:31:21 EST


On Wed, Oct 23, 2019 at 02:27:08PM +0200, Thomas Gleixner wrote:
> That function returns immediately after conditionally reenabling interrupts which
> is more than pointless and requires the ASM code to disable interrupts again.
>
> Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> ---
> arch/x86/kernel/traps.c | 1 -
> 1 file changed, 1 deletion(-)
>
> --- a/arch/x86/kernel/traps.c
> +++ b/arch/x86/kernel/traps.c
> @@ -871,7 +871,6 @@ do_simd_coprocessor_error(struct pt_regs
> dotraplinkage void
> do_spurious_interrupt_bug(struct pt_regs *regs, long error_code)
> {
> - cond_local_irq_enable(regs);
> }

I think we can just remove this handler altogether. The Intel and AMD
manuals say vector 15 (X86_TRAP_SPURIOUS) is reserved.

--
Josh