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

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

[Leno Hou]

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"))
-- 
2.1.0

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

[Rick "Zero_Chaos" Farina]

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"))
> 

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

[Anthony G. Basile]

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

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

[Mike Frysinger]

[-- Attachment #1: Type: text/plain, Size: 871 bytes --]

On 21 Sep 2015 13:15, Anthony G. Basile wrote:
> On 9/21/15 10:02 AM, Rick "Zero_Chaos" Farina wrote:
> > On 09/21/2015 08:38 AM, Leno Hou wrote:
> >> +		self.settings["CHOST"]="powerpc64le-linux-gnu"
> 
> 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.

powerpc64le-linux-gnu is a valid tuple (vendor can generally be omitted),
but in Gentoo we do use "unknown" for our ppc ports.  so do as Anthony
says ;).
-mike

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

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

[Leno Hou]

[-- Attachment #1: Type: text/plain, Size: 1216 bytes --]

On Tue, Sep 22, 2015 at 2:21 AM, Mike Frysinger <vapier@gentoo.org> wrote:

> On 21 Sep 2015 13:15, Anthony G. Basile wrote:
> > On 9/21/15 10:02 AM, Rick "Zero_Chaos" Farina wrote:
> > > On 09/21/2015 08:38 AM, Leno Hou wrote:
> > >> +          self.settings["CHOST"]="powerpc64le-linux-gnu"
> >
> > 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.
>
> powerpc64le-linux-gnu is a valid tuple (vendor can generally be omitted),
> but in Gentoo we do use "unknown" for our ppc ports.  so do as Anthony
> says ;).
> -mike
>

When set CHOST="powerpc64le-linux-gnu", successfully compiled stage1.
But CHOST="powerpc64-unknown-linux-gnu", something happens as following.
https://bpaste.net/show/189b2b12d8e2

So I think CHOST="powerpc64le-linux-gnu" would be better.

-Leno Hou

