Re: [Qemu-devel] [PATCH] arm cpu state loading fix

2011-10-17 Thread Peter Maydell 
On 17 October 2011 12:50, Juan Quintela  wrote:
> Dmitry Koshelev  wrote:
>> Floating registers loading fix.
>>
>> Signed-off-by: Dmitry Koshelev 
>
> Reviewed-by: Juan Quintela 
>
> /me really wonders if arm migration has ever worked :-(

I've used save/restore successfully on versatilepb models
before, but only for debug, not for serious work (which
would be why I didn't hit this bug -- the test case I was
working on was probably integer only code); and in order
to get that to work I had to fix a lot of devices. There
are still some ARM devboard device models which are missing
save/restore support.

-- PMM

Re: [Qemu-devel] [PATCH] arm cpu state loading fix

2011-10-17 Thread Juan Quintela 
Dmitry Koshelev  wrote:
> Floating registers loading fix.
>
> Signed-off-by: Dmitry Koshelev 

Reviewed-by: Juan Quintela 

/me really wonders if arm migration has ever worked :-(

Re: [Qemu-devel] [PATCH] arm cpu state loading fix

2011-10-17 Thread Peter Maydell 
On 17 October 2011 11:53, Dmitry Koshelev  wrote:
> Floating registers loading fix.
>
> Signed-off-by: Dmitry Koshelev 
> ---
>  target-arm/machine.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/target-arm/machine.c b/target-arm/machine.c
> index 3925d3a..73d82c9 100644
> --- a/target-arm/machine.c
> +++ b/target-arm/machine.c
> @@ -175,7 +175,7 @@ int cpu_load(QEMUFile *f, void *opaque, int version_id)
>         env->vfp.vec_stride = qemu_get_be32(f);
>
>         if (arm_feature(env, ARM_FEATURE_VFP3)) {
> -            for (i = 0;  i < 16; i++) {
> +            for (i = 16;  i < 32; i++) {
>                 CPU_DoubleU u;
>                 u.l.upper = qemu_get_be32(f);
>                 u.l.lower = qemu_get_be32(f);

Reviewed-by: Peter Maydell 

-- PMM

[Qemu-devel] [PATCH] arm cpu state loading fix

2011-10-17 Thread Dmitry Koshelev 
Floating registers loading fix.

Signed-off-by: Dmitry Koshelev 
---
 target-arm/machine.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/target-arm/machine.c b/target-arm/machine.c
index 3925d3a..73d82c9 100644
--- a/target-arm/machine.c
+++ b/target-arm/machine.c
@@ -175,7 +175,7 @@ int cpu_load(QEMUFile *f, void *opaque, int version_id)
 env->vfp.vec_stride = qemu_get_be32(f);

 if (arm_feature(env, ARM_FEATURE_VFP3)) {
-for (i = 0;  i < 16; i++) {
+for (i = 16;  i < 32; i++) {
 CPU_DoubleU u;
 u.l.upper = qemu_get_be32(f);
 u.l.lower = qemu_get_be32(f);