Re: [Qemu-devel] [PATCH2] ppc: provide PIR register on all book-S CPUs

[Alexander Graf]

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, ...).