[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 1/2] Force timedrift=none on previous machine
From: |
Stefano Stabellini |
Subject: |
Re: [Qemu-devel] [PATCH v3 1/2] Force timedrift=none on previous machines |
Date: |
Thu, 22 Mar 2012 10:37:48 +0000 |
User-agent: |
Alpine 2.00 (DEB 1167 2008-08-23) |
On Wed, 21 Mar 2012, Paolo Bonzini wrote:
> Il 21/03/2012 17:06, Crístian Viana ha scritto:
> > The current value for the -rtc timedrift option is none. This patch
> > makes sure that the old machines configuration will work the same way
> > even after that option changes its default value.
> >
> > Signed-off-by: Crístian Viana <address@hidden>
> > ---
> > hw/pc_piix.c | 39 +++++++++++++++++++++++++++++++++++++++
> > 1 files changed, 39 insertions(+), 0 deletions(-)
> >
> > diff --git a/hw/pc_piix.c b/hw/pc_piix.c
> > index 3f99f9a..08255b5 100644
> > --- a/hw/pc_piix.c
> > +++ b/hw/pc_piix.c
> > @@ -386,6 +386,10 @@ static QEMUMachine pc_machine_v1_0 = {
> > .driver = "isa-fdc",
> > .property = "check_media_rate",
> > .value = "off",
> > + }, {
> > + .driver = "mc146818rtc",
> > + .property = "lost_tick_policy",
> > + .value = "none",
> > },
> > { /* end of list */ }
> > },
> > @@ -405,6 +409,10 @@ static QEMUMachine pc_machine_v0_15 = {
> > .driver = "isa-fdc",
> > .property = "check_media_rate",
> > .value = "off",
> > + }, {
> > + .driver = "mc146818rtc",
> > + .property = "lost_tick_policy",
> > + .value = "none",
> > },
> > { /* end of list */ }
> > },
> > @@ -449,6 +457,10 @@ static QEMUMachine pc_machine_v0_14 = {
> > .driver = "pc-sysfw",
> > .property = "rom_only",
> > .value = stringify(1),
> > + }, {
> > + .driver = "mc146818rtc",
> > + .property = "lost_tick_policy",
> > + .value = "none",
> > },
> > { /* end of list */ }
> > },
> > @@ -505,6 +517,10 @@ static QEMUMachine pc_machine_v0_13 = {
> > .driver = "pc-sysfw",
> > .property = "rom_only",
> > .value = stringify(1),
> > + }, {
> > + .driver = "mc146818rtc",
> > + .property = "lost_tick_policy",
> > + .value = "none",
> > },
> > { /* end of list */ }
> > },
> > @@ -565,6 +581,10 @@ static QEMUMachine pc_machine_v0_12 = {
> > .driver = "pc-sysfw",
> > .property = "rom_only",
> > .value = stringify(1),
> > + }, {
> > + .driver = "mc146818rtc",
> > + .property = "lost_tick_policy",
> > + .value = "none",
> > },
> > { /* end of list */ }
> > }
> > @@ -633,6 +653,10 @@ static QEMUMachine pc_machine_v0_11 = {
> > .driver = "pc-sysfw",
> > .property = "rom_only",
> > .value = stringify(1),
> > + }, {
> > + .driver = "mc146818rtc",
> > + .property = "lost_tick_policy",
> > + .value = "none",
> > },
> > { /* end of list */ }
> > }
> > @@ -713,6 +737,10 @@ static QEMUMachine pc_machine_v0_10 = {
> > .driver = "pc-sysfw",
> > .property = "rom_only",
> > .value = stringify(1),
> > + }, {
> > + .driver = "mc146818rtc",
> > + .property = "lost_tick_policy",
> > + .value = "none",
> > },
> > { /* end of list */ }
> > },
> > @@ -728,6 +756,10 @@ static QEMUMachine isapc_machine = {
> > .driver = "pc-sysfw",
> > .property = "rom_only",
> > .value = stringify(1),
> > + }, {
> > + .driver = "mc146818rtc",
> > + .property = "lost_tick_policy",
> > + .value = "none",
> > },
> > { /* end of list */ }
> > },
> > @@ -740,6 +772,13 @@ static QEMUMachine xenfv_machine = {
> > .init = pc_xen_hvm_init,
> > .max_cpus = HVM_MAX_VCPUS,
> > .default_machine_opts = "accel=xen",
> > + .compat_props = (GlobalProperty[]) {
> > + {
> > + .driver = "mc146818rtc",
> > + .property = "lost_tick_policy",
> > + .value = "none",
> > + },
> > + { /* end of list */ }
> > };
> > #endif
> >
>
> Stefano, what do you want for Xen? Anyhow,
I would like to keep the old policy for Xen, so the patch is fine by me.
Thanks for pinging me!