Re: [PATCH V3 0/7] x86/entry: Clean up entry code

From: Peter Zijlstra
Date: Wed Mar 16 2022 - 11:13:40 EST


On Tue, Mar 15, 2022 at 03:39:42PM +0800, Lai Jiangshan wrote:

> Lai Jiangshan (7):
> x86/entry: Use idtentry macro for entry_INT80_compat
> x86/traps: Move pt_regs only in fixup_bad_iret()
> x86/entry: Switch the stack after error_entry() returns
> x86/entry: move PUSH_AND_CLEAR_REGS out of error_entry
> x86/entry: Move cld to the start of idtentry
> x86/entry: Don't call error_entry for XENPV
> x86/entry: Convert SWAPGS to swapgs and remove the definition of
> SWAPGS

So AFAICT these patches are indeed correct.

I do however worry a little bit about the I$ impact of patch 4, and
there's a few niggles, but otherwise looks good.

I'd love for some of the other x86 people to also look at this, but a
tentative ACK on this.