| Message ID | 1332345997-4772-2-git-send-email-vianac@linux.vnet.ibm.com |
|---|---|
| State | New |
| Headers | show |
Il 21/03/2012 17:06, Crístian Viana ha scritto: > Windows 2008+ is very sensitive to missed ticks. The RTC is used by default as > the time source. If time drift is not enabled, Windows is prone to > blue screening. > > Signed-off-by: Crístian Viana <vianac@linux.vnet.ibm.com> > --- > hw/mc146818rtc.c | 2 +- > vl.c | 10 +++++----- > 2 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/hw/mc146818rtc.c b/hw/mc146818rtc.c > index 2b59c36..2b31587 100644 > --- a/hw/mc146818rtc.c > +++ b/hw/mc146818rtc.c > @@ -726,7 +726,7 @@ ISADevice *rtc_init(ISABus *bus, int base_year, qemu_irq intercept_irq) > static Property mc146818rtc_properties[] = { > DEFINE_PROP_INT32("base_year", RTCState, base_year, 1980), > DEFINE_PROP_LOSTTICKPOLICY("lost_tick_policy", RTCState, > - lost_tick_policy, LOST_TICK_DISCARD), > + lost_tick_policy, LOST_TICK_SLEW), > DEFINE_PROP_END_OF_LIST(), > }; > > diff --git a/vl.c b/vl.c > index 112b0e0..11817e5 100644 > --- a/vl.c > +++ b/vl.c > @@ -539,18 +539,18 @@ static void configure_rtc(QemuOpts *opts) > value = qemu_opt_get(opts, "driftfix"); > if (value) { > if (!strcmp(value, "slew")) { > - static GlobalProperty slew_lost_ticks[] = { > + /* slew is default */ > + } else if (!strcmp(value, "none")) { > + static GlobalProperty discard_lost_ticks[] = { > { > .driver = "mc146818rtc", > .property = "lost_tick_policy", > - .value = "slew", > + .value = "discard", > }, > { /* end of list */ } > }; > > - qdev_prop_register_global_list(slew_lost_ticks); > - } else if (!strcmp(value, "none")) { > - /* discard is default */ > + qdev_prop_register_global_list(discard_lost_ticks); > } else { > fprintf(stderr, "qemu: invalid option value '%s'\n", value); > exit(1); Reviewed-by: Paolo Bonzini <pbonzini@redhat.com> Paolo
diff --git a/hw/mc146818rtc.c b/hw/mc146818rtc.c index 2b59c36..2b31587 100644 --- a/hw/mc146818rtc.c +++ b/hw/mc146818rtc.c @@ -726,7 +726,7 @@ ISADevice *rtc_init(ISABus *bus, int base_year, qemu_irq intercept_irq) static Property mc146818rtc_properties[] = { DEFINE_PROP_INT32("base_year", RTCState, base_year, 1980), DEFINE_PROP_LOSTTICKPOLICY("lost_tick_policy", RTCState, - lost_tick_policy, LOST_TICK_DISCARD), + lost_tick_policy, LOST_TICK_SLEW), DEFINE_PROP_END_OF_LIST(), }; diff --git a/vl.c b/vl.c index 112b0e0..11817e5 100644 --- a/vl.c +++ b/vl.c @@ -539,18 +539,18 @@ static void configure_rtc(QemuOpts *opts) value = qemu_opt_get(opts, "driftfix"); if (value) { if (!strcmp(value, "slew")) { - static GlobalProperty slew_lost_ticks[] = { + /* slew is default */ + } else if (!strcmp(value, "none")) { + static GlobalProperty discard_lost_ticks[] = { { .driver = "mc146818rtc", .property = "lost_tick_policy", - .value = "slew", + .value = "discard", }, { /* end of list */ } }; - qdev_prop_register_global_list(slew_lost_ticks); - } else if (!strcmp(value, "none")) { - /* discard is default */ + qdev_prop_register_global_list(discard_lost_ticks); } else { fprintf(stderr, "qemu: invalid option value '%s'\n", value); exit(1);
Windows 2008+ is very sensitive to missed ticks. The RTC is used by default as the time source. If time drift is not enabled, Windows is prone to blue screening. Signed-off-by: Crístian Viana <vianac@linux.vnet.ibm.com> --- hw/mc146818rtc.c | 2 +- vl.c | 10 +++++----- 2 files changed, 6 insertions(+), 6 deletions(-)