[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH2] ppc: provide PIR register on all book-S CPUs
From: |
Alexander Graf |
Subject: |
Re: [Qemu-devel] [PATCH2] ppc: provide PIR register on all book-S CPUs |
Date: |
Mon, 4 Jul 2011 17:57:39 +0200 |
On 04.07.2011, at 17:43, Nathan Whitehorn wrote:
> On 07/04/11 10:21, Alexander Graf wrote:
>> On 12.06.2011, at 17:50, Nathan Whitehorn wrote:
>>
>>> The PIR register is architecturally specified on all PowerPC non-embedded
>>> CPUs, but currently is only available on the 604, 620, and G4. Add it to
>>> all 601-derived CPUs.
>> Looking through the respective specs, I found the following matrix on PIR
>> support:
>>
>> 601 yes
>> 603e ? (no)
>> 604e yes
>> 750 ? (no)
>> 7540 yes
>> 970 yes
>>
>> 601 and 750 user manuals don't mention PIR. The overall 32-bit
>> microarchitecture states PIR as "optional". So I'd say 750 and 603 simply
>> don't implement it. However, that means it's missing for 970 and 601. Could
>> you please update your patch accordingly?
>
> The actual 750 next to my desk has it, at least, but Freescale claims the 603
> doesn't. Should I update it for all of the individual CPUs, then, instead of
> the 6xx patch?
If you have actual hardware to verify against, I'd say that overrules specs.
But yes, please update it for the individual CPUs :).
Alex
PS: Please always reply-to-all on mailing list mails unless there's a very good
reason not to (personal, nda, ...).