opensbi.git/include/sbi/sbi_trap.h, branch v1.0

firware: optimize the exception exit code

2021-04-01T05:18:35Z

There are two copies of the same abnormal exit code, this patch deletes one Signed-off-by: Xiang W Reviewed-by: Anup Patel

include: headers: Replace ASSEMBLY with ASSEMBLER

2021-03-22T10:32:48Z

GCC has already a predefined macro __ASSEMBLER__ therefore, it can be used without the need to define a new flag with -D__ASSEMBLY__. This is useful when adding the library to projects having a build system such one can build without the need to make changes. THe build system does not use the Makefile in the sources tree. Signed-off-by: Marouene Boubakri Signed-off-by: Anup Patel

include: sbi: No need to pack struct sbi_trap_regs

2021-01-15T05:10:29Z

All members in struct sbi_trap_regs are of machine word size so no need to pack this structure. This also allows GCC to generate more optimized instruction sequence when accessing members of struct sbi_trap_regs. Reported-by: Paul Campbell Signed-off-by: Anup Patel Reviewed-by: Alistair Francis Reviewed-by: Atish Patra

lib: sbi: Introduce sbi_trap_exit() API

2021-01-07T04:23:17Z

We introduce sbi_trap_exit() API which can help non-firmware (i.e. generic or platform) code to force exit trap/interrupt handling and resume execution at context pointed by parameter "const struct sbi_trap_regs *regs". This new sbi_trap_exit() API will help Keystone Enclave project to resume execution of enclave from custom SBI call handler. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: sbi_trap: Simplify sbi_trap_handler() API

2020-03-28T08:02:23Z

This patch simplify sbi_trap_handler() API as follows: 1. Remove current hartid local variable because sbi_trap_handler() itself does not need it. 2. Remove scratch parameter because none of the functions directly called by sbi_trap_handler() require it. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: sbi_trap: Remove scratch parameter from sbi_trap_redirect()

2020-03-28T08:02:04Z

The scratch parameter of sbi_trap_redirect() is not used hence we remove it. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: Optimize unpriv load/store implementation

2020-03-28T08:01:53Z

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: Extend struct sbi_trap_info for mtval2 and mtinst

2019-12-16T09:18:19Z

We have two new trap CSRs namely mtval2 and mtinst when RISC-V hypervisor extension is available hence we extend struct sbi_trap_info accordingly. Signed-off-by: Anup Patel Reviewed-by: Alistair Francis

lib: Better naming of unpriv APIs for wider use

2019-11-21T08:27:33Z

The unpriv APIs can be useful to external firmware and out-of-tree platform support code. This patch adds "sbi_" prefix to unpriv load/store APIs and rename struct riscv_unpriv to struct sbi_trap_info everywhere. We also place struct sbi_trap_info in sbi/sbi_trap.h so that we can use it for sbi_trap_redirect() as well. Overall, this patch will make naming of unpriv APIs consistent with other OpenSBI APIs. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: Extend sbi_trap_redirect() for hypervisor extension

2019-09-30T09:59:37Z

When hypervisor extension is available, we can get traps from VS/VU modes. We should be able to force redirect some of these traps to HS-mode. In other words, we should be able forward traps from VS/VU mode to HS-mode using sbi_trap_redirect() hence this patch. Signed-off-by: Atish Patra Signed-off-by: Anup Patel Signed-off-by: Alistair Francis