qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [Qemu-ppc] [PATCH v3] Modifications to fix PowerPC on 6


From: Peter Maydell
Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH v3] Modifications to fix PowerPC on 64bit host, bug 1052857
Date: Thu, 3 Jan 2013 17:51:05 +0000

On 3 January 2013 17:37, Alexander Graf <address@hidden> wrote:
> On 03.01.2013, at 18:28, Peter Maydell wrote:
>> Untested, and I haven't looked up the PPC ABI to check that the
>> function is overall doing the right thing, but with those caveats:
>> Reviewed-by: Peter Maydell <address@hidden>
>
> Considering the state it was in before, I'd say applying the patch
> is an improvement regardless on how broken any code around it
> might be :).

Forgot, this patch doesn't address the other issue I mentioned,
where

    env->gpr[4] = (target_ulong) h2g(sc);

is passing the guest a pointer to potentially about to be freed
memory and should be doing something like
   env->gpr[4] = frame_addr + offsetof(struct target_sigframe, sctx);
instead.

-- PMM



reply via email to

[Prev in Thread] Current Thread [Next in Thread]