opensbi.git/lib, branch v0.8

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

2020-06-20T05:06:13Z

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: sbi: Fix 32/64 bits variable compatibility

2020-06-20T03:17:11Z

On RV64,"unsigned long" is 64bit and "unsigned int" is 32bit. So in function "pmp_get" and "pmp_set", if "pmpcfg_shift >= 32", "0xff << pmpcfg_shift" will go beyond "unsigned int" width. This patch tries to fix this issue. In function 'pmp_get': cfgmask = (0xff << pmpcfg_shift); --> cfgmask = (0xffUL << pmpcfg_shift); In function 'pmp_set': cfgmask = ~(0xff << pmpcfg_shift); --> cfgmask = ~(0xffUL << pmpcfg_shift); Signed-off-by: Liush Reviewed-by: Anup Patel

platform: Add support for Shakti C-class SoC from IIT-M

2020-06-19T03:51:46Z

C-Class is a member of the SHAKTI family of processors from Indian Institute of Technology - Madras(IIT-M). It is an extremely configurable and commercial-grade 5-stage in-order core supporting the standard RV64GCSUN ISA extensions. https://gitlab.com/shaktiproject/cores/c-class/blob/master/README.md We add OpenSBI support for Shakti C-class SoC. Signed-off-by: Vijai Kumar K Reviewed-by: Anup Patel

lib: Fix the SBI_HART_HAS_MCOUNTEREN feature check

2020-06-18T03:28:13Z

Signed-off-by: Alistair Francis Reviewed-by: Atish Patra Reviewed-by: Anup Patel

lib: Don't print delegation CSRs if there is no S-Mode

2020-06-18T03:26:54Z

If the platform doesn't support S-Mode don't print the delegation registers. Signed-off-by: Alistair Francis Reviewed-by: Atish Patra Reviewed-by: Anup Patel

platform/lib: Allow the OS to map the regions that are protected by PMP

2020-06-15T03:55:44Z

This is achieved by removing the 'no-map' property from the 'reserved-memory' node when PMP is present, otherwise we keep it as it offers a small protection if the OS does not map this region at all. A new callback in platform_override is introduced and allows to fixup the device-tree. It is used here to override this new default behaviour on SiFive Fu540 platforms that has an erratum that prevents S-mode software to access a PMP protected region using 1GB page table mapping. If PMP is present, telling the OS not to map the reserved regions does not add much protection since it only avoids access to regions that are already protected by PMP. But by not allowing the OS to map those regions, it creates holes in the OS system memory map and prevents the use of hugepages which would generate, among other benefits, less TLB miss. Signed-off-by: Alexandre Ghiti Reviewed-by: Atish Patra Reviewed-by: Anup Patel

lib: Fix __sbi_hfence_gvma_vmid_gpa() and __sbi_hfence_vvma_asid_va()

2020-06-15T03:54:27Z

The arguments/parameters of __sbi_hfence_gvma_vmid_gpa() and __sbi_hfence_vvma_asid_va() functions are swapped so we fix it. Currently, we did not face any issues because QEMU does a full TLB flush for all HFENCE instructions. We also improve documentation of HFENCE.GVMA and HFENCE.VVMA instruction encoding. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: Add RISC-V hypervisor v0.6.1 support

2020-06-08T10:11:23Z

To support RISC-V hypervisor v0.6.1, we: 1. Don't need to explicitly forward WFI traps from VS/VU-mode 2. Have to delegate virtual instruction trap to HS-mode 3. Have to update trap redirection for changes in HSTATUS CSR Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: utils/timer: Initialize all matching timer DT nodes

2020-05-23T05:06:53Z

We can have multiple matching DT nodes of the same FDT timer driver so in this case we should call cold_init() for all matching DT nodes instead of just first matching DT node. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: utils/ipi: Initialize all matching ipi DT nodes

2020-05-23T05:06:49Z

We can have multiple matching DT nodes of the same FDT ipi driver so in this case we should call cold_init() for all matching DT nodes instead of just first matching DT node. Signed-off-by: Anup Patel Reviewed-by: Atish Patra