[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] ARM: GIC: error out early when creating VGICv2
From: |
Andrew Jones |
Subject: |
Re: [Qemu-devel] [PATCH] ARM: GIC: error out early when creating VGICv2 failed |
Date: |
Thu, 17 Jan 2019 10:25:57 +0100 |
User-agent: |
Mutt/1.6.0.1 (2016-04-01) |
On Thu, Jan 17, 2019 at 11:09:17AM +0800, Shannon Zhao wrote:
> Currently if creating a gicv2 VM on the platform without
> VGICv2 support, it failed in kvm_arm_pmu_set_irq while
> trying to set the interrupt number for PMU with below
> error message.
> KVM_SET_DEVICE_ATTR failed: Invalid argument
>
> This error log is a little confused. Also it should error
> out in kvm_arm_gic_realize early.
>
> Signed-off-by: Shannon Zhao <address@hidden>
> ---
> hw/intc/arm_gic_kvm.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/hw/intc/arm_gic_kvm.c b/hw/intc/arm_gic_kvm.c
> index a611e8e..b4f2133 100644
> --- a/hw/intc/arm_gic_kvm.c
> +++ b/hw/intc/arm_gic_kvm.c
> @@ -554,8 +554,9 @@ static void kvm_arm_gic_realize(DeviceState *dev, Error
> **errp)
> KVM_DEV_ARM_VGIC_CTRL_INIT, NULL, true,
> &error_abort);
> }
> - } else if (ret != -ENODEV && ret != -ENOTSUP) {
> - error_setg_errno(errp, -ret, "error creating in-kernel VGIC");
> + } else if (ret != -ENOTSUP) {
> + error_setg_errno(errp, -ret, "error creating in-kernel VGIC, maybe "
> + "vGICv2 not support on this platform");
vGICv2 is not supported on this platform
> return;
> }
>
> --
> 1.8.3.1
>
>
With the error message change
Reviewed-by: Andrew Jones <address@hidden>