[Qemu-devel] [PATCH v2] rtc: Remove TARGET_I386 from qemu-config.c, enables driftfix

[Qemu-devel] [PATCH v2] rtc: Remove TARGET_I386 from qemu-config.c, enables driftfix

[Amit Shah]

qemu-config.c doesn't contain any target-specific code, and the
TARGET_I386 conditional code didn't get compiled as a result. Removing
this enables the driftfix parameter for rtc.

Signed-off-by: Amit Shah <amit.shah@redhat.com>
---
 qemu-config.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/qemu-config.c b/qemu-config.c
index 95abe61..730ffd9 100644
--- a/qemu-config.c
+++ b/qemu-config.c
@@ -247,11 +247,9 @@ QemuOptsList qemu_rtc_opts = {
         },{
             .name = "clock",
             .type = QEMU_OPT_STRING,
-#ifdef TARGET_I386
         },{
             .name = "driftfix",
             .type = QEMU_OPT_STRING,
-#endif
         },
         { /* end if list */ }
     },
-- 
1.7.0.1

[Qemu-devel] Re: [PATCH v2] rtc: Remove TARGET_I386 from qemu-config.c, enables driftfix

[Juan Quintela]

Amit Shah <amit.shah@redhat.com> wrote:
> qemu-config.c doesn't contain any target-specific code, and the
> TARGET_I386 conditional code didn't get compiled as a result. Removing
> this enables the driftfix parameter for rtc.
>
> Signed-off-by: Amit Shah <amit.shah@redhat.com>

Acked-by: Juan Quintela <quintela@redhat.com>

Agreed.  /me had forgot that vl.c is not target dependent anymore :(

Later, Juan.

[Qemu-devel] Re: [PATCH v2] rtc: Remove TARGET_I386 from qemu-config.c, enables driftfix

[Paolo Bonzini]

On 06/23/2010 04:44 PM, Amit Shah wrote:
> qemu-config.c doesn't contain any target-specific code, and the
> TARGET_I386 conditional code didn't get compiled as a result. Removing
> this enables the driftfix parameter for rtc.
>
> Signed-off-by: Amit Shah<amit.shah@redhat.com>
> ---
>   qemu-config.c |    2 --
>   1 files changed, 0 insertions(+), 2 deletions(-)
>
> diff --git a/qemu-config.c b/qemu-config.c
> index 95abe61..730ffd9 100644
> --- a/qemu-config.c
> +++ b/qemu-config.c
> @@ -247,11 +247,9 @@ QemuOptsList qemu_rtc_opts = {
>           },{
>               .name = "clock",
>               .type = QEMU_OPT_STRING,
> -#ifdef TARGET_I386
>           },{
>               .name = "driftfix",
>               .type = QEMU_OPT_STRING,
> -#endif
>           },
>           { /* end if list */ }
>       },

Acked-by: Paolo Bonzini <pbonzini@redhat.com>

Paolo

Re: [Qemu-devel] [PATCH v2] rtc: Remove TARGET_I386 from qemu-config.c, enables driftfix

[Jes Sorensen]

On 06/23/10 16:44, Amit Shah wrote:
> qemu-config.c doesn't contain any target-specific code, and the
> TARGET_I386 conditional code didn't get compiled as a result. Removing
> this enables the driftfix parameter for rtc.
> 
> Signed-off-by: Amit Shah <amit.shah@redhat.com>
> ---
>  qemu-config.c |    2 --
>  1 files changed, 0 insertions(+), 2 deletions(-)

I really don't like exposing all these options in targets where they are
not functional. It would be really nice if we could come up with a
better way to handle this without just exposing them globally.

Cheers,
Jes

Re: [Qemu-devel] [PATCH v2] rtc: Remove TARGET_I386 from qemu-config.c, enables driftfix

[Amit Shah]

On (Wed) Jun 23 2010 [20:14:04], Amit Shah wrote:
> qemu-config.c doesn't contain any target-specific code, and the
> TARGET_I386 conditional code didn't get compiled as a result. Removing
> this enables the driftfix parameter for rtc.
> 
> Signed-off-by: Amit Shah <amit.shah@redhat.com>

Ping

		Amit

[Qemu-devel] [PING^2] [PATCH v2] rtc: Remove TARGET_I386 from qemu-config.c, enables driftfix

[Paolo Bonzini]

On 07/14/2010 03:32 PM, Amit Shah wrote:
> On (Wed) Jun 23 2010 [20:14:04], Amit Shah wrote:
>> qemu-config.c doesn't contain any target-specific code, and the
>> TARGET_I386 conditional code didn't get compiled as a result. Removing
>> this enables the driftfix parameter for rtc.
>>
>> Signed-off-by: Amit Shah<amit.shah@redhat.com>
>
> Ping

Ping^2

Paolo

Re: [Qemu-devel] [PATCH v2] rtc: Remove TARGET_I386 from qemu-config.c, enables driftfix

[Daniel P. Berrange]

On Wed, Jun 23, 2010 at 08:14:04PM +0530, Amit Shah wrote:
> qemu-config.c doesn't contain any target-specific code, and the
> TARGET_I386 conditional code didn't get compiled as a result. Removing
> this enables the driftfix parameter for rtc.
> 
> Signed-off-by: Amit Shah <amit.shah@redhat.com>
> ---
>  qemu-config.c |    2 --
>  1 files changed, 0 insertions(+), 2 deletions(-)
> 
> diff --git a/qemu-config.c b/qemu-config.c
> index 95abe61..730ffd9 100644
> --- a/qemu-config.c
> +++ b/qemu-config.c
> @@ -247,11 +247,9 @@ QemuOptsList qemu_rtc_opts = {
>          },{
>              .name = "clock",
>              .type = QEMU_OPT_STRING,
> -#ifdef TARGET_I386
>          },{
>              .name = "driftfix",
>              .type = QEMU_OPT_STRING,
> -#endif
>          },
>          { /* end if list */ }
>      },


