csky: coding convention: Use task_stack_page

guoren83 · guoren83 · commit 0f231dcfc664 · 2019-02-13T09:48:14.000+08:00
Use task_stack_page instead of p-&gt;stack to get stack. Follow the coding
convention style. Also for init_stack, the same with other archs.

Signed-off-by: Guo Ren &lt;ren_guo@c-sky.com&gt;
diff --git a/arch/csky/include/asm/processor.h b/arch/csky/include/asm/processor.h
@@ -49,7 +49,7 @@ struct thread_struct {
 };
 
 #define INIT_THREAD  { \
-	.ksp = (unsigned long) init_thread_union.stack + THREAD_SIZE, \
+	.ksp = sizeof(init_stack) + (unsigned long) &init_stack, \
 	.sr = DEFAULT_PSR_VALUE, \
 }
 
@@ -95,7 +95,7 @@ unsigned long get_wchan(struct task_struct *p);
 #define KSTK_ESP(tsk)		(task_pt_regs(tsk)->usp)
 
 #define task_pt_regs(p) \
-	((struct pt_regs *)(THREAD_SIZE + p->stack) - 1)
+	((struct pt_regs *)(THREAD_SIZE + task_stack_page(p)) - 1)
 
 #define cpu_relax() barrier()
 
diff --git a/arch/csky/kernel/smp.c b/arch/csky/kernel/smp.c
@@ -160,7 +160,8 @@ int __cpu_up(unsigned int cpu, struct task_struct *tidle)
 {
 	unsigned long mask = 1 << cpu;
 
-	secondary_stack = (unsigned int)tidle->stack + THREAD_SIZE - 8;
+	secondary_stack =
+		(unsigned int) task_stack_page(tidle) + THREAD_SIZE - 8;
 	secondary_hint = mfcr("cr31");
 	secondary_ccr  = mfcr("cr18");
 

Original file line number	Diff line number	Diff line change
`@@ -160,7 +160,8 @@ int __cpu_up(unsigned int cpu, struct task_struct *tidle)`
`160`	`160`	`{`
`161`	`161`	`unsigned long mask = 1 << cpu;`
`162`	`162`
`163`		`- secondary_stack = (unsigned int)tidle->stack + THREAD_SIZE - 8;`
	`163`	`+ secondary_stack =`
	`164`	`+ (unsigned int) task_stack_page(tidle) + THREAD_SIZE - 8;`
`164`	`165`	`secondary_hint = mfcr("cr31");`
`165`	`166`	`secondary_ccr = mfcr("cr18");`
`166`	`167`