[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [Qemu-devel] [PATCH qom-cpu-next] e500: Replace open-code
From: |
Andreas Färber |
Subject: |
Re: [Qemu-ppc] [Qemu-devel] [PATCH qom-cpu-next] e500: Replace open-coded loop with qemu_get_cpu() |
Date: |
Fri, 15 Feb 2013 18:24:37 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130105 Thunderbird/17.0.2 |
Am 15.02.2013 17:45, schrieb Alexander Graf:
>
> On 15.02.2013, at 15:35, Andreas Färber wrote:
>
>> Am 15.02.2013 15:29, schrieb Alexander Graf:
>>>
>>> On 15.02.2013, at 15:25, Andreas Färber wrote:
>>>
>>>> Since we still need env for ppc-specific fields, obtain it via the new
>>>> env_ptr fields to avoid "cpu" name conflicts between CPUState and
>>>> PowerPCCPU for now.
>>>>
>>>> This fixes a potential issue with env being NULL at the end of the loop
>>>> but cpu still being a valid pointer corresponding to a previous env.
>>>>
>>>> Signed-off-by: Andreas Färber <address@hidden>
>>>> ---
>>>> Alex, this is a rebased and slimmed-down version of yesterday's paste.
>>>> Can you ack please? Thanks, /-F
>>>
>>> In your QOM conversion you also converted other places like this. Have you
>>> verified that these don't suffer from the same problem?
>>
>> Not yet. Not all callers are using this pattern though, so solutions
>> would differ and be per-case patches.
>
> This specific patch is:
>
> Acked-by: Alexander Graf <address@hidden>
Thanks, applied to qom-cpu-next queue:
https://github.com/afaerber/qemu-cpu/commits/qom-cpu-next
> However, please make sure to fix the other places too :)
In whole there's two e500 patches fixing a potential bug, plus
non-functional cleanups for sPAPR, monitor and QMP code.
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg