diff options
| author | Samuel Holland <[email protected]> | 2025-02-20 09:53:01 -0800 |
|---|---|---|
| committer | Anup Patel <[email protected]> | 2025-03-24 17:00:59 +0530 |
| commit | 86c01a73ff9d34d21f7e7afca79315a411bfce15 (patch) | |
| tree | dfc48321b498c80f14b963ffe40cb912806f7398 | |
| parent | 98c0a3860a3c883a76a617de369a39508fde148a (diff) | |
lib: sbi: Avoid GOT indirection for global symbol references
OpenSBI is compiled with -fPIE, which generally implies dynamic linking.
This causes the compiler to generate GOT references for global symbols
in order to support runtime symbol interposition. However, OpenSBI does
not actually perform dynamic linking, so the GOT indirection just adds
unnecessary overhead.
The GOT references can be avoided by declaring global symbols with
hidden visibility, thus making them local to this dynamic object and
non-interposable. GCC/Clang's -fvisibility parameter is insufficient for
this purpose when referencing objects from other translation units;
either __attribute__((visibility(...)) or the pragma is required. Use
the pragma since it is easier to apply to every symbol. Additionally
clean up the one GOT reference from inline assembly.
With this change, a firmware linked with LLD does not contain either a
GOT or a PLT, and a firmware linked with BFD ld contains only a GOT with
a single (unreferenced, legacy) _GLOBAL_OFFSET_TABLE_ entry.
Signed-off-by: Samuel Holland <[email protected]>
Reviewed-by: Anup Patel <[email protected]>
| -rw-r--r-- | Makefile | 1 | ||||
| -rw-r--r-- | include/sbi/sbi_visibility.h | 16 | ||||
| -rw-r--r-- | lib/utils/serial/semihosting.c | 2 |
3 files changed, 18 insertions, 1 deletions
@@ -350,6 +350,7 @@ ifeq ($(BUILD_INFO),y) GENFLAGS += -DOPENSBI_BUILD_TIME_STAMP="\"$(OPENSBI_BUILD_TIME_STAMP)\"" GENFLAGS += -DOPENSBI_BUILD_COMPILER_VERSION="\"$(OPENSBI_BUILD_COMPILER_VERSION)\"" endif +GENFLAGS += -include $(include_dir)/sbi/sbi_visibility.h ifdef PLATFORM GENFLAGS += -include $(KCONFIG_AUTOHEADER) endif diff --git a/include/sbi/sbi_visibility.h b/include/sbi/sbi_visibility.h new file mode 100644 index 00000000..e9c401c5 --- /dev/null +++ b/include/sbi/sbi_visibility.h @@ -0,0 +1,16 @@ +/* + * SPDX-License-Identifier: BSD-2-Clause + * + * Copyright (c) 2025 SiFive + */ + +#ifndef __SBI_VISIBILITY_H__ +#define __SBI_VISIBILITY_H__ + +/* + * Declare all global objects with hidden visibility so access is PC-relative + * instead of going through the GOT. + */ +#pragma GCC visibility push(hidden) + +#endif diff --git a/lib/utils/serial/semihosting.c b/lib/utils/serial/semihosting.c index 19ebaa07..3a42ba60 100644 --- a/lib/utils/serial/semihosting.c +++ b/lib/utils/serial/semihosting.c @@ -67,7 +67,7 @@ bool semihosting_enabled(void) " mret\n" "_semihost_test_vector_next:\n" - " la %[tmp], _semihost_test_vector\n" + " lla %[tmp], _semihost_test_vector\n" " csrrw %[tmp], mtvec, %[tmp]\n" " .align 4\n" " slli zero, zero, 0x1f\n" |