Re: [PATCH 8/8] arm64: Enable livepatch for ARM64

2025-01-31 Thread Prasanna Kumar T S M
On 28-01-2025 03:03, Weinan Liu wrote: Since SFrame is considered as reliable stacktrace, enable livepatch in arch/arm64/Kconfig Signed-off-by: Weinan Liu --- arch/arm64/Kconfig | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig index 100570a048c

Re: [PATCH 6/8] unwind: arm64: add reliable stacktrace support for arm64

2025-01-30 Thread Prasanna Kumar T S M
ntry_data { bool (*consume_entry)(void *cookie, u64 ip, u64 sp, u64 fp); void *cookie; Why not fold the previous patch and this into one? But the code looks good to me. Reviewed-by: Prasanna Kumar T S M .

Re: [PATCH 5/8] unwind: arm64: Add sframe unwinder on arm64

2025-01-30 Thread Prasanna Kumar T S M
NDER + .cfa = 0, +#endif }, }; Looks good to me. Reviewed-by: Prasanna Kumar T S M .

Re: [PATCH 4/8] unwind: Implement generic sframe unwinder library

2025-01-30 Thread Prasanna Kumar T S M
On 30-01-2025 15:52, Prasanna Kumar T S M wrote: On 28-01-2025 03:03, Weinan Liu wrote: This change introduces a kernel space unwinder using sframe table for architectures without ORC unwinder support. The implementation is adapted from Josh's userspace sframe unwinder proposal[1] acco

Re: [PATCH 4/8] unwind: Implement generic sframe unwinder library

2025-01-30 Thread Prasanna Kumar T S M
+ sftbl.sfhdr_p->fdes_off); + sftbl.fre_p = __start_sframe_header + SFRAME_HDR_SIZE(*sftbl.sfhdr_p) + + sftbl.sfhdr_p->fres_off; + sframe_init = true; +} Other than the minor suggestion, the code looks good to me. Reviewed-by: Prasanna Kumar T S M .

Re: [PATCH 7/8] arm64: Define TIF_PATCH_PENDING for livepatch

2025-01-30 Thread Prasanna Kumar T S M
klp_update_patch_state(current); + if (thread_flags & (_TIF_SIGPENDING | _TIF_NOTIFY_SIGNAL)) do_signal(regs); Looks good to me. Reviewed-by: Prasanna Kumar T S M .

Re: [PATCH 8/8] arm64: Enable livepatch for ARM64

2025-01-30 Thread Prasanna Kumar T S M
" Looks good to me. Reviewed-by: Prasanna Kumar T S M .

Re: [PATCH 3/8] unwind: add sframe v2 header

2025-01-30 Thread Prasanna Kumar T S M
& 0x1) << 7) | ((fre_info) & 0x7f)) + +#define SFRAME_FRE_CFA_BASE_REG_ID(data) ((data) & 0x1) +#define SFRAME_FRE_OFFSET_COUNT(data)(((data) >> 1) & 0xf) +#define SFRAME_FRE_OFFSET_SIZE(data) (((data) >> 5) & 0x3) +#define SFRAME_FRE_MANGLED_RA_P(data)(((data) >> 7) & 0x1) + +#endif /* _SFRAME_H */ Looks good to me. Reviewed-by: Prasanna Kumar T S M .

Re: [PATCH 1/8] unwind: build kernel with sframe info

2025-01-30 Thread Prasanna Kumar T S M
_index : AT(ADDR(.printk_index) - LOAD_OFFSET) { \ Looks good to me. Reviewed-by: Prasanna Kumar T S M .