trace: clang compatible handling of gd register
authorHeinrich Schuchardt <xypron.glpk@gmx.de>
Wed, 27 May 2020 18:04:22 +0000 (20:04 +0200)
committerTom Rini <trini@konsulko.com>
Tue, 7 Jul 2020 22:23:48 +0000 (18:23 -0400)
On ARM systems gd is stored in register r9 or x18. When compiling with
clang gd is defined as a macro calling function gd_ptr(). So we can not
make assignments to gd.

Use function set_gd() for setting the register on ARM.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Reviewed-by: Simon Glass <sjg@chromium.org>
lib/trace.c

index ea8c8e0..831283c 100644 (file)
@@ -57,12 +57,12 @@ static inline uintptr_t __attribute__((no_instrument_function))
        return offset / FUNC_SITE_SIZE;
 }
 
-#ifdef CONFIG_EFI_LOADER
+#if defined(CONFIG_EFI_LOADER) && defined(CONFIG_ARM)
 
 /**
  * trace_gd - the value of the gd register
  */
-static volatile void *trace_gd;
+static volatile gd_t *trace_gd;
 
 /**
  * trace_save_gd() - save the value of the gd register
@@ -82,10 +82,10 @@ static void __attribute__((no_instrument_function)) trace_save_gd(void)
  */
 static void __attribute__((no_instrument_function)) trace_swap_gd(void)
 {
-       volatile void *temp_gd = trace_gd;
+       volatile gd_t *temp_gd = trace_gd;
 
        trace_gd = gd;
-       gd = temp_gd;
+       set_gd(temp_gd);
 }
 
 #else