Re: [gentoo-catalyst][PATCH v1] arch/powerpc.py:add support for subarch power8 bug#560818

["Anthony G. Basile" <blueness@gentoo.org>]

On 9/21/15 10:02 AM, Rick "Zero_Chaos" Farina wrote:
> On 09/21/2015 08:38 AM, Leno Hou wrote:
>
> I see no issues here, but I defer to PPC team.
>
> -Zero_Chaos
>
>> Currently catalyst does not support ppc64le arch, we add
>> subarch power8 to support ppc64le and make stage files.
>>
>> Signed-off-by: Leno Hou <lenohou@gmail.com>
>> ---
>>   catalyst/arch/powerpc.py | 11 ++++++++++-
>>   1 file changed, 10 insertions(+), 1 deletion(-)
>>
>> diff --git a/catalyst/arch/powerpc.py b/catalyst/arch/powerpc.py
>> index f903b38..49bdd92 100644
>> --- a/catalyst/arch/powerpc.py
>> +++ b/catalyst/arch/powerpc.py
>> @@ -108,6 +108,14 @@ class arch_power6(arch_ppc64):
>>   		self.settings["CFLAGS"]="-O2 -pipe -mcpu=power6 -mtune=power6"
>>   		self.settings["HOSTUSE"]=["altivec","ibm"]
>>   
>> +class arch_power8(arch_ppc64):
>> +	"builder class for power8 under ppc64le"
>> +	def __init__(self,myspec):
>> +		arch_ppc64.__init__(self,myspec)
>> +		self.settings["CFLAGS"]="-O2 -pipe -mcpu=power8 -mtune=power8 -mabi=elfv2"
>> +		self.settings["CHOST"]="powerpc64le-linux-gnu"
>> +		self.settings["HOSTUSE"]=["altivec","ibm"]
>> +
>>   def register():
>>   	"Inform main catalyst program of the contents of this plugin."
>>   	return ({
>> @@ -122,6 +130,7 @@ def register():
>>   		"power4"	: arch_power4,
>>   		"power5"	: arch_power5,
>>   		"power6"	: arch_power6,
>> +		"power8"	: arch_power8,
>>   		"ppc"		: arch_ppc,
>>   		"ppc64"		: arch_ppc64
>> -	}, ("ppc","ppc64","powerpc","powerpc64"))
>> +	}, ("ppc","ppc64","ppc64le","powerpc","powerpc64","powerpc64le"))
>>

The CHOST is incorrect.  It should powerpc64le-unknown-linux-gnu. Leno, 
can you please try to switch it to that and do another catalyst run.  
You can begin with your current stage3 with the above CHOST and it 
should work out correctly to the new CHOST name.  In another place you 
said there is no compiler "powerpc64le-unknown-linux-gnu" but that makes 
no sense.  The tuple is parsed by gnuconfig script and it should work 
like all other CHOST names we have in gentoo.

-- 
Anthony G. Basile, Ph.D.
Gentoo Linux Developer [Hardened]
E-Mail    : blueness@gentoo.org
GnuPG FP  : 1FED FAD9 D82C 52A5 3BAB  DC79 9384 FA6E F52D 4BBA
GnuPG ID  : F52D4BBA