starfive-tech/opensbi.git/lib/sbi/sbi_ecall.c, branch debug

lib: sbi: Replace args with trap registers in ecall handler

2021-01-07T04:23:14+00:00

We had added args pointer in ecall handler to ensure that ecall handler only implements functionality and does not deal with SBI calling convention. This also helped us to keep SBI calling convention related code in one place at sbi_ecall_handler(). The Keystone Enclavce project needs access to the trap regsiters in their ecall handler so that they can context switch enclaves in custom SBI calls. To help the Keystone Enclave project, we replace the args pointer in ecall handler parameter with a const pointer to trap registers. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: sbi: Implement System Reset (SRST) SBI extension

2020-12-01T11:41:11+00:00

The SBI SRST extension has been accepted and merged in the latest SBI v0.3-draft specification. (Refer, https://github.com/riscv/riscv-sbi-doc) It allows to S-mode software to request system shutdown, cold reboot, and warm reboot. This patch implements SBI SRST extension as a replacement of the legacy sbi_shutdown() call of SBI v0.1 specification. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: Don't return any invalid error from SBI ecall

2020-06-20T05:06:13+00:00

We should only return valid error codes from SBI ecalls as defined by the RISC-V SBI spec. To achieve this: 1. We use SBI_Exxxx defines for OpenSBI internal errors with error values starting from -1000 2. We use SBI_ERR_xxxx defines for errors defined by SBI spec 3. We map some of the SBI_Exxxx defines to SBI_ERR_xxxx defines which are semantically same 4. We throw a error print and force return error code to SBI_ERR_FAILED in sbi_ecall_handler() if we see an invalid error code being returned to S-mode Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: Allow overriding SBI implementation ID

2020-04-19T11:30:05+00:00

Ideally, the SBI implementation ID for OpenSBI should always be 0x1 (as mentioned in SBI v0.2 spec) but external firmware (such as EDK2) which use OpenSBI as library might want to override the SBI implementation ID with their custom implementation ID. Signed-off-by: Anup Patel Reviewed-by: Bin Meng Reviewed-by: Atish Patra

lib: sbi_ecall: Remove mcause, scratch and hartid parameters

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

We remove mcause, scratch and hartid parameters from various functions for ecall handling because we can always get current HART id and current scratch pointer using just one CSR access. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: sbi_trap: Remove scratch parameter from sbi_trap_redirect()

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

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

lib: Fix sbi_ecall_register_extension to prevent extension IDs overlap

2020-03-19T04:46:17+00:00

The original code does not prevent the following scenarios: > sbi_ecall_register_extension(ext1); /* extension id (70-80) */ > sbi_ecall_register_extension(ext2); /* extension id (50-100) */ Signed-off-by: Xiang Wang Reviewed-by: Anup Patel

lib: sbi: Fix coding style issues

2020-03-10T04:57:28+00:00

This fixes various coding style issues found in the SBI codes. No functional changes. Signed-off-by: Bin Meng Reviewed-by: Anup Patel

lib: Implement Hart State Management (HSM) SBI extension

2020-02-24T12:43:23+00:00

This patch adds support HSM extension. The specification is available at https://github.com/riscv/riscv-sbi-doc. It allows to implement hart hotplug and fixed ordered hart booting in supervisor. Signed-off-by: Atish Patra Reviewed-by: Anup Patel

lib: Initialize out value in SBI calls

2020-02-13T04:27:22+00:00

As per the SBI specification, the return value in sbiret is undefined if not explicitly described in the function. However, supervisor may check this value by mistake and get a garbage value. Initialize it to zero to avoid nasty supervisor bugs. Signed-off-by: Atish Patra Reviewed-by: Anup Patel