software/src/opensbi.git/lib/sbi/sbi_unpriv.c, branch master

lib: sbi: Add variable-length unprivilege access functions

2026-06-17T06:11:11+00:00

sbi_load/store_loop read/write variable-length buffer unprivileged. Both function use the widest aligned 8/4/2/1 byte load/stores in each loop to reduce the total number of iterations. Also switch the scalar/vector misaligned handlers to make use of such functions to simplify code. Miscellaneous: remove the unnecessary [taddr] in inline assembly Signed-off-by: Bo Gan Tested-by: Anirudh Srinivasan Reviewed-by: Anup Patel Link: https://lore.kernel.org/r/20260609060024.706-4-ganboing@gmail.com Signed-off-by: Anup Patel

lib: sbi: Remove redundant call to sbi_hart_expected_trap_addr()

2025-07-20T15:24:34+00:00

The variable "sbi_hart_expected_trap" has already been extern variable. Therefore, the program can directly refer to it instead of calling sbi_hart_expected_trap_addr(). Signed-off-by: Alvin Chang Reviewed-by: Anup Patel Link: https://lore.kernel.org/r/20250703151957.2545958-2-alvinga@andestech.com Signed-off-by: Anup Patel

lib: sbi: Remove unnecessary semicolon

2023-05-11T07:01:34+00:00

We have redundant semicolon at quite a few places so let's remove it. Signed-off-by: Xiang W Reviewed-by: Anup Patel

lib: sbi: Fix tval and tinst for sbi_get_insn()

2022-06-21T03:01:56+00:00

We should not change trap->tval to mepc because mtval already points to the faulting portion of the emulated instruction fetch, which is also what stval is expected to be. In addition, htinst is only allowed to be zero for instruction access faults or page faults, and is only allowed to be zero or a psuedoinstruction for instruction guest-page faults. Fix trap->tinst for these cases. Signed-off-by: dramforever Reviewed-by: Anup Patel

lib: Add error messages via conditional compilation for the future

2022-03-27T03:24:12+00:00

On 128-bit machines, sbi_load_xx/sbi_store_xx needs to be improved. Through this conditional compile, the corresponding implementation can be prompted to be added. Signed-off-by: Xiang W Reviewed-by: Anup Patel

lib: Optimize inline assembly for unprivilege access functions

2020-05-16T12:22:42+00:00

Currently, unprivileged access functions uses few additional instructions which are redundant. It also uses specific registers which are not necessary. Remove those additional instructions and let the compiler choose the registers. Signed-off-by: Atish Patra Reviewed-by: Anup Patel

lib: Rename unprivileged trap handler

2020-05-10T04:29:16+00:00

Unprivileged trap handler can be reused for any cases where the executing code expects a trap. Rename it to "expected" trap handler as it will be used in other cases in future. Signed-off-by: Atish Patra Tested-by: Jonathan Balkind Reviewed-by: Anup Patel

lib: Remove scratch parameter from unpriv load/store functions

2020-03-28T08:02:01+00:00

The scratch parameter of unpriv load/store functions is now redundant hence we remove it. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: Optimize unpriv load/store implementation

2020-03-28T08:01:53+00:00

This patch optimize unpriv load/store implementation by having dedicated unpriv trap handler (just like KVM RISC-V). As a result of this optimization: 1. We have reduced roughly 13+ instruction in all unpriv load/store functions. The reduced instruction also include two function calls. 2. Per-HART trap info pointer in scratch space is now redundant hence removed. 3. The sbi_trap_handler() is now much cleaner because we don't have to handle unpriv load/store traps. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

include: Move bits related defines and macros to sbi_bitops.h

2020-03-08T05:36:18+00:00

The right location for all bits related defines and macros is sbi_bitops.h hence this patch. With this patch, the sbi_bits.h is redundant so we remove it. Signed-off-by: Anup Patel Reviewed-by: Atish Patra