[OpenBIOS] [PATCH] ppc: force target CPU

[Mark Cave-Ayland]

On 20/01/16 08:33, Laurent Vivier wrote:

> Le 19/01/2016 21:33, Mark Cave-Ayland a écrit :
>> On 19/01/16 18:34, Hervé Poussineau wrote:
>>
>>> Le 19/01/2016 19:05, Andreas Färber a écrit :
>>>> Am 19.01.2016 um 18:42 schrieb Segher Boessenkool:
>>>>> On Tue, Jan 19, 2016 at 01:50:13AM +0100, Laurent Vivier wrote:
>>>>>> According to this series (from Hervé):
>>>>>>
>>>>>>      https://lists.gnu.org/archive/html/qemu-ppc/2015-06/msg00110.html
>>>>>>
>>>>>> RS/6000 7020/40p is a PReP machine.
>>>>>>
>>>>>> And according to this page:
>>>>>>
>>>>>>     https://en.wikipedia.org/wiki/RS/6000#Type_7020
>>>>>>
>>>>>> RS/6000 7020/40p is PowerPC 601 based.
>>>>>
>>>>> 601 doesn't have much to do with 602.  601 is not fully PowerPC; you do
>>>>> not want to deal with it.
>>>>
>>>> I believe Hervé was using a real firmware from IBM FTP for that one, not
>>>> OpenBIOS.
>>>>
>>>> Andreas
>>>>
>>>
>>> Yes, I was using the real IBM firmware for my tests.
>>> As IBM 7020 was upgradable to 604, real firmware works with 601 and 604
>>> types.
>>> Let's say that QEMU emulates an upgraded 7020, by using a 604 cpu by
>>> default.
>>> As a consequence, I see no problem compiling OpenBIOS for a 604 CPU.
>>
>> Great! If we go with Segher's suggestion for 64-bit then we end up with
>> this:
>>
>> 32-bit: -mcpu=604
>> 64-bit: -mcpu=970 -mno-altivec
> 
> I think you can use "-mcpu=power4" here, this is the old default value
> for gcc.

It seems they are effectively the same, so I don't have any strong
preference either way.

Laurent, would you like to resubmit a revised patch with the above
changes to the list? As soon as Andreas ACKs it then I'll apply it to
SVN trunk.


ATB,

Mark.