[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/2] PPC: Add dump_mmu() for 6xx
From: |
Fabien Chouteau |
Subject: |
Re: [Qemu-devel] [PATCH 1/2] PPC: Add dump_mmu() for 6xx |
Date: |
Tue, 18 Jun 2013 18:04:25 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130329 Thunderbird/17.0.5 |
On 06/18/2013 05:31 PM, Alexander Graf wrote:
>> + for (type = 0; type < 2; type++)
>
> You need braces on these. Please run your patch through checkpatch.pl :).
I did ;)
-> ./scripts/checkpatch.pl 0001-PPC-Add-dump_mmu-for-6xx.patch
total: 0 errors, 0 warnings, 51 lines checked
0001-PPC-Add-dump_mmu-for-6xx.patch has no obvious style problems and is ready
for submission.
>
>> + for (way = 0; way < env->nb_ways; way++)
>> + for (entry = env->nb_tlb * type + env->tlb_per_way * way;
>> + entry < (env->nb_tlb * type + env->tlb_per_way * (way +
>> 1));
>> + entry++) {
>> +
>> + tlb = &env->tlb.tlb6[entry];
>> + cpu_fprintf(f, "TLB %02d/%02d %s way:%d %s ["
>> + TARGET_FMT_lx " " TARGET_FMT_lx "]\n",
>> + entry % env->nb_tlb, env->nb_tlb,
>> + type ? "code" : "data", way,
>> + pte_is_valid(tlb->pte0) ? "valid" : "inval",
>> + tlb->EPN, tlb->EPN + TARGET_PAGE_SIZE);
>> + }
>
> I thought 6xx and 74xx also support HTAB and SRs? Shouldn't we dump those as
> well?
>
I don't know what that is, can you send me an example of what the printf line
should be?
Thanks for the review,
--
Fabien Chouteau