Is there any reason this patch hasn't been applied to GIT yet ? I'm told 
that using this option is critical to making certain guests work reliably
so we want to use it from libvirt/virt-manager for the OS in question.

Regards,
Daniel
-- 
|: Red Hat, Engineering, London    -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org -o- http://virt-manager.org -o- http://deltacloud.org :|
|: http://autobuild.org        -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|

Re: [Qemu-devel] [PATCH v2] rtc: Remove TARGET_I386 from qemu-config.c, enables driftfix

[Amit Shah]

On (Fri) Aug 06 2010 [12:12:45], Daniel P. Berrange wrote:
> On Wed, Jun 23, 2010 at 08:14:04PM +0530, Amit Shah wrote:
> > qemu-config.c doesn't contain any target-specific code, and the
> > TARGET_I386 conditional code didn't get compiled as a result. Removing
> > this enables the driftfix parameter for rtc.
> > 
> > Signed-off-by: Amit Shah <amit.shah@redhat.com>
> > ---
> >  qemu-config.c |    2 --
> >  1 files changed, 0 insertions(+), 2 deletions(-)
> > 
> > diff --git a/qemu-config.c b/qemu-config.c
> > index 95abe61..730ffd9 100644
> > --- a/qemu-config.c
> > +++ b/qemu-config.c
> > @@ -247,11 +247,9 @@ QemuOptsList qemu_rtc_opts = {
> >          },{
> >              .name = "clock",
> >              .type = QEMU_OPT_STRING,
> > -#ifdef TARGET_I386
> >          },{
> >              .name = "driftfix",
> >              .type = QEMU_OPT_STRING,
> > -#endif
> >          },
> >          { /* end if list */ }
> >      },
> 
> 
> Is there any reason this patch hasn't been applied to GIT yet ? I'm told 
> that using this option is critical to making certain guests work reliably
> so we want to use it from libvirt/virt-manager for the OS in question.

Multiple pings have gone out already; I hope it's in someone's queue to
be applied.

		Amit

Re: [Qemu-devel] [PATCH v2] rtc: Remove TARGET_I386 from qemu-config.c, enables driftfix

[Anthony Liguori]

On 08/06/2010 03:40 PM, Amit Shah wrote:
> On (Fri) Aug 06 2010 [12:12:45], Daniel P. Berrange wrote:
>    
>> On Wed, Jun 23, 2010 at 08:14:04PM +0530, Amit Shah wrote:
>>      
>>> qemu-config.c doesn't contain any target-specific code, and the
>>> TARGET_I386 conditional code didn't get compiled as a result. Removing
>>> this enables the driftfix parameter for rtc.
>>>
>>> Signed-off-by: Amit Shah<amit.shah@redhat.com>
>>> ---
>>>   qemu-config.c |    2 --
>>>   1 files changed, 0 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/qemu-config.c b/qemu-config.c
>>> index 95abe61..730ffd9 100644
>>> --- a/qemu-config.c
>>> +++ b/qemu-config.c
>>> @@ -247,11 +247,9 @@ QemuOptsList qemu_rtc_opts = {
>>>           },{
>>>               .name = "clock",
>>>               .type = QEMU_OPT_STRING,
>>> -#ifdef TARGET_I386
>>>           },{
>>>               .name = "driftfix",
>>>               .type = QEMU_OPT_STRING,
>>> -#endif
>>>           },
>>>           { /* end if list */ }
>>>       },
>>>        
>>
>> Is there any reason this patch hasn't been applied to GIT yet ? I'm told
>> that using this option is critical to making certain guests work reliably
>> so we want to use it from libvirt/virt-manager for the OS in question.
>>      
> Multiple pings have gone out already; I hope it's in someone's queue to
> be applied
>    

Sorry, I've got it now.  In the future, please resend patches with 
[RESEND] as opposed to just bumping it.

Regards,

Anthony Liguori

> 		Amit
>    

Re: [Qemu-devel] [PATCH v2] rtc: Remove TARGET_I386 from qemu-config.c, enables driftfix

[Anthony Liguori]

On 06/23/2010 09:44 AM, Amit Shah wrote:
> qemu-config.c doesn't contain any target-specific code, and the
> TARGET_I386 conditional code didn't get compiled as a result. Removing
> this enables the driftfix parameter for rtc.
>
> Signed-off-by: Amit Shah<amit.shah@redhat.com>
>    

Applied.  Thanks.

Regards,

Anthony Liguori
> ---
>   qemu-config.c |    2 --
>   1 files changed, 0 insertions(+), 2 deletions(-)
>
> diff --git a/qemu-config.c b/qemu-config.c
> index 95abe61..730ffd9 100644
> --- a/qemu-config.c
> +++ b/qemu-config.c
> @@ -247,11 +247,9 @@ QemuOptsList qemu_rtc_opts = {
>           },{
>               .name = "clock",
>               .type = QEMU_OPT_STRING,
> -#ifdef TARGET_I386
>           },{
>               .name = "driftfix",
>               .type = QEMU_OPT_STRING,
> -#endif
>           },
>           { /* end if list */ }
>       },
>