On Thu, 2020-02-13 at 14:28 +0100, Ján Tomko wrote: > On Fri, Feb 07, 2020 at 03:27:04PM +0100, Andrea Bolognani wrote: > > +++ b/src/qemu/qemu_domain.c > > @@ -5430,6 +5430,39 @@ qemuDomainDefValidateClockTimers(const virDomainDef *def, > > break; > > > > case VIR_DOMAIN_TIMER_NAME_ARMVTIMER: > > Missing check for present == 0. Good catch! Assuming Drew confirms the timer can't be disabled, then we should definitely error out for present='no'. Do you want me to respin, or can I just squash in the diff below? diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c index 29ec203413..143ddc508e 100644 --- a/src/qemu/qemu_domain.c +++ b/src/qemu/qemu_domain.c @@ -5441,6 +5441,12 @@ qemuDomainDefValidateClockTimers(const virDomainDef *def, def->os.machine); return -1; } + if (timer->present == 0) { + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, + _("The '%s' timer can't be disabled"), + virDomainTimerNameTypeToString(timer->name)); + return -1; + } if (!virQEMUCapsGet(qemuCaps, QEMU_CAPS_CPU_KVM_NO_ADJVTIME)) { virReportError(VIR_ERR_CONFIG_UNSUPPORTED, _("Configuring the '%s' timer is not supported " -- Andrea Bolognani / Red Hat / Virtualization