linux-metag.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] metag/kernel/stacktrace.c: Use current_stack_pointer instead of current_sp
@ 2015-09-05  9:33 gang.chen.5i5j-Re5JQEeQqe8AvxtiuMwx3w
  0 siblings, 0 replies; 2+ messages in thread
From: gang.chen.5i5j-Re5JQEeQqe8AvxtiuMwx3w @ 2015-09-05  9:33 UTC (permalink / raw)
  To: james.hogan-1AXoQHu6uovQT0dZR+AlfA
  Cc: linux-metag-u79uwXL29TY76Z2rM5mHXA,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA, gchen_5i5j-0GSiYjBpcig,
	Chen Gang

From: Chen Gang <gang.chen.5i5j-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>

Since we have defined current_stack_pointer, current_sp is redudancy, so
remove it.

Signed-off-by: Chen Gang <gang.chen.5i5j-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
 arch/metag/kernel/stacktrace.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/arch/metag/kernel/stacktrace.c b/arch/metag/kernel/stacktrace.c
index 5510361..5db1176 100644
--- a/arch/metag/kernel/stacktrace.c
+++ b/arch/metag/kernel/stacktrace.c
@@ -165,11 +165,9 @@ void save_stack_trace_tsk(struct task_struct *tsk, struct stack_trace *trace)
 		frame.pc = thread_saved_pc(tsk);
 #endif
 	} else {
-		register unsigned long current_sp asm ("A0StP");
-
 		data.no_sched_functions = 0;
 		frame.fp = (unsigned long)__builtin_frame_address(0);
-		frame.sp = current_sp;
+		frame.sp = current_stack_pointer;
 		frame.lr = (unsigned long)__builtin_return_address(0);
 		frame.pc = (unsigned long)save_stack_trace_tsk;
 	}
-- 
1.9.3

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [PATCH] metag/kernel/stacktrace.c: Use current_stack_pointer instead  of current_sp
@ 2015-09-05 13:58 Chen Gang
  0 siblings, 0 replies; 2+ messages in thread
From: Chen Gang @ 2015-09-05 13:58 UTC (permalink / raw)
  To: james.hogan@imgtec.com, linux-metag@vger.kernel.org,
	kernel mailing list

[-- Attachment #1: Type: text/plain, Size: 1189 bytes --]


From 89b00006f4ac64f5204752fd98998df9db058b6b Mon Sep 17 00:00:00 2001
From: Chen Gang <gang.chen.5i5j@gmail.com>
Date: Sat, 5 Sep 2015 17:29:23 +0800
Subject: [PATCH] metag/kernel/stacktrace.c: Use current_stack_pointer instead
 of current_sp

Since we have defined current_stack_pointer, current_sp is redudancy, so
remove it.

Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
---
 arch/metag/kernel/stacktrace.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/arch/metag/kernel/stacktrace.c b/arch/metag/kernel/stacktrace.c
index 5510361..5db1176 100644
--- a/arch/metag/kernel/stacktrace.c
+++ b/arch/metag/kernel/stacktrace.c
@@ -165,11 +165,9 @@ void save_stack_trace_tsk(struct task_struct *tsk, struct stack_trace *trace)
 		frame.pc = thread_saved_pc(tsk);
 #endif
 	} else {
-		register unsigned long current_sp asm ("A0StP");
-
 		data.no_sched_functions = 0;
 		frame.fp = (unsigned long)__builtin_frame_address(0);
-		frame.sp = current_sp;
+		frame.sp = current_stack_pointer;
 		frame.lr = (unsigned long)__builtin_return_address(0);
 		frame.pc = (unsigned long)save_stack_trace_tsk;
 	}
-- 
1.9.3

 		 	   		  

[-- Attachment #2: 0001-metag-kernel-stacktrace.c-Use-current_stack_pointer-.patch --]
[-- Type: application/octet-stream, Size: 1141 bytes --]

From 89b00006f4ac64f5204752fd98998df9db058b6b Mon Sep 17 00:00:00 2001
From: Chen Gang <gang.chen.5i5j@gmail.com>
Date: Sat, 5 Sep 2015 17:29:23 +0800
Subject: [PATCH] metag/kernel/stacktrace.c: Use current_stack_pointer instead
 of current_sp

Since we have defined current_stack_pointer, current_sp is redudancy, so
remove it.

Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
---
 arch/metag/kernel/stacktrace.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/arch/metag/kernel/stacktrace.c b/arch/metag/kernel/stacktrace.c
index 5510361..5db1176 100644
--- a/arch/metag/kernel/stacktrace.c
+++ b/arch/metag/kernel/stacktrace.c
@@ -165,11 +165,9 @@ void save_stack_trace_tsk(struct task_struct *tsk, struct stack_trace *trace)
 		frame.pc = thread_saved_pc(tsk);
 #endif
 	} else {
-		register unsigned long current_sp asm ("A0StP");
-
 		data.no_sched_functions = 0;
 		frame.fp = (unsigned long)__builtin_frame_address(0);
-		frame.sp = current_sp;
+		frame.sp = current_stack_pointer;
 		frame.lr = (unsigned long)__builtin_return_address(0);
 		frame.pc = (unsigned long)save_stack_trace_tsk;
 	}
-- 
1.9.3


^ permalink raw reply related	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-09-05 13:58 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-09-05  9:33 [PATCH] metag/kernel/stacktrace.c: Use current_stack_pointer instead of current_sp gang.chen.5i5j-Re5JQEeQqe8AvxtiuMwx3w
  -- strict thread matches above, loose matches on Subject: below --
2015-09-05 13:58 Chen Gang

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).