From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50906) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YEHwc-0006sS-6s for qemu-devel@nongnu.org; Thu, 22 Jan 2015 08:37:11 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YEHwX-00045R-8z for qemu-devel@nongnu.org; Thu, 22 Jan 2015 08:37:10 -0500 Message-ID: <54C0FCFE.7060204@suse.de> Date: Thu, 22 Jan 2015 14:37:02 +0100 From: Alexander Graf MIME-Version: 1.0 References: <1421856072-25026-1-git-send-email-mark.cave-ayland@ilande.co.uk> <1421856072-25026-6-git-send-email-mark.cave-ayland@ilande.co.uk> In-Reply-To: <1421856072-25026-6-git-send-email-mark.cave-ayland@ilande.co.uk> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCHv2 5/7] target-ppc: force update of msr bits in cpu_post_load List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Mark Cave-Ayland , qemu-devel@nongnu.org, qemu-ppc@nongnu.org On 21.01.15 17:01, Mark Cave-Ayland wrote: > Since env->msr has already been restored by the time cpu_post_load is called, > make sure that ppc_store_msr() is explicitly called with all msr bits except > MSR_TGPR marked as invalid. > > This solves the issue where MSR flags aren't set correctly when restoring a VM > snapshot, in particular the internal env->excp_prefix value when MSR_EP has > been altered by a guest. > > Signed-off-by: Mark Cave-Ayland > --- > target-ppc/machine.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/target-ppc/machine.c b/target-ppc/machine.c > index c801b82..fc8ddcd 100644 > --- a/target-ppc/machine.c > +++ b/target-ppc/machine.c > @@ -159,6 +159,7 @@ static int cpu_post_load(void *opaque, int version_id) > PowerPCCPU *cpu = opaque; > CPUPPCState *env = &cpu->env; > int i; > + target_ulong msr; > > /* > * We always ignore the source PVR. The user or management > @@ -190,7 +191,12 @@ static int cpu_post_load(void *opaque, int version_id) > /* Restore htab_base and htab_mask variables */ > ppc_store_sdr1(env, env->spr[SPR_SDR1]); > } > - hreg_compute_hflags(env); > + > + /* Mark msr bits except MSR_TGPR invalid before restoring */ > + msr = env->msr; > + env->msr ^= ~(1 << MSR_TGPR); Doesn't this need to be 1ULL? Alex > + ppc_store_msr(env, msr); > + > hreg_compute_mem_idx(env); > > return 0; >