From: Thomas Gleixner <tglx@linutronix.de> To: LKML <linux-kernel@vger.kernel.org> Cc: x86@kernel.org, Steven Rostedt <rostedt@goodmis.org>, Brian Gerst <brgerst@gmail.com>, Juergen Gross <jgross@suse.com>, Paolo Bonzini <pbonzini@redhat.com>, Arnd Bergmann <arnd@arndb.de>, Sean Christopherson <sean.j.christopherson@intel.com>, Alexandre Chartre <alexandre.chartre@oracle.com>, "Peter Zijlstra (Intel)" <peterz@infradead.org> Subject: [patch 04/10] x86/traps: Remove pointless irq enable from do_spurious_interrupt_bug() Date: Tue, 25 Feb 2020 22:36:40 +0100 [thread overview] Message-ID: <20200225220216.518575042@linutronix.de> (raw) In-Reply-To: <20200225213636.689276920@linutronix.de> From: Thomas Gleixner <tglx@linutronix.de> 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@linutronix.de> Reviewed-by: Sean Christopherson <sean.j.christopherson@intel.com> Reviewed-by: Alexandre Chartre <alexandre.chartre@oracle.com> Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org> Link: https://lore.kernel.org/r/20191023123117.871608831@linutronix.de --- arch/x86/kernel/traps.c | 1 - 1 file changed, 1 deletion(-) --- a/arch/x86/kernel/traps.c +++ b/arch/x86/kernel/traps.c @@ -862,7 +862,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); } dotraplinkage void
next prev parent reply other threads:[~2020-02-25 23:26 UTC|newest] Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-02-25 21:36 [patch 00/10] x86/entry: Consolidation - Part I Thomas Gleixner 2020-02-25 21:36 ` [patch 01/10] x86/entry/32: Add missing ASM_CLAC to general_protection entry Thomas Gleixner 2020-02-26 1:00 ` Frederic Weisbecker 2020-02-27 14:15 ` [tip: x86/entry] " tip-bot2 for Thomas Gleixner 2020-02-25 21:36 ` [patch 02/10] x86/mce: Disable tracing and kprobes on do_machine_check() Thomas Gleixner 2020-02-26 1:13 ` Frederic Weisbecker 2020-02-26 5:29 ` Andy Lutomirski 2020-02-26 13:28 ` Peter Zijlstra 2020-02-26 15:10 ` Andy Lutomirski 2020-02-26 16:08 ` Peter Zijlstra 2020-02-26 17:28 ` Andy Lutomirski 2020-02-26 18:42 ` Borislav Petkov 2020-02-26 18:59 ` Peter Zijlstra 2020-02-26 19:09 ` Andy Lutomirski 2020-02-26 20:59 ` Steven Rostedt 2020-02-26 11:18 ` Borislav Petkov 2020-02-27 14:15 ` [tip: x86/entry] " tip-bot2 for Andy Lutomirski 2020-02-25 21:36 ` [patch 03/10] x86/entry/32: Force MCE through do_mce() Thomas Gleixner 2020-02-26 1:11 ` Frederic Weisbecker 2020-02-27 14:15 ` [tip: x86/entry] " tip-bot2 for Thomas Gleixner 2020-02-25 21:36 ` Thomas Gleixner [this message] 2020-02-26 1:19 ` [patch 04/10] x86/traps: Remove pointless irq enable from do_spurious_interrupt_bug() Frederic Weisbecker 2020-02-25 21:36 ` [patch 05/10] x86/traps: Document do_spurious_interrupt_bug() Thomas Gleixner 2020-02-26 17:08 ` Frederic Weisbecker 2020-02-27 14:15 ` [tip: x86/entry] " tip-bot2 for Thomas Gleixner 2020-02-25 21:36 ` [patch 06/10] x86/traps: Remove redundant declaration of do_double_fault() Thomas Gleixner 2020-02-27 14:15 ` [tip: x86/entry] " tip-bot2 for Thomas Gleixner 2020-02-25 21:36 ` [patch 07/10] x86/irq: Remove useless return value from do_IRQ() Thomas Gleixner 2020-02-27 14:15 ` [tip: x86/entry] " tip-bot2 for Thomas Gleixner 2020-02-25 21:36 ` [patch 08/10] x86/entry/32: Remove the 0/-1 distinction from exception entries Thomas Gleixner 2020-02-26 5:34 ` Andy Lutomirski 2020-02-26 18:42 ` Thomas Gleixner 2020-02-26 18:57 ` Andy Lutomirski 2020-02-26 19:15 ` Thomas Gleixner 2020-02-27 14:24 ` [patch V2 " Thomas Gleixner 2020-02-29 11:49 ` [tip: x86/entry] " tip-bot2 for Thomas Gleixner 2020-02-25 21:36 ` [patch 09/10] x86/entry/entry_32: Route int3 through common_exception Thomas Gleixner 2020-02-26 17:35 ` Frederic Weisbecker 2020-02-27 14:15 ` [tip: x86/entry] " tip-bot2 for Thomas Gleixner 2020-02-25 21:36 ` [patch 10/10] x86/traps: Stop using ist_enter/exit() in do_int3() Thomas Gleixner 2020-02-27 14:15 ` [tip: x86/entry] " tip-bot2 for Thomas Gleixner 2020-02-27 14:33 ` tip-bot2 for Andy Lutomirski 2020-02-26 5:26 ` [patch 00/10] x86/entry: Consolidation - Part I Andy Lutomirski 2020-02-26 5:35 ` Andy Lutomirski 2020-02-27 11:01 ` Alexandre Chartre
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=20200225220216.518575042@linutronix.de \ --to=tglx@linutronix.de \ --cc=alexandre.chartre@oracle.com \ --cc=arnd@arndb.de \ --cc=brgerst@gmail.com \ --cc=jgross@suse.com \ --cc=linux-kernel@vger.kernel.org \ --cc=pbonzini@redhat.com \ --cc=peterz@infradead.org \ --cc=rostedt@goodmis.org \ --cc=sean.j.christopherson@intel.com \ --cc=x86@kernel.org \ --subject='Re: [patch 04/10] x86/traps: Remove pointless irq enable from do_spurious_interrupt_bug()' \ /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
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.