Re: [Qemu-devel] [PATCH-v3 2/2] kvm: Fix unused-but-set-variable warning

2011-06-14 Thread Christophe Fergeau
On Sat, Jun 11, 2011 at 02:06:00PM +0100, Stefan Hajnoczi wrote: > On Sat, Jun 11, 2011 at 8:46 AM, Jan Kiszka wrote: > > Please skip this one. I'll post a patch fixing the issue that this > > warning revealed (lacking save/restore of FPU OP, IP and DP). > > Okay, dropped from trivial-patches in

Re: [Qemu-devel] [PATCH-v3 2/2] kvm: Fix unused-but-set-variable warning

2011-06-11 Thread Stefan Hajnoczi
On Sat, Jun 11, 2011 at 8:46 AM, Jan Kiszka wrote: > On 2011-06-10 20:46, Stefan Hajnoczi wrote: >> On Tue, May 31, 2011 at 09:53:49AM +0200, Christophe Fergeau wrote: >>> Based on a patch from Hans de Goede >>> >>> This warning is new in gcc 4.6. >>> >>> Acked-by: Amit Shah >>> Signed-off-by: C

Re: [Qemu-devel] [PATCH-v3 2/2] kvm: Fix unused-but-set-variable warning

2011-06-11 Thread Jan Kiszka
On 2011-06-10 20:46, Stefan Hajnoczi wrote: > On Tue, May 31, 2011 at 09:53:49AM +0200, Christophe Fergeau wrote: >> Based on a patch from Hans de Goede >> >> This warning is new in gcc 4.6. >> >> Acked-by: Amit Shah >> Signed-off-by: Christophe Fergeau >> --- >> target-i386/kvm.c |3 +-- >>

Re: [Qemu-devel] [PATCH-v3 2/2] kvm: Fix unused-but-set-variable warning

2011-06-10 Thread Stefan Hajnoczi
On Tue, May 31, 2011 at 09:53:49AM +0200, Christophe Fergeau wrote: > Based on a patch from Hans de Goede > > This warning is new in gcc 4.6. > > Acked-by: Amit Shah > Signed-off-by: Christophe Fergeau > --- > target-i386/kvm.c |3 +-- > 1 files changed, 1 insertions(+), 2 deletions(-) T

Re: [Qemu-devel] [PATCH-v3 2/2] kvm: Fix unused-but-set-variable warning

2011-06-10 Thread Stefan Hajnoczi
On Fri, Jun 10, 2011 at 11:43 AM, Christophe Fergeau wrote: > Ping? Let me add these patches to the trivial-patches tree since they haven't been picked up. Stefan

Re: [Qemu-devel] [PATCH-v3 2/2] kvm: Fix unused-but-set-variable warning

2011-06-10 Thread Christophe Fergeau
Ping? On Tue, May 31, 2011 at 09:53:49AM +0200, Christophe Fergeau wrote: > Based on a patch from Hans de Goede > > This warning is new in gcc 4.6. > > Acked-by: Amit Shah > Signed-off-by: Christophe Fergeau > --- > target-i386/kvm.c |3 +-- > 1 files changed, 1 insertions(+), 2 deletion

[Qemu-devel] [PATCH-v3 2/2] kvm: Fix unused-but-set-variable warning

2011-05-31 Thread Christophe Fergeau
Based on a patch from Hans de Goede This warning is new in gcc 4.6. Acked-by: Amit Shah Signed-off-by: Christophe Fergeau --- target-i386/kvm.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/target-i386/kvm.c b/target-i386/kvm.c index faedc6c..58a70bc 100644 --- a/