Dan Kenigsberg has submitted this change and it was merged.

Change subject: vm: clock: always use rtc timer
......................................................................


vm: clock: always use rtc timer

We should always use the <timer name='rtc' tickpolicy='catchup'/> attribute.
It was removed only because of a misunderstandment of the libvirt docs.

<timer name='rtc' tickpolicy='catchup'/> is actually needed for Windows
guests as well. It provides the guest with time drift fix that is particularly
required on loaded servers.

Change-Id: If9c3996e95ac0848f9b0a6c7c77b1aeb6da860d7
Bug-Url: https://bugzilla.redhat.com/1215610
Signed-off-by: Francesco Romani <from...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/43195
Continuous-Integration: Jenkins CI
Reviewed-by: Michal Skrivanek <michal.skriva...@redhat.com>
Reviewed-by: Martin Polednik <mpoled...@redhat.com>
Reviewed-by: Dan Kenigsberg <dan...@redhat.com>
---
M tests/vmTests.py
M vdsm/virt/vmxml.py
2 files changed, 3 insertions(+), 2 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Verified
  Michal Skrivanek: Looks good to me, but someone else must approve
  Martin Polednik: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/43195
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If9c3996e95ac0848f9b0a6c7c77b1aeb6da860d7
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <mpoled...@redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com>
Gerrit-Reviewer: Michal Skrivanek <mskri...@redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeen...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to