Am 30.01.2013 12:02, schrieb Alexander Graf:
>
> On 30.01.2013, at 11:40, Andreas Färber wrote:
>
>> Am 30.01.2013 11:15, schrieb Alexander Graf:
>>>
>>> On 27.01.2013, at 14:32, Andreas Färber wrote:
>>>
>>>> To keep log format backwards compatible, cast to target_ulong
>>>> rather than using HWADDR_PRIx.
>>>>
>>>> Signed-off-by: Andreas Färber <afaer...@suse.de>
>>>> ---
>>>> target-ppc/mmu_helper.c | 8 +++++---
>>>> 1 Datei geändert, 5 Zeilen hinzugefügt(+), 3 Zeilen entfernt(-)
>>>>
>>>> diff --git a/target-ppc/mmu_helper.c b/target-ppc/mmu_helper.c
>>>> index 0aee7a9..14fa25a 100644
>>>> --- a/target-ppc/mmu_helper.c
>>>> +++ b/target-ppc/mmu_helper.c
>>>> @@ -589,7 +589,8 @@ static inline int find_pte2(CPUPPCState *env,
>>>> mmu_ctx_t *ctx, int is_64b, int h,
>>>> r = pte64_check(ctx, pte0, pte1, h, rw, type);
>>>> LOG_MMU("Load pte from " TARGET_FMT_lx " => " TARGET_FMT_lx " "
>>>> TARGET_FMT_lx " %d %d %d " TARGET_FMT_lx "\n",
>>>> - pteg_off + (i * 16), pte0, pte1, (int)(pte0 & 1), h,
>>>> + (target_ulong)pteg_off + (i * 16), pte0, pte1,
>>>> + (int)(pte0 & 1), h,
>>>
>>> Please change the format string to reflect the variables' types instead of
>>> doing these casts please :). Same for the ones below.
>>
>> Please specify how you would like the format string to look like then.
>> As indicated above, we only have HWADDR_PRIx, so we must hardcode the
>> amount of zeroes to use then. With TARGET_FMT_lx it uses %08 for
>> ppc[emb] and %016 for ppc64. An #ifdef TARGET_PPC64 seemed uglier. ;)
>
> Just make it %08 always. Higher addresses on PPC usually have the highest 4
> bits set, so it would still align well.
Fine with me.
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
