[PATCH 0/5] common: clean up usage of structure jmp_buf_data
Jerome Forissier
jerome.forissier at linaro.org
Tue Mar 4 10:10:52 CET 2025
Hi Heinrich,
On 3/2/25 15:21, Heinrich Schuchardt wrote:
> When implementing setjmp() we did not clearly separate the architecture
> specific part from the non-specific part. The lead to code directly
> accessing the architecture specific struct jmp_buf_data instead of using
> jmp_buf.
>
> With this series include setjmp.h is carved out from the architecture
> specific implementations asm/setjmp.h.
>
> All direct usages of asm/setjmp.h and struct jmp_buf_data are corrected.
>
> Thanks to Zi for starting this clean-up.
>
> Heinrich Schuchardt (4):
> sandbox: remove linux/types.h dependency in setjmp.h
> arm: include asm-generic/int-ll64.h in setjmp.h
> common: clean up setjmp.h
> arm: use type jmp_buf instead of struct jmp_buf_data
>
> Yao Zi (1):
> efi_loader: Clean up usage of structure jmp_buf_data
>
> arch/arm/cpu/armv7/exception_level.c | 8 +++---
> arch/arm/cpu/armv8/exception_level.c | 8 +++---
> arch/arm/include/asm/setjmp.h | 17 ++++--------
> arch/arm/mach-rockchip/bootrom.c | 2 +-
> arch/riscv/include/asm/setjmp.h | 15 +++--------
> arch/sandbox/cpu/cpu.c | 2 +-
> arch/sandbox/include/asm/setjmp.h | 18 +++----------
> arch/x86/include/asm/setjmp.h | 11 +++-----
> include/efi_loader.h | 4 +--
> include/interrupt.h | 2 +-
> include/setjmp.h | 40 ++++++++++++++++++++++++++++
> lib/efi_loader/efi_boottime.c | 9 +++----
> test/lib/longjmp.c | 2 +-
> 13 files changed, 73 insertions(+), 65 deletions(-)
> create mode 100644 include/setjmp.h
>
For the whole series:
Reviewed-by: Jerome Forissier <jerome.forissier at linaro.org>
That's a nice cleanup. I will rebase my uthreads series [1] onto this
for v3.
[1] https://lists.denx.de/pipermail/u-boot/2025-February/581626.html
Thanks,
--
Jerome
More information about the U-Boot
mailing list