On Thu, Feb 13, 2020 at 05:45:14PM +0100, Andrea Bolognani wrote:
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 "
Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx> Jano
Attachment:
signature.asc
Description: PGP signature
