[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v11 5/5] hw/arm/virt: Add gic-version option to
From: |
Pavel Fedin |
Subject: |
Re: [Qemu-devel] [PATCH v11 5/5] hw/arm/virt: Add gic-version option to virt machine |
Date: |
Tue, 25 Aug 2015 16:53:15 +0300 |
Hello!
> In KVM case instead of assigning GICv2 as default, can we probe the KVM
> for GICv3 presence using KVM_CREATE_DEVICE_TEST and give priority to
> GICv3?
We could, but i think this would hurt usability. I suggest that when the user
chooses a
configuration, he/she expects to get something particular. And IMHO it's much
better to default to
GICv2 (to be backwards-compatible), and just report error in case if this is
not supported by KVM,
than to default to something, which is different on different machines and
different kernels.
Libvirt guys already blame "virt" machine for being too flexible...
Kind regards,
Pavel Fedin
Expert Engineer
Samsung Electronics Research center Russia
- [Qemu-devel] [PATCH v11 0/5] vGICv3 support, Pavel Fedin, 2015/08/19
- [Qemu-devel] [PATCH v11 3/5] arm_kvm: Do not assume particular GIC type in kvm_arch_irqchip_create(), Pavel Fedin, 2015/08/19
- [Qemu-devel] [PATCH v11 1/5] hw/intc: Implement GIC-500 base class, Pavel Fedin, 2015/08/19
- [Qemu-devel] [PATCH v11 2/5] intc/gic: Extract some reusable vGIC code, Pavel Fedin, 2015/08/19
- [Qemu-devel] [PATCH v11 5/5] hw/arm/virt: Add gic-version option to virt machine, Pavel Fedin, 2015/08/19
- [Qemu-devel] [PATCH v11 4/5] hw/intc: Initial implementation of vGICv3, Pavel Fedin, 2015/08/19
- Re: [Qemu-devel] [PATCH v11 0/5] vGICv3 support, Ashok Kumar, 2015/08/25