[kvm-unit-tests PATCH v6 5/5] riscv: sbi: Add SSE extension tests

Andrew Jones andrew.jones at linux.dev
Wed Jan 15 05:12:41 PST 2025


On Fri, Jan 10, 2025 at 12:12:44PM +0100, Clément Léger wrote:
> Add SBI SSE extension tests for the following features:
> - Test attributes errors (invalid values, RO, etc)
> - Registration errors
> - Simple events (register, enable, inject)
> - Events with different priorities
> - Global events dispatch on different harts
> - Local events on all harts
> - Hart mask/unmask events
> 
> Signed-off-by: Clément Léger <cleger at rivosinc.com>
> ---
>  riscv/Makefile          |   3 +-
>  lib/riscv/asm/csr.h     |   2 +
>  riscv/sse.h             |  41 ++
>  riscv/sse-asm.S         | 104 +++++
>  riscv/sbi-asm-offsets.c |   8 +
>  riscv/sbi-sse.c         | 936 ++++++++++++++++++++++++++++++++++++++++
>  riscv/sbi.c             |   3 +
>  riscv/sse.c             | 132 ++++++

I'm not reviewing this whole file since I have comments on v4 that I think
mostly apply. However, I noticed the new riscv/sse.[ch] files being added.
Those should have 'sbi' somewhere in their names or, if they just define
wrappers for the SSE functions (not test-specific stuff), then they can go
in lib/riscv/sbi.c. Remaining test-specific stuff should stay in the riscv
dir. Header stuff could go in riscv/sbi-tests.h or a new riscv/sbi-sse.h.
Everything else should be in riscv/sbi-sse.c

And riscv/sse-asm.S should be named riscv/sbi-sse-asm.S

Thanks,
drew



More information about the kvm-riscv mailing list