[PATCH] gcc-runtime.inc: disable libitm for little endian MIPS too

[PATCH] gcc-runtime.inc: disable libitm for little endian MIPS too

[Andre McCurdy]

libitm is already disabled for big endian MIPS, but needs to be
disabled for little endian MIPS targets too.

Signed-off-by: Andre McCurdy <armccurdy@gmail.com>
---
 meta/recipes-devtools/gcc/gcc-runtime.inc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-devtools/gcc/gcc-runtime.inc b/meta/recipes-devtools/gcc/gcc-runtime.inc
index 707db37..1c3dd54 100644
--- a/meta/recipes-devtools/gcc/gcc-runtime.inc
+++ b/meta/recipes-devtools/gcc/gcc-runtime.inc
@@ -12,7 +12,9 @@ EXTRA_OECONF_append_linuxstdbase = " --enable-clocale=gnu"
 
 RUNTIMELIBITM = "libitm"
 RUNTIMELIBITM_mips = ""
+RUNTIMELIBITM_mipsel = ""
 RUNTIMELIBITM_mips64 = ""
+RUNTIMELIBITM_mips64el = ""
 
 RUNTIMETARGET = "libssp libstdc++-v3 libgomp libatomic ${RUNTIMELIBITM} \
     ${@bb.utils.contains_any('FORTRAN', [',fortran',',f77'], 'libquadmath', '', d)} \
-- 
1.9.1

Re: [PATCH] gcc-runtime.inc: disable libitm for little endian MIPS too

[Mark Hatle]

On 3/2/16 11:07 PM, Andre McCurdy wrote:
> libitm is already disabled for big endian MIPS, but needs to be
> disabled for little endian MIPS targets too.

No objection to the patch, but do we have any little endian MIPS currently
defined or buildable to test?

All of the samples I've seen are all big endian these days.  (QEMU or actual
boards.)

--Mark

> Signed-off-by: Andre McCurdy <armccurdy@gmail.com>
> ---
>  meta/recipes-devtools/gcc/gcc-runtime.inc | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/meta/recipes-devtools/gcc/gcc-runtime.inc b/meta/recipes-devtools/gcc/gcc-runtime.inc
> index 707db37..1c3dd54 100644
> --- a/meta/recipes-devtools/gcc/gcc-runtime.inc
> +++ b/meta/recipes-devtools/gcc/gcc-runtime.inc
> @@ -12,7 +12,9 @@ EXTRA_OECONF_append_linuxstdbase = " --enable-clocale=gnu"
>  
>  RUNTIMELIBITM = "libitm"
>  RUNTIMELIBITM_mips = ""
> +RUNTIMELIBITM_mipsel = ""
>  RUNTIMELIBITM_mips64 = ""
> +RUNTIMELIBITM_mips64el = ""
>  
>  RUNTIMETARGET = "libssp libstdc++-v3 libgomp libatomic ${RUNTIMELIBITM} \
>      ${@bb.utils.contains_any('FORTRAN', [',fortran',',f77'], 'libquadmath', '', d)} \
> 

Re: [PATCH] gcc-runtime.inc: disable libitm for little endian MIPS too

[Khem Raj]

On Wed, Mar 2, 2016 at 10:04 PM, Mark Hatle <mark.hatle@windriver.com> wrote:
> On 3/2/16 11:07 PM, Andre McCurdy wrote:
>> libitm is already disabled for big endian MIPS, but needs to be
>> disabled for little endian MIPS targets too.
>
> No objection to the patch, but do we have any little endian MIPS currently
> defined or buildable to test?
>

most of mips based set-top boxes are mipsel.

> All of the samples I've seen are all big endian these days.  (QEMU or actual
> boards.)
>
> --Mark
>
>> Signed-off-by: Andre McCurdy <armccurdy@gmail.com>
>> ---
>>  meta/recipes-devtools/gcc/gcc-runtime.inc | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/meta/recipes-devtools/gcc/gcc-runtime.inc b/meta/recipes-devtools/gcc/gcc-runtime.inc
>> index 707db37..1c3dd54 100644
>> --- a/meta/recipes-devtools/gcc/gcc-runtime.inc
>> +++ b/meta/recipes-devtools/gcc/gcc-runtime.inc
>> @@ -12,7 +12,9 @@ EXTRA_OECONF_append_linuxstdbase = " --enable-clocale=gnu"
>>
>>  RUNTIMELIBITM = "libitm"
>>  RUNTIMELIBITM_mips = ""
>> +RUNTIMELIBITM_mipsel = ""
>>  RUNTIMELIBITM_mips64 = ""
>> +RUNTIMELIBITM_mips64el = ""
>>
>>  RUNTIMETARGET = "libssp libstdc++-v3 libgomp libatomic ${RUNTIMELIBITM} \
>>      ${@bb.utils.contains_any('FORTRAN', [',fortran',',f77'], 'libquadmath', '', d)} \
>>
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [PATCH] gcc-runtime.inc: disable libitm for little endian MIPS too

[Andre McCurdy]

On Wed, Mar 2, 2016 at 10:24 PM, Khem Raj <raj.khem@gmail.com> wrote:
> On Wed, Mar 2, 2016 at 10:04 PM, Mark Hatle <mark.hatle@windriver.com> wrote:
>> On 3/2/16 11:07 PM, Andre McCurdy wrote:
>>> libitm is already disabled for big endian MIPS, but needs to be
>>> disabled for little endian MIPS targets too.
>>
>> No objection to the patch, but do we have any little endian MIPS currently
>> defined or buildable to test?
>
> most of mips based set-top boxes are mipsel.
>
>> All of the samples I've seen are all big endian these days.  (QEMU or actual
>> boards.)

Of the MIPS boards supported by OpenWRT, there seem to be an even
mixture of big and little endian:

  https://dev.openwrt.org/wiki/platforms

I don't know how many (if any) of these boards are supported by OE though...


>> --Mark
>>
>>> Signed-off-by: Andre McCurdy <armccurdy@gmail.com>
>>> ---
>>>  meta/recipes-devtools/gcc/gcc-runtime.inc | 2 ++
>>>  1 file changed, 2 insertions(+)
>>>
>>> diff --git a/meta/recipes-devtools/gcc/gcc-runtime.inc b/meta/recipes-devtools/gcc/gcc-runtime.inc
>>> index 707db37..1c3dd54 100644
>>> --- a/meta/recipes-devtools/gcc/gcc-runtime.inc
>>> +++ b/meta/recipes-devtools/gcc/gcc-runtime.inc
>>> @@ -12,7 +12,9 @@ EXTRA_OECONF_append_linuxstdbase = " --enable-clocale=gnu"
>>>
>>>  RUNTIMELIBITM = "libitm"
>>>  RUNTIMELIBITM_mips = ""
>>> +RUNTIMELIBITM_mipsel = ""
>>>  RUNTIMELIBITM_mips64 = ""
>>> +RUNTIMELIBITM_mips64el = ""
>>>
>>>  RUNTIMETARGET = "libssp libstdc++-v3 libgomp libatomic ${RUNTIMELIBITM} \
>>>      ${@bb.utils.contains_any('FORTRAN', [',fortran',',f77'], 'libquadmath', '', d)} \
>>>
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core