[-- Attachment #2: Type: text/html, Size: 2301 bytes --]

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

[Mike Frysinger]

[-- Attachment #1: Type: text/plain, Size: 1454 bytes --]

On 22 Sep 2015 14:34, Leno Hou wrote:
> On Tue, Sep 22, 2015 at 2:21 AM, Mike Frysinger <vapier@gentoo.org> wrote:
> 
> > On 21 Sep 2015 13:15, Anthony G. Basile wrote:
> > > On 9/21/15 10:02 AM, Rick "Zero_Chaos" Farina wrote:
> > > > On 09/21/2015 08:38 AM, Leno Hou wrote:
> > > >> +          self.settings["CHOST"]="powerpc64le-linux-gnu"
> > >
> > > 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.
> >
> > powerpc64le-linux-gnu is a valid tuple (vendor can generally be omitted),
> > but in Gentoo we do use "unknown" for our ppc ports.  so do as Anthony
> > says ;).
> > -mike
> >
> 
> When set CHOST="powerpc64le-linux-gnu", successfully compiled stage1.
> But CHOST="powerpc64-unknown-linux-gnu", something happens as following.
> https://bpaste.net/show/189b2b12d8e2
> 
> So I think CHOST="powerpc64le-linux-gnu" would be better.

i think you're misreading things.  we didn't say "powerpc64-unknown-linux-gnu",
we said "powerpc64le-unknown-linux-gnu".
-mike

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

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

[Leno Hou]

[-- Attachment #1: Type: text/plain, Size: 1794 bytes --]

On Tue, Sep 22, 2015 at 9:15 PM, Mike Frysinger <vapier@gentoo.org> wrote:

> On 22 Sep 2015 14:34, Leno Hou wrote:
> > On Tue, Sep 22, 2015 at 2:21 AM, Mike Frysinger <vapier@gentoo.org>
> wrote:
> >
> > > On 21 Sep 2015 13:15, Anthony G. Basile wrote:
> > > > On 9/21/15 10:02 AM, Rick "Zero_Chaos" Farina wrote:
> > > > > On 09/21/2015 08:38 AM, Leno Hou wrote:
> > > > >> +          self.settings["CHOST"]="powerpc64le-linux-gnu"
> > > >
> > > > 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.
> > >
> > > powerpc64le-linux-gnu is a valid tuple (vendor can generally be
> omitted),
> > > but in Gentoo we do use "unknown" for our ppc ports.  so do as Anthony
> > > says ;).
> > > -mike
> > >
> >
> > When set CHOST="powerpc64le-linux-gnu", successfully compiled stage1.
> > But CHOST="powerpc64-unknown-linux-gnu", something happens as following.
> > https://bpaste.net/show/189b2b12d8e2
> >
> > So I think CHOST="powerpc64le-linux-gnu" would be better.
>
> i think you're misreading things.  we didn't say
> "powerpc64-unknown-linux-gnu",
> we said "powerpc64le-unknown-linux-gnu".
> -mike
>

I've tested CHOST="powerpc64le-unknown-linux-gnu" and proved you're right.
The results is that got compiler called "powerpc64le-unknown-linux-gnu-*"
in stage 1.
So could you help me to commit PATCH v2 ?  thanks.
-Leno Hou

[-- Attachment #2: Type: text/html, Size: 3062 bytes --]

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

[Anthony G. Basile]

On 9/22/15 1:00 PM, Leno Hou wrote:
> On Tue, Sep 22, 2015 at 9:15 PM, Mike Frysinger <vapier@gentoo.org> wrote:
>
>> On 22 Sep 2015 14:34, Leno Hou wrote:
>>> On Tue, Sep 22, 2015 at 2:21 AM, Mike Frysinger <vapier@gentoo.org>
>> wrote:
>>>> On 21 Sep 2015 13:15, Anthony G. Basile wrote:
>>>>> On 9/21/15 10:02 AM, Rick "Zero_Chaos" Farina wrote:
>>>>>> On 09/21/2015 08:38 AM, Leno Hou wrote:
>>>>>>> +          self.settings["CHOST"]="powerpc64le-linux-gnu"
>>>>> 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.
>>>> powerpc64le-linux-gnu is a valid tuple (vendor can generally be
>> omitted),
>>>> but in Gentoo we do use "unknown" for our ppc ports.  so do as Anthony
>>>> says ;).
>>>> -mike
>>>>
>>> When set CHOST="powerpc64le-linux-gnu", successfully compiled stage1.
>>> But CHOST="powerpc64-unknown-linux-gnu", something happens as following.
>>> https://bpaste.net/show/189b2b12d8e2
>>>
>>> So I think CHOST="powerpc64le-linux-gnu" would be better.
>> i think you're misreading things.  we didn't say
>> "powerpc64-unknown-linux-gnu",
>> we said "powerpc64le-unknown-linux-gnu".
>> -mike
>>
> I've tested CHOST="powerpc64le-unknown-linux-gnu" and proved you're right.
> The results is that got compiler called "powerpc64le-unknown-linux-gnu-*"
> in stage 1.
> So could you help me to commit PATCH v2 ?  thanks.
> -Leno Hou
>

Where is it?  I have commit to the catalyst repo.

-- 
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

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

[Mike Frysinger]

[-- Attachment #1: Type: text/plain, Size: 532 bytes --]

On 22 Sep 2015 16:20, Anthony G. Basile wrote:
> On 9/22/15 1:00 PM, Leno Hou wrote:
> > I've tested CHOST="powerpc64le-unknown-linux-gnu" and proved you're right.
> > The results is that got compiler called "powerpc64le-unknown-linux-gnu-*"
> > in stage 1.
> > So could you help me to commit PATCH v2 ?  thanks.
> 
> Where is it?  I have commit to the catalyst repo.

he posted it to the list, but cc-ed you & i.  maybe that messed up your
filters ?  look for 1442906214-29644-1-git-send-email-lenohou@gmail.com.
-mike

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

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

[Anthony G. Basile]

On 9/22/15 5:53 PM, Mike Frysinger wrote:
> On 22 Sep 2015 16:20, Anthony G. Basile wrote:
>> On 9/22/15 1:00 PM, Leno Hou wrote:
>>> I've tested CHOST="powerpc64le-unknown-linux-gnu" and proved you're right.
>>> The results is that got compiler called "powerpc64le-unknown-linux-gnu-*"
>>> in stage 1.
>>> So could you help me to commit PATCH v2 ?  thanks.
>> Where is it?  I have commit to the catalyst repo.
> he posted it to the list, but cc-ed you & i.  maybe that messed up your
> filters ?  look for 1442906214-29644-1-git-send-email-lenohou@gmail.com.
> -mike
Weird.  anyhow i found it and pushed it to catalyst 2.X and master 
branches.  If you emerge catalyst-2.0.9999 you should get it.  And it 
should be out in the next release 2.0.19.

-- 
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