opensbi.git/lib/sbi/sbi_fwft.c, branch master

lib: sbi: Fix hw a/d updating defaults

2026-05-11T13:46:20Z

The Svade dt-binding description states that Svadu should only be enabled at boot time when only Svadu is present in the DT. Ensure that's the case. Also, when only Svadu is supported, disable FWFT.PTE_AD_HW_UPDATING, as we need both to support toggling. Signed-off-by: Andrew Jones Reviewed-by: Radim Krčmář Link: https://lore.kernel.org/r/20260401220845.190680-1-andrew.jones@oss.qualcomm.com Signed-off-by: Anup Patel

lib: sbi: Flush TLBs upon FWFT ADUE change

2025-12-16T14:49:21Z

A clarification has been added to the RISC-V privileged specification regarding synchronization requirements when xenvcfg.ADUE changes. (Refer, the following commit in the RISC-V Privileged ISA spec https://github.com/riscv/riscv-isa-manual/commit/4e540263db8ae3a27d132a1752cc0fad222facd8) As-per these requirements, the SBI FWFT ADUE implementation must flush TLBs upon changes in ADUE state on a hart. Signed-off-by: Andrew Waterman Signed-off-by: Anup Patel Link: https://lore.kernel.org/r/20251127112121.334023-3-apatel@ventanamicro.com Signed-off-by: Anup Patel

lib: sbi: fwft: Use only the provided PMLEN value

2025-06-15T13:26:33Z

As of riscv-sbi-doc commit c7d3d1f7dcaa ("ext-fwft: use the provided value in fwft_set(POINTER_MASKING_PMLEN)"), the SBI implementation must use only the provided PMLEN value or else fail. It may not fall back to a larger PMLEN value. Signed-off-by: Samuel Holland Reviewed-by: Radim Krčmář Link: https://lore.kernel.org/r/20250522013503.2556053-1-samuel.holland@sifive.com Signed-off-by: Anup Patel

lib: sbi: fwft: Return SBI_ERR_DENIED_LOCKED when setting a locked feature

2025-02-19T16:43:21Z

Latest modifications to the spec mandates that a set on a lock feature returns SBI_ERR_DENIED_LOCKED. Signed-off-by: Clément Léger Reviewed-by: Anup Patel

sbi: fwft: clear the config lock when warm boot

2024-11-28T06:43:38Z

The hotplug cpu should clear the fwft config's SBI_FWFT_SET_FLAG_LOCK in the warm boot flow otherwise the cpu can't set the menvcfg.sse by SBI_EXT_FWFT_SET sbi call and cause the illegal instruction when accessing the CSR_SSP in kernel. Signed-off-by: Nick Hu Reviewed-by: Anup Patel

lib: sbi: implement firmware feature SBI_FWFT_DOUBLE_TRAP

2024-10-25T18:29:27Z

Add support for double trap firmware feature. Link: https://lists.riscv.org/g/tech-prs/message/985 [1] Signed-off-by: Clément Léger Reviewed-by: Samuel Holland

lib: sbi: fwft: factorize menvcfg read/write

2024-10-25T18:28:58Z

MENVCFG access will be used as well for double trap, landing pad and shadow stack fwft support. Factorize that in a common function. Signed-off-by: Clément Léger Reviewed-by: Samuel Holland

lib: sbi: fwft: implement landing pad and shadow stack fwft interface

2024-09-23T13:32:48Z

Supervisor software can enable control flow integrity features for itself using fwft feature `SBI_FWFT_LANDING_PAD` and `SBI_FWFT_SHADOW_STACK`. This patch implements the mechanism to enable both these fwft. Signed-off-by: Deepak Gupta Reviewed-by: Atish Patra Reviewed-by: Clément Léger

lib: sbi: fwft: add support for SBI_FWFT_POINTER_MASKING_PMLEN

2024-09-23T12:20:40Z

Add support for controlling the pointer masking mode on harts which support the Smnpm extension. This extension can only exist on harts where XLEN >= 64 bits. This implementation selects the mode with the smallest PMLEN that satisfies the caller's requested lower bound. Signed-off-by: Samuel Holland Reviewed-by: Anup Patel

lib: sbi: fwft: fix incorrect size passed to sbi_zalloc()

2024-08-02T03:15:07Z

The fwt_hart_state struct inciludes a flexible array member, so its allocation size will be that of the struct itself, plus that of each of the members in the array. When calculating this size, instead of taking the size of the struct, the size of a pointer to it was taken, which is incorrect. Luckily, this happenned to not produce memory corruption because the size of the non-flexible members of the struct is the same as the size of a pointer. Signed-off-by: Carlos López Reviewed-by: Anup Patel