From: Lai Jiangshan <jiangshanlai@gmail.com>
To: linux-kernel@vger.kernel.org
Cc: Borislav Petkov <bp@alien8.de>,
Peter Zijlstra <peterz@infradead.org>,
Josh Poimboeuf <jpoimboe@redhat.com>,
Andy Lutomirski <luto@kernel.org>,
Thomas Gleixner <tglx@linutronix.de>,
Juergen Gross <jgross@suse.com>,
x86@kernel.org, Lai Jiangshan <jiangshan.ljs@antgroup.com>
Subject: [PATCH V6 0/8] x86/entry: Clean up entry code
Date: Thu, 21 Apr 2022 22:10:47 +0800 [thread overview]
Message-ID: <20220421141055.316239-1-jiangshanlai@gmail.com> (raw)
From: Lai Jiangshan <jiangshan.ljs@antgroup.com>
This patchset moves the stack-switch code to the place where
error_entry() return, unravels error_entry() from XENpv and makes
entry_INT80_compat() uses idtentry macro.
This patchset is highly related to XENpv, because it does the extra
cleanup to convert SWAPGS to swapgs after major cleanup is done.
The patches are the 6th version to pick patches from the patchset
https://lore.kernel.org/lkml/20211126101209.8613-1-jiangshanlai@gmail.com/
which converts ASM code to C code. These patches are prepared for that
purpose. But this patchset has it own value: it simplifies the stack
switch, avoids leaving the old stack inside a function call, and
separates XENpv code with native code without adding new code.
Peter said in V3:
> So AFAICT these patches are indeed correct.
>
> I'd love for some of the other x86 people to also look at this,
> but a tentative ACK on this.
[V5]: https://lore.kernel.org/lkml/20220412121541.4595-1-jiangshanlai@gmail.com/
[V4]: https://lore.kernel.org/lkml/20220318143016.124387-1-jiangshanlai@gmail.com/
[V3]: https://lore.kernel.org/lkml/20220315073949.7541-1-jiangshanlai@gmail.com/
[V2]: https://lore.kernel.org/lkml/20220303035434.20471-1-jiangshanlai@gmail.com/
[V1]: https://lore.kernel.org/lkml/20211208110833.65366-1-jiangshanlai@gmail.com/
Changed from V5:
Add a new ASM function to wrap PUSH_AND_CLEAR_REGS rather than
inline it in the marco idtentry to reduce text size.
Remove the branch in sync_regs() (pick it from V1 and update
the changelog a little)
Changed from V4:
Update changelog largely of patch 1 and patch 2
Update changelog slightly of other patches
Unbreak the line of fixup_bad_iret() in patch1
Add Reviewed-by from Juergen Gross <jgross@suse.com> in patch 6
since he gave the Reviewed-by in one of the squashed patches.
Changed from V3:
Only reorder the int80 thing as the last patch to make patches
ordering more natural. (Both orders are correct)
Other interactions in V3:
Peter raised several questions and I think I answered them and I
don't think the code need to be updated unless I missed some
points. (Except reordering the patches)
Josh asked to remove UNWIND_HINT_REGS in patch5, but I think
UNWIND_HINT_REGS is old code before this patchset and I don't
want to do a cleanup that is not relate to preparing converting
ASM code C code in this patchset. He also asked to remove
ENCODE_FRAME_POINTER in xenpv case, and I think it just
complicates the code for just optimizing out a single assignment
to %rbp. I would not always stick to these reasons of mine,
but I just keep the code unchanged since he hasn't emphasized it
again nor other people has requested it.
Changed from V2:
Make the patch of folding int80 thing as the first patch
Add more changelog in "Switch the stack after error_entry() returns"
Changed from V1
Squash cleanup patches converting SWAPGS to swapgs into one patch
Use my official email address (Ant Group). The work is backed
by my company and I was incorrectly misunderstood that
XXX@linux.alibaba.com is the only portal for opensource work
in the corporate group.
Lai Jiangshan (8):
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 macro
x86/entry: Don't call error_entry() for XENPV
x86/entry: Convert SWAPGS to swapgs and remove the definition of
SWAPGS
x86/entry: Remove the branch in sync_regs()
x86/entry: Use idtentry macro for entry_INT80_compat()
arch/x86/entry/entry_64.S | 71 ++++++++++++++++-----
arch/x86/entry/entry_64_compat.S | 105 +------------------------------
arch/x86/include/asm/idtentry.h | 47 ++++++++++++++
arch/x86/include/asm/irqflags.h | 8 ---
arch/x86/include/asm/proto.h | 4 --
arch/x86/include/asm/traps.h | 2 +-
arch/x86/kernel/traps.c | 24 +++----
7 files changed, 112 insertions(+), 149 deletions(-)
--
2.19.1.6.gb485710b
next reply other threads:[~2022-04-21 14:10 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-21 14:10 Lai Jiangshan [this message]
2022-04-21 14:10 ` [PATCH V6 1/8] x86/traps: Move pt_regs only in fixup_bad_iret() Lai Jiangshan
2022-04-21 14:10 ` [PATCH V6 2/8] x86/entry: Switch the stack after error_entry() returns Lai Jiangshan
2022-04-21 14:10 ` [PATCH V6 3/8] x86/entry: Move PUSH_AND_CLEAR_REGS out of error_entry() Lai Jiangshan
2022-04-27 17:45 ` Borislav Petkov
2022-04-28 0:33 ` Lai Jiangshan
2022-04-28 10:26 ` Borislav Petkov
2022-05-02 12:42 ` Juergen Gross
2022-05-02 18:09 ` Borislav Petkov
2022-04-21 14:10 ` [PATCH V6 4/8] x86/entry: Move cld to the start of idtentry macro Lai Jiangshan
2022-04-21 14:10 ` [PATCH V6 5/8] x86/entry: Don't call error_entry() for XENPV Lai Jiangshan
2022-04-21 14:10 ` [PATCH V6 6/8] x86/entry: Convert SWAPGS to swapgs and remove the definition of SWAPGS Lai Jiangshan
2022-04-29 9:56 ` Borislav Petkov
2022-04-29 11:45 ` Lai Jiangshan
2022-04-29 12:22 ` Borislav Petkov
2022-05-02 12:18 ` Juergen Gross
2022-05-02 17:56 ` Borislav Petkov
2022-05-03 10:43 ` Borislav Petkov
2022-04-21 14:10 ` [PATCH V6 7/8] x86/entry: Remove the branch in sync_regs() Lai Jiangshan
2022-04-21 14:10 ` [PATCH V6 8/8] x86/entry: Use idtentry macro for entry_INT80_compat() Lai Jiangshan
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=20220421141055.316239-1-jiangshanlai@gmail.com \
--to=jiangshanlai@gmail.com \
--cc=bp@alien8.de \
--cc=jgross@suse.com \
--cc=jiangshan.ljs@antgroup.com \
--cc=jpoimboe@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=luto@kernel.org \
--cc=peterz@infradead.org \
--cc=tglx@linutronix.de \
--cc=x86@kernel.org \
/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
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.