diff options
| author | Jesse Taube <jesse@rivosinc.com> | 2025-06-04 06:52:25 -0700 |
|---|---|---|
| committer | Anup Patel <anup@brainfault.org> | 2025-06-16 10:14:20 +0530 |
| commit | 033e0e2353ddab76039e62b46c9f77e178bac689 (patch) | |
| tree | c70d52800485790ddf2c6b23eadc3b7b848315a1 /lib | |
| parent | 9f64f0619334eea34df66a1816b1be589870ac4e (diff) | |
| download | opensbi-033e0e2353ddab76039e62b46c9f77e178bac689.zip opensbi-033e0e2353ddab76039e62b46c9f77e178bac689.tar.gz opensbi-033e0e2353ddab76039e62b46c9f77e178bac689.tar.bz2 | |
lib: sbi: dbtr: Fix shared memory layout
The existing sbi_dbtr_shmem_entry has a size of 5 * XLEN with the final
entry being idx. This is in contrast to the SBI v3.0-rc7 Chapter 19.
Debug Triggers Extension [0] where idx and trig_state share the same
offset (0) in shared memory, with a total size of 4 * XLEN for all the
SBI calls.
Replace struct with union to match memory layout described in SBI.
[0] https://github.com/riscv-non-isa/riscv-sbi-doc/tree/v3.0-rc7/src/ext-debug-triggers.adoc
Fixes: 97f234f15c96 ("lib: sbi: Introduce the SBI debug triggers extension support")
Signed-off-by: Jesse Taube <jesse@rivosinc.com>
Reviewed-by: Charlie Jenkins <charlie@rivosinc.com>
Tested-by: Charlie Jenkins <charlie@rivosinc.com>
Reviewed-by: Himanshu Chauhan <hchauhan@ventanamicro.com>
Tested-by: Himanshu Chauhan <hchauhan@ventanamicro.com>
Link: https://lore.kernel.org/r/20250604135225.842241-1-jesse@rivosinc.com
Signed-off-by: Anup Patel <anup@brainfault.org>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/sbi/sbi_dbtr.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/sbi/sbi_dbtr.c b/lib/sbi/sbi_dbtr.c index f0e9128..cfb6b2a 100644 --- a/lib/sbi/sbi_dbtr.c +++ b/lib/sbi/sbi_dbtr.c @@ -506,7 +506,7 @@ int sbi_dbtr_read_trig(unsigned long smode, { struct sbi_dbtr_data_msg *xmit; struct sbi_dbtr_trigger *trig; - struct sbi_dbtr_shmem_entry *entry; + union sbi_dbtr_shmem_entry *entry; void *shmem_base = NULL; struct sbi_dbtr_hart_triggers_state *hs = NULL; @@ -541,7 +541,7 @@ int sbi_dbtr_install_trig(unsigned long smode, unsigned long trig_count, unsigned long *out) { void *shmem_base = NULL; - struct sbi_dbtr_shmem_entry *entry; + union sbi_dbtr_shmem_entry *entry; struct sbi_dbtr_data_msg *recv; struct sbi_dbtr_id_msg *xmit; unsigned long ctrl; @@ -659,7 +659,7 @@ int sbi_dbtr_update_trig(unsigned long smode, struct sbi_dbtr_data_msg *recv; unsigned long uidx = 0; struct sbi_dbtr_trigger *trig; - struct sbi_dbtr_shmem_entry *entry; + union sbi_dbtr_shmem_entry *entry; void *shmem_base = NULL; struct sbi_dbtr_hart_triggers_state *hs = NULL; |