hexagon: work around compiler crash
authorNick Desaulniers <ndesaulniers@google.com>
Sat, 4 Jan 2020 21:00:02 +0000 (13:00 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Sat, 4 Jan 2020 21:55:09 +0000 (13:55 -0800)
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>
arch/hexagon/kernel/stacktrace.c

index 35f29423fda808f6edd8202da932ce89ae728eae..5ed02f699479ad68c516444dceae7223452c1e0c 100644 (file)
@@ -11,8 +11,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;
@@ -30,7 +28,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;