[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] ARM GIC and CPU state saving/loading fix
From: |
Paul Brook |
Subject: |
Re: [Qemu-devel] [PATCH] ARM GIC and CPU state saving/loading fix |
Date: |
Fri, 14 Oct 2011 16:04:40 +0100 |
User-agent: |
KMail/1.13.7 (Linux/3.0.0-1-amd64; KDE/4.6.5; x86_64; ; ) |
> Fixes two trivial indices errors.
No.
You're doing two much in a single patch. While both happen to be bug in the
save/restore code involving arrays, these are not two instances of the same
bug. The justification for each change is completely different.
Even if each change was obviously correct, I believe putting them together
into a single commit makes the result non-trivial. The fact your patch
introduces a bug strongly suggests it shouldn't have been considered trivial
to start with.
> @@ -53,7 +53,7 @@ void cpu_save(QEMUFile *f, void *opaque)
> if (arm_feature(env, ARM_FEATURE_VFP)) {
> - for (i = 0; i < 16; i++) {
> + for (i = 16; i < 32; i++) {
> CPU_DoubleU u;
> u.d = env->vfp.regs[i];
I'm pretty sure this is wrong.
Paul