mbox series

[v2,0/2] target/riscv/kvm: riscv-aia fixes

Message ID 20241028182037.290171-1-dbarboza@ventanamicro.com (mailing list archive)
Headers show
Series target/riscv/kvm: riscv-aia fixes | expand

Message

Daniel Henrique Barboza Oct. 28, 2024, 6:20 p.m. UTC
Hi,

In this second version we removed patches 3 and 4 from v1. The reasoning
behind is that the deprecation process is too harsh in comparison with
the user benefit from using bools instead of strings.

We'll expose the string property as bools when it's time to expose them
in QMP and users will keep using the existing str prop.

Patches based on alistair/riscv-to-apply.next.

Changes from v1:
- patches 3 and 4 from v1: dropped
- v1 link: https://lore.kernel.org/qemu-riscv/20240924124407.766495-1-dbarboza@ventanamicro.com/

Daniel Henrique Barboza (2):
  target/riscv/kvm: set 'aia_mode' to default in error path
  target/riscv/kvm: clarify how 'riscv-aia' default works

 target/riscv/kvm/kvm-cpu.c | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

Comments

Alistair Francis Oct. 30, 2024, 1:46 a.m. UTC | #1
On Tue, Oct 29, 2024 at 4:21 AM Daniel Henrique Barboza
<dbarboza@ventanamicro.com> wrote:
>
> Hi,
>
> In this second version we removed patches 3 and 4 from v1. The reasoning
> behind is that the deprecation process is too harsh in comparison with
> the user benefit from using bools instead of strings.
>
> We'll expose the string property as bools when it's time to expose them
> in QMP and users will keep using the existing str prop.
>
> Patches based on alistair/riscv-to-apply.next.
>
> Changes from v1:
> - patches 3 and 4 from v1: dropped
> - v1 link: https://lore.kernel.org/qemu-riscv/20240924124407.766495-1-dbarboza@ventanamicro.com/
>
> Daniel Henrique Barboza (2):
>   target/riscv/kvm: set 'aia_mode' to default in error path
>   target/riscv/kvm: clarify how 'riscv-aia' default works

Thanks!

Applied to riscv-to-apply.next

Alistair

>
>  target/riscv/kvm/kvm-cpu.c | 22 ++++++++++++----------
>  1 file changed, 12 insertions(+), 10 deletions(-)
>
> --
> 2.45.2
>
>