hexagon: work around compiler crash
authorNick Desaulniers <ndesaulniers@google.com>
Sat, 4 Jan 2020 21:00:02 +0000 (13:00 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 17 Jan 2020 18:45:55 +0000 (19:45 +0100)
[ Upstream commit 63e80314ab7cf4783526d2e44ee57a90514911c9 ]

Clang cannot translate the string "r30" into a valid register yet.

Link: https://github.com/ClangBuiltLinux/linux/issues/755
Link: http://lkml.kernel.org/r/20191028155722.23419-1-ndesaulniers@google.com
Signed-off-by: Nick Desaulniers <ndesaulniers@google.com>
Suggested-by: Sid Manning <sidneym@quicinc.com>
Reviewed-by: Brian Cain <bcain@codeaurora.org>
Cc: Allison Randal <allison@lohutok.net>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Richard Fontana <rfontana@redhat.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
arch/hexagon/kernel/stacktrace.c

index 41866a06adf7f394c784e2d6703b6aa1665f4ab8..ec4ef682923d970e499b1cd1317f4dc112c69fa3 100644 (file)
@@ -24,8 +24,6 @@
 #include <linux/thread_info.h>
 #include <linux/module.h>
 
-register unsigned long current_frame_pointer asm("r30");
-
 struct stackframe {
        unsigned long fp;
        unsigned long rets;
@@ -43,7 +41,7 @@ void save_stack_trace(struct stack_trace *trace)
 
        low = (unsigned long)task_stack_page(current);
        high = low + THREAD_SIZE;
-       fp = current_frame_pointer;
+       fp = (unsigned long)__builtin_frame_address(0);
 
        while (fp >= low && fp <= (high - sizeof(*frame))) {
                frame = (struct stackframe *)fp;