[libvirt] [PATCH] Fix localtime handling for Xen-PV domains

[Philipp Hahn]

Ping?

On Monday 15 August 2011 19:52:20 Philipp Hahn wrote:
> At least Xen-3.4.3 translates the /vm/localtime SXPR value to
> /domain/platform/localtime and /domain/image/{linux,hvm}/localtime when
> the domain is defined.  When reading back that information libvirt only
> handles HVM domains, but not PV domains: This results in libvirtd always
> returning
> 	<clock offset="utc"/>
> while Xend used (localtime 1).
>
> For PV domains use /domain/image/linux/localtime.
>
> Signed-off-by: Philipp Hahn <hahn at univention.de>
> ---
> This is tracked at
> <https://forge.univention.org/bugzilla/show_bug.cgi?id=22321> ---
>  src/xenxs/xen_sxpr.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/src/xenxs/xen_sxpr.c b/src/xenxs/xen_sxpr.c
> index 1f5be5f..ff8326f 100644
> --- a/src/xenxs/xen_sxpr.c
> +++ b/src/xenxs/xen_sxpr.c
> @@ -1195,6 +1195,9 @@ xenParseSxpr(const struct sexpr *root,
>              def->clock.ntimers = 1;
>              def->clock.timers[0] = timer;
>          }
> +    } else { /* !hvm */
> +        if (sexpr_int(root, "domain/image/linux/localtime"))
> +            def->clock.offset = VIR_DOMAIN_CLOCK_OFFSET_LOCALTIME;
>      }
>
>      /* Current XenD allows localtime here, for PV and HVM */


-- 
Philipp Hahn           Open Source Software Engineer      hahn at univention.de
Univention GmbH        Linux for Your Business        fon: +49 421 22 232- 0
Mary-Somerville-Str.1  D-28359 Bremen                 fax: +49 421 22 232-99
                                                   http://www.univention.de/
----------------------------------------------------------------------------
Treffen Sie Univention auf der IT&Business vom 20. bis 22. September 2011
auf dem Gemeinschaftsstand der Open Source Business Alliance in Stuttgart in
Halle 3 Stand 3D27-7.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20110903/f6ad5e33/attachment-0001.sig>