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 <[email protected]> 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
