commit 5e992a8e337e710ea2d02f35668ac55a80e15f99
author Eduardo Habkost <ehabkost@redhat.com> Fri Apr 24 15:49:15 2015 -0300
committer Eduardo Habkost <ehabkost@redhat.com> Tue Sep 27 16:17:17 2016 -0300
tree e644d74361c1e9699479d4d240f114eb158f0aa6
parent 0456441b5eb6694a561ad5bb8dad52483e6a08d0

target-i386: Add a marker to end of the region zeroed on reset

Instead of using cpuid_level, use an empty struct as a marker
(like we already did with {start,end}_init_save). This will avoid
accidentaly resetting the wrong fields if we change the field
ordering on CPUX86State.

Reviewed-by: Richard Henderson <rth@twiddle.net>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>

target-i386/cpu.c

diff --git a/target-i386/cpu.c b/target-i386/cpu.c
index 920b78f..26f0e59 100644
--- a/target-i386/cpu.c
+++ b/target-i386/cpu.c
@@ -2714,7 +2714,7 @@
 
     xcc->parent_reset(s);
 
-    memset(env, 0, offsetof(CPUX86State, cpuid_level));
+    memset(env, 0, offsetof(CPUX86State, end_reset_fields));
 
     tlb_flush(s, 1);