* [PATCH] x86/irq/64: fix the missing update on comment
@ 2019-07-19 8:16 Cao jin
2019-07-22 8:53 ` Thomas Gleixner
2019-07-22 8:58 ` [tip:x86/urgent] x86/irq/64: Update stale comment tip-bot for Cao jin
0 siblings, 2 replies; 4+ messages in thread
From: Cao jin @ 2019-07-19 8:16 UTC (permalink / raw)
To: x86, linux-kernel; +Cc: tglx, mingo, bp, hpa, luto
Commit e6401c130931 ("x86/irq/64: Split the IRQ stack into its own pages")
missed to update one piece of comment as it did to its peer in Xen, which
will confuse people who still need to read comment.
A bonus fix to identation in ZO's linker script: spaces -> tab.
Signed-off-by: Cao jin <caoj.fnst@cn.fujitsu.com>
---
arch/x86/boot/compressed/vmlinux.lds.S | 4 ++--
arch/x86/kernel/head_64.S | 8 ++++----
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/arch/x86/boot/compressed/vmlinux.lds.S b/arch/x86/boot/compressed/vmlinux.lds.S
index 508cfa6828c5..23100c52a7d0 100644
--- a/arch/x86/boot/compressed/vmlinux.lds.S
+++ b/arch/x86/boot/compressed/vmlinux.lds.S
@@ -64,8 +64,8 @@ SECTIONS
_ebss = .;
}
#ifdef CONFIG_X86_64
- . = ALIGN(PAGE_SIZE);
- .pgtable : {
+ . = ALIGN(PAGE_SIZE);
+ .pgtable : {
_pgtable = . ;
*(.pgtable)
_epgtable = . ;
diff --git a/arch/x86/kernel/head_64.S b/arch/x86/kernel/head_64.S
index bcd206c8ac90..cba94468795e 100644
--- a/arch/x86/kernel/head_64.S
+++ b/arch/x86/kernel/head_64.S
@@ -195,10 +195,10 @@ ENTRY(secondary_startup_64)
/* Set up %gs.
*
- * The base of %gs always points to the bottom of the irqstack
- * union. If the stack protector canary is enabled, it is
- * located at %gs:40. Note that, on SMP, the boot cpu uses
- * init data section till per cpu areas are set up.
+ * The base of %gs always points to fixed_percpu_data. If the
+ * stack protector canary is enabled, it is located at %gs:40.
+ * Note that, on SMP, the boot cpu uses init data section till
+ * per cpu areas are set up.
*/
movl $MSR_GS_BASE,%ecx
movl initial_gs(%rip),%eax
--
2.17.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] x86/irq/64: fix the missing update on comment
2019-07-19 8:16 [PATCH] x86/irq/64: fix the missing update on comment Cao jin
@ 2019-07-22 8:53 ` Thomas Gleixner
2019-07-22 9:13 ` Cao jin
2019-07-22 8:58 ` [tip:x86/urgent] x86/irq/64: Update stale comment tip-bot for Cao jin
1 sibling, 1 reply; 4+ messages in thread
From: Thomas Gleixner @ 2019-07-22 8:53 UTC (permalink / raw)
To: Cao jin; +Cc: x86, linux-kernel, mingo, bp, hpa, luto
Cao,
On Fri, 19 Jul 2019, Cao jin wrote:
> Commit e6401c130931 ("x86/irq/64: Split the IRQ stack into its own pages")
> missed to update one piece of comment as it did to its peer in Xen, which
> will confuse people who still need to read comment.
>
> A bonus fix to identation in ZO's linker script: spaces -> tab.
Please don't add 'bonus' changes. A patch which fixes a stale comment has
nothing to do with a indentation change in an unrelated file.
Thanks,
tglx
^ permalink raw reply [flat|nested] 4+ messages in thread
* [tip:x86/urgent] x86/irq/64: Update stale comment
2019-07-19 8:16 [PATCH] x86/irq/64: fix the missing update on comment Cao jin
2019-07-22 8:53 ` Thomas Gleixner
@ 2019-07-22 8:58 ` tip-bot for Cao jin
1 sibling, 0 replies; 4+ messages in thread
From: tip-bot for Cao jin @ 2019-07-22 8:58 UTC (permalink / raw)
To: linux-tip-commits; +Cc: linux-kernel, mingo, hpa, tglx, caoj.fnst
Commit-ID: 385065734cd417b9d7739b2ebb62c960aeb3ccb5
Gitweb: https://git.kernel.org/tip/385065734cd417b9d7739b2ebb62c960aeb3ccb5
Author: Cao jin <caoj.fnst@cn.fujitsu.com>
AuthorDate: Fri, 19 Jul 2019 16:16:35 +0800
Committer: Thomas Gleixner <tglx@linutronix.de>
CommitDate: Mon, 22 Jul 2019 10:54:27 +0200
x86/irq/64: Update stale comment
Commit e6401c130931 ("x86/irq/64: Split the IRQ stack into its own pages")
missed to update one piece of comment as it did to its peer in Xen, which
will confuse people who still need to read comment.
Signed-off-by: Cao jin <caoj.fnst@cn.fujitsu.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Link: https://lkml.kernel.org/r/20190719081635.26528-1-caoj.fnst@cn.fujitsu.com
---
arch/x86/kernel/head_64.S | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/x86/kernel/head_64.S b/arch/x86/kernel/head_64.S
index a6342c899be5..f3d3e9646a99 100644
--- a/arch/x86/kernel/head_64.S
+++ b/arch/x86/kernel/head_64.S
@@ -193,10 +193,10 @@ ENTRY(secondary_startup_64)
/* Set up %gs.
*
- * The base of %gs always points to the bottom of the irqstack
- * union. If the stack protector canary is enabled, it is
- * located at %gs:40. Note that, on SMP, the boot cpu uses
- * init data section till per cpu areas are set up.
+ * The base of %gs always points to fixed_percpu_data. If the
+ * stack protector canary is enabled, it is located at %gs:40.
+ * Note that, on SMP, the boot cpu uses init data section until
+ * the per cpu areas are set up.
*/
movl $MSR_GS_BASE,%ecx
movl initial_gs(%rip),%eax
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] x86/irq/64: fix the missing update on comment
2019-07-22 8:53 ` Thomas Gleixner
@ 2019-07-22 9:13 ` Cao jin
0 siblings, 0 replies; 4+ messages in thread
From: Cao jin @ 2019-07-22 9:13 UTC (permalink / raw)
To: Thomas Gleixner; +Cc: x86, linux-kernel, mingo, bp, hpa, luto
On 7/22/19 4:53 PM, Thomas Gleixner wrote:
> Cao,
>
> On Fri, 19 Jul 2019, Cao jin wrote:
>
>> Commit e6401c130931 ("x86/irq/64: Split the IRQ stack into its own pages")
>> missed to update one piece of comment as it did to its peer in Xen, which
>> will confuse people who still need to read comment.
>>
>> A bonus fix to identation in ZO's linker script: spaces -> tab.
>
> Please don't add 'bonus' changes. A patch which fixes a stale comment has
> nothing to do with a indentation change in an unrelated file.
>
Kept that in mind. Sorry for the inconvenience.
--
Sincerely,
Cao jin
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-07-22 9:12 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-19 8:16 [PATCH] x86/irq/64: fix the missing update on comment Cao jin
2019-07-22 8:53 ` Thomas Gleixner
2019-07-22 9:13 ` Cao jin
2019-07-22 8:58 ` [tip:x86/urgent] x86/irq/64: Update stale comment tip-bot for Cao jin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).