[libvirt] [PATCH 4/7] xen: fix timer bug found by updated test

Erik Skultety eskultet at redhat.com
Tue Dec 8 15:53:37 UTC 2015


On 04/12/15 20:30, Pavel Hrdina wrote:
> Only 'tsc' timer allows set mode and track is valid only for 'rtc' and
> 'platform' timers.
> 
> Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
> ---
>  src/xenconfig/xen_common.c | 2 ++
>  src/xenconfig/xen_sxpr.c   | 2 ++
>  2 files changed, 4 insertions(+)
> 
> diff --git a/src/xenconfig/xen_common.c b/src/xenconfig/xen_common.c
> index 0890c73..ed6978f 100644
> --- a/src/xenconfig/xen_common.c
> +++ b/src/xenconfig/xen_common.c
> @@ -553,6 +553,8 @@ xenParseCPUFeatures(virConfPtr conf, virDomainDefPtr def)
>              timer->name = VIR_DOMAIN_TIMER_NAME_HPET;
>              timer->present = val;
>              timer->tickpolicy = -1;
> +            timer->mode = -1;
> +            timer->track = -1;
>  
>              def->clock.ntimers = 1;
>              def->clock.timers[0] = timer;
> diff --git a/src/xenconfig/xen_sxpr.c b/src/xenconfig/xen_sxpr.c
> index f9a3187..09989ac 100644
> --- a/src/xenconfig/xen_sxpr.c
> +++ b/src/xenconfig/xen_sxpr.c
> @@ -1262,6 +1262,8 @@ xenParseSxpr(const struct sexpr *root,
>              timer->name = VIR_DOMAIN_TIMER_NAME_HPET;
>              timer->present = sexpr_int(root, "domain/image/hvm/hpet");
>              timer->tickpolicy = -1;
> +            timer->mode = -1;
> +            timer->track = -1;
>  
>              def->clock.ntimers = 1;
>              def->clock.timers[0] = timer;
> 

ACK after release.
Erik




More information about the libvir-list mailing list