[libvirt] [PATCH 2/2] qemu: Refresh RTC adjustment on qemuProcessReconnect

Peter Krempa pkrempa at redhat.com
Tue May 3 06:14:10 UTC 2016


On Mon, May 02, 2016 at 18:35:35 +0200, Michal Privoznik wrote:
> On 02.05.2016 16:36, Peter Krempa wrote:
> > On Fri, Apr 29, 2016 at 18:43:10 +0200, Michal Privoznik wrote:
> >> https://bugzilla.redhat.com/show_bug.cgi?id=1139766

[...]

> > Any failure in qemuProcessReconnect results into libvirtd killing the
> > process. I don't think this failure is criticall enough to allow such
> > brutality.
> > 
> > Peter
> > 
> 
> Well, if guest has updated its RTC while we were not running, we would
> report spurious time in our XML. Since we are doing something similar
> for ejectable disks, i.e. CDROMs too, I presumed it is okay. But I don't
> mind changing this to something softer.

As long as it's guaranteed that any target qemu supported by libvirt 
does support the interrogation you are going to do it's probably fine.

The problem with your patch is that qom-get was added in qemu 1.1
whereas libvirt supports versions since 0.12

This would effectively reduce compatibility to 1.1 which we should not
do as a side effect of a unrelated change.

Peter
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20160503/7a4f24ee/attachment-0001.sig>


More information about the libvir-list mailing list