Re: [Qemu-devel] [PATCH v4] mc146818rtc: add rtc-reset-reinjection QMP command

2014-06-25 Thread Michael S. Tsirkin
On Wed, Jun 25, 2014 at 03:51:30PM -0400, Luiz Capitulino wrote: > On Wed, 25 Jun 2014 14:40:12 +0300 > "Michael S. Tsirkin" wrote: > > > It is necessary to reset RTC interrupt reinjection backlog if > > guest time is synchronized via a different mechanism, such as > > QGA's guest-set-time comman

Re: [Qemu-devel] [PATCH v4] mc146818rtc: add rtc-reset-reinjection QMP command

2014-06-25 Thread Luiz Capitulino
On Wed, 25 Jun 2014 14:40:12 +0300 "Michael S. Tsirkin" wrote: > It is necessary to reset RTC interrupt reinjection backlog if > guest time is synchronized via a different mechanism, such as > QGA's guest-set-time command. > > Failing to do so causes both corrections to be applied (summed), > re

Re: [Qemu-devel] [PATCH v4] mc146818rtc: add rtc-reset-reinjection QMP command

2014-06-25 Thread Marcelo Tosatti
On Wed, Jun 25, 2014 at 02:40:12PM +0300, Michael S. Tsirkin wrote: > It is necessary to reset RTC interrupt reinjection backlog if > guest time is synchronized via a different mechanism, such as > QGA's guest-set-time command. > > Failing to do so causes both corrections to be applied (summed), >

Re: [Qemu-devel] [PATCH v4] mc146818rtc: add rtc-reset-reinjection QMP command

2014-06-25 Thread Eric Blake
On 06/25/2014 05:40 AM, Michael S. Tsirkin wrote: > It is necessary to reset RTC interrupt reinjection backlog if > guest time is synchronized via a different mechanism, such as > QGA's guest-set-time command. > > Failing to do so causes both corrections to be applied (summed), > resulting in an i

[Qemu-devel] [PATCH v4] mc146818rtc: add rtc-reset-reinjection QMP command

2014-06-25 Thread Michael S. Tsirkin
It is necessary to reset RTC interrupt reinjection backlog if guest time is synchronized via a different mechanism, such as QGA's guest-set-time command. Failing to do so causes both corrections to be applied (summed), resulting in an incorrect guest time. Signed-off-by: Marcelo Tosatti Reviewed