[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH] hw/arm/virt: Added preliminary GICv3 suppor
From: |
Eric Auger |
Subject: |
Re: [Qemu-devel] [RFC PATCH] hw/arm/virt: Added preliminary GICv3 support for kvm mode |
Date: |
Thu, 21 May 2015 16:24:27 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 |
On 05/21/2015 04:10 PM, Pavel Fedin wrote:
> Hello!
>
>> So to me it is sensible to instantiate GICV2 through legacy
>> KVM_CREATE_IRQCHIP API if both KVM_CREATE_DEVICE(test mode=true) failed.
>
> I disagree because at this point we already know which GIC version the user
> wants. This
> is because kvm_irqchip_create() is called after machine instance is created
> (and
> virt_instance_init() has been called). At this point we already know all the
> options. At
> this point i think the scenario should be:
> a) If we want GICv3 - test for KVM_CREATE_DEVICE(GICv3) and fail if we don't
> have one.
> b) If we want GICv2 - test for KVM_CREATE_DEVICE(GICv2). If it fails, try
> KVM_CREATE_IRQCHIP.
> IMHO there is little sense to fall back from v3 to v2 or vice versa because
> other
> important parameters (like number of CPUs) depend on it.
> Implementing this behavior costs only one more integer in MachineState
> structure. Is it
> too large ? If you want, i can post my patches as RFC, i think now they are
> more or less
> OK.
Hi Pavel,
yes sure please post your RFC and let's wait for other reviewer
feedback. I just shared my understanding ;-)
Best Regards
Eric
>
> Kind regards,
> Pavel Fedin
> Expert Engineer
> Samsung Electronics Research center Russia
>
>
Re: [Qemu-devel] [RFC PATCH] hw/arm/virt: Added preliminary GICv3 support for kvm mode, Eric Auger, 2015/05/19
Re: [Qemu-devel] [RFC PATCH] hw/arm/virt: Added preliminary GICv3 support for kvm mode, Eric Auger, 2015/05/18