opensbi.git/lib/sbi/sbi_misaligned_ldst.c, branch v1.4

lib: sbi_misaligned_ldst: Fix handling of C.SWSP and C.SDSP

2023-10-09T08:23:20Z

Unlike C.LWSP/C.LDSP, these encodings can be used with the zero register, so checking that the rs2 field is non-zero is unnecessary. Additionally, the previous check was incorrect since it was checking the immediate field of the instruction instead of the rs2 field. Signed-off-by: Amanieu d'Antras Reviewed-by: Anup Patel

lib: sbi_misaligned_ldst: Set GVA if not emulating

2022-09-13T11:13:07Z

If a particular misaligned load or store cannot be emulated at all, for the redirected trap, trap.gva is set to 0, but it should be the same as mstatus[h].GVA of the original trap. Fix this so that if the trap is destined for HS-mode, hstatus.GVA is then set correctly. Fixes: 1c4ce74f5128 ("lib: sbi: Set gva when creating sbi_trap_info") Signed-off-by: Vivian Wang Reviewed-by: Andrew Jones Reviewed-by: Anup Patel

lib: sbi: Set gva when creating sbi_trap_info

2022-08-22T03:19:49Z

In some cases the sbi_trap_info argument passed to sbi_trap_redirect is created from scratch by filling its fields. Since we previously added a gva field to struct sbi_trap_info, initialize gva in these cases also. Suggested-by: Andrew Jones Signed-off-by: Vivian Wang Reviewed-by: Andrew Jones Reviewed-by: Anup Patel

lib: sbi: Fixup tinst for exceptions in sbi_misaligned_*()

2022-06-21T03:01:14Z

If there is an exception while emulating a misaligned load/store, fixup uptrap.tinst before redirecting. Otherwise, HS-mode software may receive an htinst describing the lbu/sb instruction that faulted during emulation[1]. [1]: https://github.com/riscv-software-src/opensbi/issues/258 Signed-off-by: dramforever Reviewed-by: Anup Patel

lib: sbi: Implement firmware counters

2021-07-11T05:08:23Z

RISC-V SBI v0.3 specification defines a set of firmware events that can provide additional information about the current firmware context. All of the firmware event monitoring are enabled now. The firmware events must be defined as raw perf event with MSB set as specified in the specification. Reviewed-by: Anup Patel Signed-off-by: Atish Patra

lib: sbi: Fix sign-extension in sbi_misaligned_load_handler()

2020-12-24T11:05:28Z

The misaligned load emulation does not sign-extend values correctly due to missing sign typecast in value passed to the SET_RD() macro. A very easy way to reproduce this issue is to load 16-bit value 0xff1e from a byte aligned address using LH instruction on hardware lacking misaligned load/store. This patch fixes sbi_misaligned_load_handler() for above issue. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: sbi_misaligned_ldst: Determine transformed instruction length correctly

2020-08-04T03:18:41Z

If MTINST[0:0] bit is 1 then we have transformed instruction encoding in MTINST CSR. For transformed instructions, if the MTINST[1:1] bit is Zero then original trapped instruction was a 16bit instruction which was converted to 32bit instruction at time of taking trap. We should use MTINST[1:1] bit to determine correct instruction length of transformed instruction. This patch updates misaligned load/store emulation as-per above. Signed-off-by: Anup Patel Reviewed-by: Atish Patra

lib: sbi_misaligned_ldst: Remove mcause, scratch and hartid parameters

2020-03-28T08:02:14Z

We remove mcause, scratch and hartid parameters from various functions for misaligned load/store 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: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: Remove scratch parameter from unpriv load/store functions

2020-03-28T08:02:01Z

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