Re: [PATCH 0/2] system/vl.c: parse all '-accel' opts

2024-07-01 Thread Daniel Henrique Barboza
On 7/1/24 1:31 PM, Paolo Bonzini wrote: On Mon, Jul 1, 2024 at 3:30 PM Daniel Henrique Barboza wrote: My initial intention was to fix a problem we're having with libvirt and RISC-V where we can't set 'riscv-aia' by appending '-accel kvm,riscv-aia=val' via in the domain XML. libvirt will add

Re: [PATCH 0/2] system/vl.c: parse all '-accel' opts

2024-07-01 Thread Paolo Bonzini
On Mon, Jul 1, 2024 at 3:30 PM Daniel Henrique Barboza wrote: > My initial intention was to fix a problem we're having with libvirt and > RISC-V where we can't set 'riscv-aia' by appending '-accel kvm,riscv-aia=val' > via in the domain XML. libvirt will add a leading > '-accel kvm' in the regular

[PATCH 0/2] system/vl.c: parse all '-accel' opts

2024-07-01 Thread Daniel Henrique Barboza
Hi, In the current handling of '-accel' only the first instance is parsed. All other instances (aside from a 'helper' command that triggers the help text and exits) is ignored. This also means that we can mix different accelerators in the same command line. In fact we can do whatever we want as