On 3/20/25 11:25 AM, Andrea Bolognani wrote:
On Mon, Mar 03, 2025 at 01:46:53PM +1000, Alistair Francis wrote:
On Mon, Feb 24, 2025 at 10:32 PM Daniel Henrique Barboza
<dbarb...@ventanamicro.com> wrote:
We're missing scounteren and senvcfg CSRs, both already present in the
KVM UAPI.
Signed-off-by: Daniel Henrique Barboza <dbarb...@ventanamicro.com>
Reviewed-by: Andrew Jones <ajo...@ventanamicro.com>
Acked-by: Alistair Francis <alistair.fran...@wdc.com>
This patch seems to have broken KVM acceleration for me:
$ ./build/qemu-system-riscv64 -display none -M virt,accel=kvm -cpu host
qemu-system-riscv64: Failed to put registers after init: No such
file or directory
Reverting it makes QEMU work again.
My host is a SiFive HiFive Premier P550 board running Fedora 41. Note
that, since the upstreaming effort for this SoC has just recently
started, I'm using the 6.6-based vendor kernel.
Perhaps the KVM UAPI additions mentioned in the commit message are
more recent than that, and we need to make QEMU's use of them
conditional rather than unconditional?
Yes, we can't assume that CSRs will be always present.
I'll work on a fix. Thanks for reporting it!
Daniel
