diff mbox

: exec: fix cpu rework fallout (was cputlb: Change tlb_flush() argument to CPUState)

Message ID 53271F18.7080201@de.ibm.com
State New
Headers show

Commit Message

Christian Borntraeger March 17, 2014, 4:13 p.m. UTC 
I need the following to not crash on migration post load:

tlb_flush wants an CPUState. This was unnoticed by gcc because env_ptr
is a void pointer.

Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
---
 exec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Andreas Färber March 17, 2014, 7:37 p.m. UTC | #1 
Am 17.03.2014 17:13, schrieb Christian Borntraeger:
> I need the following to not crash on migration post load:
> 
> tlb_flush wants an CPUState. This was unnoticed by gcc because env_ptr
> is a void pointer.
> 
> Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>

Thanks, applied to qom-cpu with reworked commit message:
https://github.com/afaerber/qemu-cpu/commits/qom-cpu

Andreas
diff mbox

Patch

--- a/exec.c
+++ b/exec.c
@@ -420,7 +420,7 @@  static int cpu_common_post_load(void *opaque, int version_id)
     /* 0x01 was CPU_INTERRUPT_EXIT. This line can be removed when the
        version_id is increased. */
     cpu->interrupt_request &= ~0x01;
-    tlb_flush(cpu->env_ptr, 1);
+    tlb_flush(cpu, 1);
 
     return 0;
 }