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.
Alex
