mbox series

[for,4.2,0/3] Remove time reset notifications

Message ID 20190722150717.10564-1-dgilbert@redhat.com (mailing list archive)
Headers show
Series Remove time reset notifications | expand

Message

Dr. David Alan Gilbert July 22, 2019, 3:07 p.m. UTC
From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>

Timers have a mechanism for detecting host clock jumps; this relied
on noticing if the time had gone backwards or if it had gone forward
more than 60s since we last read it.  This had assumed that we regularly
read the time, which isn't true any more - we might not read the host
timer until the guest explicitly reads the guest RTC (e.g. hwclock).
This falsely triggers the reset mechanism.

The reset mechanism was only used by the mc146818 (i.e. PC) RTC
anyway; so lets remove it.

Dr. David Alan Gilbert (3):
  mc146818rtc: Remove reset notifiers
  timer: Remove reset notifiers
  timer: last, remove last bits of last

 hw/timer/mc146818rtc.c   | 19 -------------------
 include/qemu/timer.h     | 35 ----------------------------------
 replay/replay-snapshot.c |  3 +--
 util/qemu-timer.c        | 41 +---------------------------------------
 4 files changed, 2 insertions(+), 96 deletions(-)

Comments

Paolo Bonzini July 22, 2019, 4:46 p.m. UTC | #1
On 22/07/19 17:07, Dr. David Alan Gilbert (git) wrote:
> From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
> 
> Timers have a mechanism for detecting host clock jumps; this relied
> on noticing if the time had gone backwards or if it had gone forward
> more than 60s since we last read it.  This had assumed that we regularly
> read the time, which isn't true any more - we might not read the host
> timer until the guest explicitly reads the guest RTC (e.g. hwclock).
> This falsely triggers the reset mechanism.
> 
> The reset mechanism was only used by the mc146818 (i.e. PC) RTC
> anyway; so lets remove it.
> 
> Dr. David Alan Gilbert (3):
>   mc146818rtc: Remove reset notifiers
>   timer: Remove reset notifiers
>   timer: last, remove last bits of last
> 
>  hw/timer/mc146818rtc.c   | 19 -------------------
>  include/qemu/timer.h     | 35 ----------------------------------
>  replay/replay-snapshot.c |  3 +--
>  util/qemu-timer.c        | 41 +---------------------------------------
>  4 files changed, 2 insertions(+), 96 deletions(-)
> 

Queued, thanks.

Paolo