[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