* [PATCH 0/3] x86: cleanups
@ 2020-04-19 14:40 Lai Jiangshan
2020-04-19 14:40 ` [PATCH 1/3] x86/entry: remove unneeded kernel cr3 switching Lai Jiangshan
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Lai Jiangshan @ 2020-04-19 14:40 UTC (permalink / raw)
To: linux-kernel; +Cc: Lai Jiangshan
Simple cleanups found while reading the code.
Lai Jiangshan (3):
x86/entry: remove unneeded kernel cr3 switching
x86: remove address operator on function machine_check()
x86/entry: remove an unused label
arch/x86/entry/entry_64.S | 2 --
arch/x86/kernel/idt.c | 4 ++--
2 files changed, 2 insertions(+), 4 deletions(-)
--
2.20.1
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/3] x86/entry: remove unneeded kernel cr3 switching
2020-04-19 14:40 [PATCH 0/3] x86: cleanups Lai Jiangshan
@ 2020-04-19 14:40 ` Lai Jiangshan
2020-05-07 15:16 ` [tip: x86/entry] x86/entry/64: Remove unneeded kernel CR3 switching tip-bot2 for Lai Jiangshan
2020-04-19 14:40 ` [PATCH 2/3] x86: remove address operator on function machine_check() Lai Jiangshan
2020-04-19 14:40 ` [PATCH 3/3] x86/entry: remove an unused label Lai Jiangshan
2 siblings, 1 reply; 7+ messages in thread
From: Lai Jiangshan @ 2020-04-19 14:40 UTC (permalink / raw)
To: linux-kernel
Cc: Lai Jiangshan, Andy Lutomirski, Thomas Gleixner, Ingo Molnar,
Borislav Petkov, x86, H. Peter Anvin
When native_load_gs_index() fails on .Lgs_change, the cr3
must be kernel cr3. Don't need to switch it.
Signed-off-by: Lai Jiangshan <laijs@linux.alibaba.com>
---
arch/x86/entry/entry_64.S | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/x86/entry/entry_64.S b/arch/x86/entry/entry_64.S
index 0e9504fabe52..b789835afbab 100644
--- a/arch/x86/entry/entry_64.S
+++ b/arch/x86/entry/entry_64.S
@@ -1339,7 +1339,6 @@ SYM_CODE_START_LOCAL(error_entry)
*/
SWAPGS
FENCE_SWAPGS_USER_ENTRY
- SWITCH_TO_KERNEL_CR3 scratch_reg=%rax
jmp .Lerror_entry_done
.Lbstep_iret:
--
2.20.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/3] x86: remove address operator on function machine_check()
2020-04-19 14:40 [PATCH 0/3] x86: cleanups Lai Jiangshan
2020-04-19 14:40 ` [PATCH 1/3] x86/entry: remove unneeded kernel cr3 switching Lai Jiangshan
@ 2020-04-19 14:40 ` Lai Jiangshan
2020-05-07 15:16 ` [tip: x86/entry] x86/idt: Remove " tip-bot2 for Lai Jiangshan
2020-04-19 14:40 ` [PATCH 3/3] x86/entry: remove an unused label Lai Jiangshan
2 siblings, 1 reply; 7+ messages in thread
From: Lai Jiangshan @ 2020-04-19 14:40 UTC (permalink / raw)
To: linux-kernel
Cc: Lai Jiangshan, Thomas Gleixner, Ingo Molnar, Borislav Petkov,
x86, H. Peter Anvin, Alexios Zavras, Allison Randal,
Armijn Hemel
machine_check is function address, the address operator
on it is nop for compiler.
Make it consistent with the other function addresses in the
same file.
Signed-off-by: Lai Jiangshan <laijs@linux.alibaba.com>
---
arch/x86/kernel/idt.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/x86/kernel/idt.c b/arch/x86/kernel/idt.c
index 87ef69a72c52..98bcb502f967 100644
--- a/arch/x86/kernel/idt.c
+++ b/arch/x86/kernel/idt.c
@@ -93,7 +93,7 @@ static const __initconst struct idt_data def_idts[] = {
INTG(X86_TRAP_DB, debug),
#ifdef CONFIG_X86_MCE
- INTG(X86_TRAP_MC, &machine_check),
+ INTG(X86_TRAP_MC, machine_check),
#endif
SYSG(X86_TRAP_OF, overflow),
@@ -186,7 +186,7 @@ static const __initconst struct idt_data ist_idts[] = {
ISTG(X86_TRAP_NMI, nmi, IST_INDEX_NMI),
ISTG(X86_TRAP_DF, double_fault, IST_INDEX_DF),
#ifdef CONFIG_X86_MCE
- ISTG(X86_TRAP_MC, &machine_check, IST_INDEX_MCE),
+ ISTG(X86_TRAP_MC, machine_check, IST_INDEX_MCE),
#endif
};
--
2.20.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 3/3] x86/entry: remove an unused label
2020-04-19 14:40 [PATCH 0/3] x86: cleanups Lai Jiangshan
2020-04-19 14:40 ` [PATCH 1/3] x86/entry: remove unneeded kernel cr3 switching Lai Jiangshan
2020-04-19 14:40 ` [PATCH 2/3] x86: remove address operator on function machine_check() Lai Jiangshan
@ 2020-04-19 14:40 ` Lai Jiangshan
2020-05-07 15:16 ` [tip: x86/entry] x86/entry/64: Remove " tip-bot2 for Lai Jiangshan
2 siblings, 1 reply; 7+ messages in thread
From: Lai Jiangshan @ 2020-04-19 14:40 UTC (permalink / raw)
To: linux-kernel
Cc: Lai Jiangshan, Andy Lutomirski, Thomas Gleixner, Ingo Molnar,
Borislav Petkov, x86, H. Peter Anvin
The label .Lcommon_\sym was introduced by 39e9543344fa.
(x86-64: Reduce amount of redundant code generated for invalidate_interruptNN)
And all the other relevant information was removed by 52aec3308db8
(x86/tlb: replace INVALIDATE_TLB_VECTOR by CALL_FUNCTION_VECTOR)
Signed-off-by: Lai Jiangshan <laijs@linux.alibaba.com>
---
arch/x86/entry/entry_64.S | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/x86/entry/entry_64.S b/arch/x86/entry/entry_64.S
index b789835afbab..7d40d1034358 100644
--- a/arch/x86/entry/entry_64.S
+++ b/arch/x86/entry/entry_64.S
@@ -798,7 +798,6 @@ _ASM_NOKPROBE(common_interrupt)
SYM_CODE_START(\sym)
UNWIND_HINT_IRET_REGS
pushq $~(\num)
-.Lcommon_\sym:
call interrupt_entry
UNWIND_HINT_REGS indirect=1
call \do_sym /* rdi points to pt_regs */
--
2.20.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [tip: x86/entry] x86/idt: Remove address operator on function machine_check()
2020-04-19 14:40 ` [PATCH 2/3] x86: remove address operator on function machine_check() Lai Jiangshan
@ 2020-05-07 15:16 ` tip-bot2 for Lai Jiangshan
0 siblings, 0 replies; 7+ messages in thread
From: tip-bot2 for Lai Jiangshan @ 2020-05-07 15:16 UTC (permalink / raw)
To: linux-tip-commits; +Cc: Lai Jiangshan, Thomas Gleixner, x86, LKML
The following commit has been merged into the x86/entry branch of tip:
Commit-ID: 3dcdb8e0c83b9502f669106e17bfa795f19f8d9b
Gitweb: https://git.kernel.org/tip/3dcdb8e0c83b9502f669106e17bfa795f19f8d9b
Author: Lai Jiangshan <laijs@linux.alibaba.com>
AuthorDate: Sun, 19 Apr 2020 14:40:48
Committer: Thomas Gleixner <tglx@linutronix.de>
CommitterDate: Thu, 07 May 2020 17:12:40 +02:00
x86/idt: Remove address operator on function machine_check()
machine_check is function address, the address operator on it is nop for
compiler.
Make it consistent with the other function addresses in the same file.
Signed-off-by: Lai Jiangshan <laijs@linux.alibaba.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Link: https://lkml.kernel.org/r/20200419144049.1906-3-laijs@linux.alibaba.com
---
arch/x86/kernel/idt.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/x86/kernel/idt.c b/arch/x86/kernel/idt.c
index 87ef69a..98bcb50 100644
--- a/arch/x86/kernel/idt.c
+++ b/arch/x86/kernel/idt.c
@@ -93,7 +93,7 @@ static const __initconst struct idt_data def_idts[] = {
INTG(X86_TRAP_DB, debug),
#ifdef CONFIG_X86_MCE
- INTG(X86_TRAP_MC, &machine_check),
+ INTG(X86_TRAP_MC, machine_check),
#endif
SYSG(X86_TRAP_OF, overflow),
@@ -186,7 +186,7 @@ static const __initconst struct idt_data ist_idts[] = {
ISTG(X86_TRAP_NMI, nmi, IST_INDEX_NMI),
ISTG(X86_TRAP_DF, double_fault, IST_INDEX_DF),
#ifdef CONFIG_X86_MCE
- ISTG(X86_TRAP_MC, &machine_check, IST_INDEX_MCE),
+ ISTG(X86_TRAP_MC, machine_check, IST_INDEX_MCE),
#endif
};
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [tip: x86/entry] x86/entry/64: Remove unneeded kernel CR3 switching
2020-04-19 14:40 ` [PATCH 1/3] x86/entry: remove unneeded kernel cr3 switching Lai Jiangshan
@ 2020-05-07 15:16 ` tip-bot2 for Lai Jiangshan
0 siblings, 0 replies; 7+ messages in thread
From: tip-bot2 for Lai Jiangshan @ 2020-05-07 15:16 UTC (permalink / raw)
To: linux-tip-commits; +Cc: Lai Jiangshan, Thomas Gleixner, x86, LKML
The following commit has been merged into the x86/entry branch of tip:
Commit-ID: f642aebc9d2a51775d86eaa79da9d90aa5dff0f7
Gitweb: https://git.kernel.org/tip/f642aebc9d2a51775d86eaa79da9d90aa5dff0f7
Author: Lai Jiangshan <laijs@linux.alibaba.com>
AuthorDate: Sun, 19 Apr 2020 14:40:47
Committer: Thomas Gleixner <tglx@linutronix.de>
CommitterDate: Thu, 07 May 2020 17:12:39 +02:00
x86/entry/64: Remove unneeded kernel CR3 switching
When native_load_gs_index() fails on .Lgs_change, CR3 must be kernel
CR3. No need to switch it.
Signed-off-by: Lai Jiangshan <laijs@linux.alibaba.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Link: https://lkml.kernel.org/r/20200419144049.1906-2-laijs@linux.alibaba.com
---
arch/x86/entry/entry_64.S | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/x86/entry/entry_64.S b/arch/x86/entry/entry_64.S
index 3d747da..7d40d10 100644
--- a/arch/x86/entry/entry_64.S
+++ b/arch/x86/entry/entry_64.S
@@ -1338,7 +1338,6 @@ SYM_CODE_START_LOCAL(error_entry)
*/
SWAPGS
FENCE_SWAPGS_USER_ENTRY
- SWITCH_TO_KERNEL_CR3 scratch_reg=%rax
jmp .Lerror_entry_done
.Lbstep_iret:
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [tip: x86/entry] x86/entry/64: Remove an unused label
2020-04-19 14:40 ` [PATCH 3/3] x86/entry: remove an unused label Lai Jiangshan
@ 2020-05-07 15:16 ` tip-bot2 for Lai Jiangshan
0 siblings, 0 replies; 7+ messages in thread
From: tip-bot2 for Lai Jiangshan @ 2020-05-07 15:16 UTC (permalink / raw)
To: linux-tip-commits; +Cc: Lai Jiangshan, Thomas Gleixner, x86, LKML
The following commit has been merged into the x86/entry branch of tip:
Commit-ID: 4446d96d7ba7eaac54f9ef968bbe858097441d50
Gitweb: https://git.kernel.org/tip/4446d96d7ba7eaac54f9ef968bbe858097441d50
Author: Lai Jiangshan <laijs@linux.alibaba.com>
AuthorDate: Sun, 19 Apr 2020 14:40:49
Committer: Thomas Gleixner <tglx@linutronix.de>
CommitterDate: Thu, 07 May 2020 17:12:39 +02:00
x86/entry/64: Remove an unused label
The label .Lcommon_\sym was introduced by 39e9543344fa.
(x86-64: Reduce amount of redundant code generated for invalidate_interruptNN)
And all the other relevant information was removed by 52aec3308db8
(x86/tlb: replace INVALIDATE_TLB_VECTOR by CALL_FUNCTION_VECTOR)
Signed-off-by: Lai Jiangshan <laijs@linux.alibaba.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Link: https://lkml.kernel.org/r/20200419144049.1906-4-laijs@linux.alibaba.com
---
arch/x86/entry/entry_64.S | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/x86/entry/entry_64.S b/arch/x86/entry/entry_64.S
index 0e9504f..3d747da 100644
--- a/arch/x86/entry/entry_64.S
+++ b/arch/x86/entry/entry_64.S
@@ -798,7 +798,6 @@ _ASM_NOKPROBE(common_interrupt)
SYM_CODE_START(\sym)
UNWIND_HINT_IRET_REGS
pushq $~(\num)
-.Lcommon_\sym:
call interrupt_entry
UNWIND_HINT_REGS indirect=1
call \do_sym /* rdi points to pt_regs */
^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2020-05-07 15:17 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-19 14:40 [PATCH 0/3] x86: cleanups Lai Jiangshan
2020-04-19 14:40 ` [PATCH 1/3] x86/entry: remove unneeded kernel cr3 switching Lai Jiangshan
2020-05-07 15:16 ` [tip: x86/entry] x86/entry/64: Remove unneeded kernel CR3 switching tip-bot2 for Lai Jiangshan
2020-04-19 14:40 ` [PATCH 2/3] x86: remove address operator on function machine_check() Lai Jiangshan
2020-05-07 15:16 ` [tip: x86/entry] x86/idt: Remove " tip-bot2 for Lai Jiangshan
2020-04-19 14:40 ` [PATCH 3/3] x86/entry: remove an unused label Lai Jiangshan
2020-05-07 15:16 ` [tip: x86/entry] x86/entry/64: Remove " tip-bot2 for Lai Jiangshan
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.