Skip to contentSkip to conversation

ARC: bump tools to 2014.12 release

PATCH New Alexey Brodkin 3 participants 12 messages Feb 2, 2015 – Apr 6, 2015

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 ARC: bump tools to 2014.12 release

Now when new shiny tools are released by Synopsys we're ready for version update in Buildroot again.

More details about arc-2014.12 release are available here:
https://github.com/foss-for-synopsys-dwc-arc-processors/toolchain/releases/tag/arc-2014.12

Following patches were removed from GCC since they are a part of release now:
* 200-size_type_unsigned_int.patch
* 300-ptrdiff_type_int.patch
* 400-call-arc_hazard-before-branch-shortening.patch
* 401-fix-length-attribute-for-casesi_load-pattern.patch
* 402-fix-length-of-instructions-that-are-in-delay-slot-and-needs-to-be-predicated.patch
* 403-update-casesi_compact_jump-instruction-length.patch

But since arc-2014.12 tools are still based on GCC 4.8 following patches ar still relevant so moving to the new folder to match ARC gcc bump.
* 100-libstdcxx-uclibc-c99.patch
* 910-gcc-poison-system-directories.patch

Binutils are still based on 2.23 so following patch still makes sense:
* 600-poison-system-directories.patch

Cc Anton Kolesov <[email protected]>
Cc Thomas Petazzoni <[email protected]>
Cc Peter Korsgaard <[email protected]>
---
 package/binutils/Config.in.host                    |  2 +-
 .../600-poison-system-directories.patch            |  0
 package/gcc/Config.in.host                         |  2 +-
 .../arc-2014.08/200-size_type_unsigned_int.patch   | 24 ------
 package/gcc/arc-2014.08/300-ptrdiff_type_int.patch | 26 ------
 ...-call-arc_hazard-before-branch-shortening.patch | 96 ----------------------
 ...-length-attribute-for-casesi_load-pattern.patch | 44 ----------
 ...-in-delay-slot-and-needs-to-be-predicated.patch | 69 ----------------
 ...te-casesi_compact_jump-instruction-length.patch | 31 -------
 .../100-libstdcxx-uclibc-c99.patch                 |  0
 .../910-gcc-poison-system-directories.patch        |  0
 package/gdb/Config.in.host                         |  2 +-
 package/uclibc/Config.in                           |  2 +-
 13 files changed, 4 insertions(+), 294 deletions(-)
 rename package/binutils/{arc-2014.08 => arc-2014.12}/600-poison-system-directories.patch (100%)
 delete mode 100644 package/gcc/arc-2014.08/200-size_type_unsigned_int.patch
 delete mode 100644 package/gcc/arc-2014.08/300-ptrdiff_type_int.patch
 delete mode 100644 package/gcc/arc-2014.08/400-call-arc_hazard-before-branch-shortening.patch
 delete mode 100644 package/gcc/arc-2014.08/401-fix-length-attribute-for-casesi_load-pattern.patch
 delete mode 100644 package/gcc/arc-2014.08/402-fix-length-of-instructions-that-are-in-delay-slot-and-needs-to-be-predicated.patch
 delete mode 100644 package/gcc/arc-2014.08/403-update-casesi_compact_jump-instruction-length.patch
 rename package/gcc/{arc-2014.08 => arc-2014.12}/100-libstdcxx-uclibc-c99.patch (100%)
 rename package/gcc/{arc-2014.08 => arc-2014.12}/910-gcc-poison-system-directories.patch (100%)
package/binutils/Config.in.host +1 -1
index 6dac3c0..0739467 100644
@@ -34,7 +34,7 @@ endchoice
 config BR2_BINUTILS_VERSION
 	string
 	default "2.18-avr32-1.0.1" if BR2_BINUTILS_VERSION_2_18_AVR32_1_0_1
-	default "arc-2014.08"	if BR2_arc
+	default "arc-2014.12"	if BR2_arc
 	default "2.22"		if BR2_BINUTILS_VERSION_2_22
 	default "2.23.2"	if BR2_BINUTILS_VERSION_2_23_2
 	default "2.24"		if BR2_BINUTILS_VERSION_2_24
package/binutils/arc-2014.12/600-poison-system-directories.patch renamed from package/binutils/arc-2014.08/600-poison-system-directories.patch +0 -0
similarity index 100%
rename from package/binutils/arc-2014.08/600-poison-system-directories.patch
rename to package/binutils/arc-2014.12/600-poison-system-directories.patch
package/gcc/Config.in.host +1 -1
index 0750807..24004c4 100644
@@ -97,7 +97,7 @@ config BR2_GCC_VERSION
 	default "4.7.4"     if BR2_GCC_VERSION_4_7_X
 	default "4.8.4"     if BR2_GCC_VERSION_4_8_X
 	default "4.9.2"     if BR2_GCC_VERSION_4_9_X
-	default "arc-2014.08" if BR2_GCC_VERSION_4_8_ARC
+	default "arc-2014.12" if BR2_GCC_VERSION_4_8_ARC
 
 config BR2_EXTRA_GCC_CONFIG_OPTIONS
 	string "Additional gcc options"
package/gcc/arc-2014.08/200-size_type_unsigned_int.patch deleted +0 -24
deleted file mode 100644
index a6a0419..0000000
@@ -1,24 +0,0 @@
-Fixes SIZE_TYPE to be "unsigned int" instead of "long unsigned int".
-
-This makes size_t to be "unsigned" ssize_t which makes happy compiler on data
-type checks.
-
-Fix is taken from current development branch of GCC for ARC and will be a
-part of the next release of ARC tools, so at that point patch should be dropped.
-
-https://github.com/foss-for-synopsys-dwc-arc-processors/gcc/commit/249f040299402647525c3f15b79d319fa7acddd3
-
-Signed-off-by: Alexey Brodkin <[email protected]>
----
-
---- a/gcc/config/arc/arc.h
-+++ b/gcc/config/arc/arc.h
-@@ -487,7 +487,7 @@ if (GET_MODE_CLASS (MODE) == MODE_INT		\
- /* Define this as 1 if `char' should by default be signed; else as 0.  */
- #define DEFAULT_SIGNED_CHAR 0
- 
--#define SIZE_TYPE "long unsigned int"
-+#define SIZE_TYPE "unsigned int"
- #define PTRDIFF_TYPE "long int"
- #define WCHAR_TYPE "int"
- #define WCHAR_TYPE_SIZE 32
package/gcc/arc-2014.08/300-ptrdiff_type_int.patch deleted +0 -26
deleted file mode 100644
index 6ee765b..0000000
@@ -1,26 +0,0 @@
-Redefine PTRDIFF_TYPE
-
-Change of SIZE_TYPE from "long unsigned int" to "unsigned int" introduced
-regression due to existing PTRDIFF_TYPE.
-
-Now to fix regression convert PTRDIFF_TYPE to simple "int".
-
-Fix is taken from current development branch of GCC for ARC and will be a
-part of the next release of ARC tools, so at that point patch should be dropped.
-
-https://github.com/foss-for-synopsys-dwc-arc-processors/gcc/commit/846e92167aa4f486259c9ff44bb4e6cce6097fa4
-
-Signed-off-by: Claudiu Zissulescu <[email protected]>
----
-
---- a/gcc/config/arc/arc.h
-+++ b/gcc/config/arc/arc.h
-@@ -488,7 +488,7 @@ if (GET_MODE_CLASS (MODE) == MODE_INT		\
- #define DEFAULT_SIGNED_CHAR 0
- 
- #define SIZE_TYPE "unsigned int"
--#define PTRDIFF_TYPE "long int"
-+#define PTRDIFF_TYPE "int"
- #define WCHAR_TYPE "int"
- #define WCHAR_TYPE_SIZE 32
- 
package/gcc/arc-2014.08/400-call-arc_hazard-before-branch-shortening.patch deleted +0 -96

96 lines not on this page · raw message

package/gcc/arc-2014.08/401-fix-length-attribute-for-casesi_load-pattern.patch deleted +0 -44

44 lines not on this page · raw message

package/gcc/arc-2014.08/402-fix-length-of-instructions-that-are-in-delay-slot-and-needs-to-be-predicated.patch deleted +0 -69

69 lines not on this page · raw message

package/gcc/arc-2014.08/403-update-casesi_compact_jump-instruction-length.patch deleted +0 -31
deleted file mode 100644
index dd188a4..0000000
@@ -1,31 +0,0 @@
-Update casesi_compact_jump instruction length
-
-Fixes autobuilder issue http://autobuild.buildroot.net/results/bc5/bc5100d6462af4e2805f2bc8d39c87a55d843e2b/
-
---->---
-$ arc-linux-gcc -c -o wps_attr_parse.o -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64  -pipe -Os  -I.../host/usr/arc-buildroot-linux-uclibc/sysroot/usr/include/libnl3/ -I.../build/hostapd-2.2/src -I.../build/hostapd-2.2/src/utils wps_attr_parse.c
-wps_attr_parse.s: Assembler messages:
-wps_attr_parse.s:860: Error: operand out of range (512 is not between -512 and 511)
---->---
-
-Fix is taken from current development branch of GCC for ARC and will be a
-part of the next release of ARC tools, so at that point patch should be dropped.
-
-https://github.com/foss-for-synopsys-dwc-arc-processors/gcc/commit/8e704c43cb50407ec79795f6f7459f09800b9f01
-
-Signed-off-by: Claudiu Zissulescu <[email protected]>
----
- 
-diff --git a/gcc/config/arc/arc.md b/gcc/config/arc/arc.md
-index 88b2e37..8e18703 100644
---- a/gcc/config/arc/arc.md
-+++ b/gcc/config/arc/arc.md
-@@ -3948,7 +3948,7 @@
-   output_asm_insn (s, xop);
-   return \"add_s %2,%2,pcl\n\tj_s%* [%2]\";
- }"
--  [(set_attr "length" "10")
-+  [(set_attr "length" "12")
-    (set_attr "type" "jump")
-    (set_attr "iscompact" "true")
-    (set_attr "cond" "nocond")])
package/gcc/arc-2014.12/100-libstdcxx-uclibc-c99.patch renamed from package/gcc/arc-2014.08/100-libstdcxx-uclibc-c99.patch +0 -0
similarity index 100%
rename from package/gcc/arc-2014.08/100-libstdcxx-uclibc-c99.patch
rename to package/gcc/arc-2014.12/100-libstdcxx-uclibc-c99.patch
package/gcc/arc-2014.12/910-gcc-poison-system-directories.patch renamed from package/gcc/arc-2014.08/910-gcc-poison-system-directories.patch +0 -0
similarity index 100%
rename from package/gcc/arc-2014.08/910-gcc-poison-system-directories.patch
rename to package/gcc/arc-2014.12/910-gcc-poison-system-directories.patch
package/gdb/Config.in.host +1 -1
index 7f9df3a..c8d6a94 100644
@@ -52,7 +52,7 @@ config BR2_GDB_VERSION
 	depends on BR2_PACKAGE_GDB || BR2_PACKAGE_HOST_GDB
 	default "6.7.1-avr32-2.1.5" if BR2_GDB_VERSION_6_7_1_AVR32_2_1_5 || \
                                    (!BR2_PACKAGE_HOST_GDB && BR2_avr32)
-	default "arc-2014.08-gdb" if BR2_arc
+	default "arc-2014.12-gdb" if BR2_arc
 	default "6be65fb56ea6694a9260733a536a023a1e2d4d57" if BR2_microblaze
 	default "7.7.1"    if BR2_GDB_VERSION_7_7 || !BR2_PACKAGE_HOST_GDB
 	default "7.8.2"	   if BR2_GDB_VERSION_7_8
package/uclibc/Config.in +1 -1
index 290f19b..020c47c 100644
@@ -42,7 +42,7 @@ config BR2_UCLIBC_VERSION_STRING
 	string
 	default 0.9.31.1	if BR2_UCLIBC_VERSION_0_9_31
 	default 0.9.33.2	if BR2_UCLIBC_VERSION_0_9_33
-	default "arc-2014.08" if BR2_arc
+	default "arc-2014.12" if BR2_arc
 	default "7bf35c8b7d4a1f97174eb49f47f33946b282114c" if BR2_UCLIBC_VERSION_XTENSA_GIT
 	default BR2_USE_UCLIBC_SNAPSHOT	if BR2_UCLIBC_VERSION_SNAPSHOT
 
Signature · 1 line
2.1.0
#1 lore raw

PATCH 1/1 ARC: bump tools to 2014.12 release

Dear Alexey Brodkin,

Quoted 20 lines · Alexey Brodkin

Now when new shiny tools are released by Synopsys we're ready for version update in Buildroot again.

More details about arc-2014.12 release are available here:
https://github.com/foss-for-synopsys-dwc-arc-processors/toolchain/releases/tag/arc-2014.12

Following patches were removed from GCC since they are a part of release now:
* 200-size_type_unsigned_int.patch
* 300-ptrdiff_type_int.patch
* 400-call-arc_hazard-before-branch-shortening.patch
* 401-fix-length-attribute-for-casesi_load-pattern.patch
* 402-fix-length-of-instructions-that-are-in-delay-slot-and-needs-to-be-predicated.patch
* 403-update-casesi_compact_jump-instruction-length.patch

But since arc-2014.12 tools are still based on GCC 4.8 following patches ar still relevant so moving to the new folder to match ARC gcc bump.
* 100-libstdcxx-uclibc-c99.patch
* 910-gcc-poison-system-directories.patch

Quoted 4 lines

Binutils are still based on 2.23 so following patch still makes sense:
* 600-poison-system-directories.patch

Applied, thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
#2 lore raw

PATCH 1/1 ARC: bump tools to 2014.12 release

In reply to Alexey Brodkin

Dear Alexey Brodkin,

Quoted 20 lines · Alexey Brodkin

Now when new shiny tools are released by Synopsys we're ready for version update in Buildroot again.

More details about arc-2014.12 release are available here:
https://github.com/foss-for-synopsys-dwc-arc-processors/toolchain/releases/tag/arc-2014.12

Following patches were removed from GCC since they are a part of release now:
* 200-size_type_unsigned_int.patch
* 300-ptrdiff_type_int.patch
* 400-call-arc_hazard-before-branch-shortening.patch
* 401-fix-length-attribute-for-casesi_load-pattern.patch
* 402-fix-length-of-instructions-that-are-in-delay-slot-and-needs-to-be-predicated.patch
* 403-update-casesi_compact_jump-instruction-length.patch

But since arc-2014.12 tools are still based on GCC 4.8 following patches ar still relevant so moving to the new folder to match ARC gcc bump.
* 100-libstdcxx-uclibc-c99.patch
* 910-gcc-poison-system-directories.patch

Quoted 8 lines

Binutils are still based on 2.23 so following patch still makes sense:
* 600-poison-system-directories.patch

Cc Anton Kolesov <[email protected]>
Cc Thomas Petazzoni <[email protected]>
Cc Peter Korsgaard <[email protected]>

It seems like the update of GDB for ARC is causing some build problems:

http://autobuild.buildroot.org/results/d62/d62f472d70f60281dfe94cffc46a3e0566fd4a3d/build-end.log

It's trying to build a shared libbfd.so using a statically compiled libiberty.a which wasn't built with -fPIC. I don't know if the problem is specific to ARC GDB or not.

Can you have a look?

Thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
#3 lore raw

PATCH 1/1 ARC: bump tools to 2014.12 release

Hi Thomas,

Quoted 32 lines · Thomas Petazzoni

Dear Alexey Brodkin,

Quoted 20 lines · Alexey Brodkin

Now when new shiny tools are released by Synopsys we're ready for version update in Buildroot again.

More details about arc-2014.12 release are available here:
https://github.com/foss-for-synopsys-dwc-arc-processors/toolchain/releases/tag/arc-2014.12

Following patches were removed from GCC since they are a part of release now:
* 200-size_type_unsigned_int.patch
* 300-ptrdiff_type_int.patch
* 400-call-arc_hazard-before-branch-shortening.patch
* 401-fix-length-attribute-for-casesi_load-pattern.patch
* 402-fix-length-of-instructions-that-are-in-delay-slot-and-needs-to-be-predicated.patch
* 403-update-casesi_compact_jump-instruction-length.patch

But since arc-2014.12 tools are still based on GCC 4.8 following patches ar still relevant so moving to the new folder to match ARC gcc bump.
* 100-libstdcxx-uclibc-c99.patch
* 910-gcc-poison-system-directories.patch

Quoted 8 lines

Binutils are still based on 2.23 so following patch still makes sense:
* 600-poison-system-directories.patch

Cc Anton Kolesov <[email protected]>
Cc Thomas Petazzoni <[email protected]>
Cc Peter Korsgaard <[email protected]>
Quoted 7 lines

It seems like the update of GDB for ARC is causing some build problems:

http://autobuild.buildroot.org/results/d62/d62f472d70f60281dfe94cffc46a3e0566fd4a3d/build-end.log

It's trying to build a shared libbfd.so using a statically compiled libiberty.a which wasn't built with -fPIC. I don't know if the problem is specific to ARC GDB or not.

Sorry for late reply.

I've just tried to re-build with defconfig from this autobuild and for me it got successfully to the end.

I'm on master branch @ commit:
--->8---
commit 6a3a9784db170ec05b300f228cf8a4bc48d838ec
Author: Michael Trimarchi <[email protected]>
Date: Sun Mar 8 09:28:13 2015 +0100

python-spidev: new package
--->8---

I'll downgrade to the same commit as mentioned in autobuild:
--->8---
commit b9f39b2b389754bb53294311b938310283fda638
Author: Gustavo Zacarias <[email protected]>
Date: Tue Mar 3 15:43:09 2015 -0300

whois: bump to version 5.2.5
--->8---
and will give it another try, but frankly I don't expect any differences because I didn't find any sensible commits in between these 2 snapshots.

-Alexey

#4 lore raw

PATCH 1/1 ARC: bump tools to 2014.12 release

Hi Alexey,

Le 13 mars 2015 08:18:48 UTC+01:00, Alexey Brodkin <[email protected]> a ?crit :

Quoted 72 lines

Hi Thomas,

Quoted 8 lines · Thomas Petazzoni

Dear Alexey Brodkin,

Quoted 5 lines · Alexey Brodkin

Now when new shiny tools are released by Synopsys we're ready for version update in Buildroot again.

More details about arc-2014.12 release are available here:

https://github.com/foss-for-synopsys-dwc-arc-processors/toolchain/releases/tag/arc-2014.12

Quoted 2 lines
Quoted 2 lines

Following patches were removed from GCC since they are a part of

release

Quoted 6 lines
Quoted 6 lines

now:
* 200-size_type_unsigned_int.patch
* 300-ptrdiff_type_int.patch
* 400-call-arc_hazard-before-branch-shortening.patch
* 401-fix-length-attribute-for-casesi_load-pattern.patch *

402-fix-length-of-instructions-that-are-in-delay-slot-and-needs-to-be-predicated.patch

Quoted 3 lines
Quoted 3 lines

* 403-update-casesi_compact_jump-instruction-length.patch

But since arc-2014.12 tools are still based on GCC 4.8 following

patches

Quoted 1 line
Quoted 1 line

ar still relevant so moving to the new folder to match ARC gcc

bump.

Quoted 4 lines
Quoted 4 lines

* 100-libstdcxx-uclibc-c99.patch
* 910-gcc-poison-system-directories.patch

Binutils are still based on 2.23 so following patch still makes

sense:

Quoted 9 lines
Quoted 7 lines

* 600-poison-system-directories.patch

Cc Anton Kolesov <[email protected]>
Cc Thomas Petazzoni <[email protected]>
Cc Peter Korsgaard <[email protected]>

It seems like the update of GDB for ARC is causing some build

problems:

Quoted 2 lines

http://autobuild.buildroot.org/results/d62/d62f472d70f60281dfe94cffc46a3e0566fd4a3d/build-end.log

Quoted 3 lines

It's trying to build a shared libbfd.so using a statically compiled libiberty.a which wasn't built with -fPIC. I don't know if the

problem

Quoted 1 line

is specific to ARC GDB or not.

Sorry for late reply.

I've just tried to re-build with defconfig from this autobuild and for me it got successfully to the end.

I'm on master branch @ commit:
--->8---
commit 6a3a9784db170ec05b300f228cf8a4bc48d838ec

Author: Michael Trimarchi <[email protected]>
Date:   Sun Mar 8 09:28:13 2015 +0100

python-spidev: new package
--->8---

I'll downgrade to the same commit as mentioned in autobuild:
--->8---
commit b9f39b2b389754bb53294311b938310283fda638

Author: Gustavo Zacarias <[email protected]>
Date:   Tue Mar 3 15:43:09 2015 -0300
Quoted 7 lines

whois: bump to version 5.2.5
--->8---
and will give it another try, but frankly I don't expect any differences
because I didn't find any sensible commits in between these 2 snapshots.

Can you have a look at this patch:
https://patchwork.ozlabs.org/patch/449686/

The issue seem to be related to avr32 removal.

Best regards,
Romain

Quoted 5 lines

-Alexey
_______________________________________________
buildroot mailing list
buildroot at busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

Signature · 1 line
Envoy? de mon t?l?phone Android avec K-9 Mail. Excusez la bri?vet?.
#5 lore raw

PATCH 1/1 ARC: bump tools to 2014.12 release

Quoted 5 lines · Romain Naour

Hi Alexey,
Can you have a look at this patch:
https://patchwork.ozlabs.org/patch/449686/

The issue seem to be related to avr32 removal.

Strange enough I got everything built even from the same git snapshot as it was in autobuilder.

That means I cannot really test proposed patch - it won't fix anything for me because everything works already.

The only reason I may think of for that difference in behavior is host gcc version. I'm on 64-bit Fedora 21 with gcc 4.9.2:
--->8---
gcc version 4.9.2 20150212 (Red Hat 4.9.2-6) (GCC) --->8---

Romain, Thomas what host gcc you/autobuilder use?

-Alexey

#6 lore raw

PATCH 1/1 ARC: bump tools to 2014.12 release

Hi Alexey,

----- Mail original -----

| De: "Alexey Brodkin" <[email protected]>
| ?: "romain naour" <[email protected]>, "thomas petazzoni" <[email protected]>
| Cc: buildroot at uclibc.org
| Envoy?: Vendredi 13 Mars 2015 10:00:34
| Objet: Re: [Buildroot] [PATCH] ARC: bump tools to 2014.12 release
| 
| On Fri, 2015-03-13 at 09:19 +0100, Romain Naour wrote:
| > Hi Alexey,
| > Can you have a look at this patch:
| > https://patchwork.ozlabs.org/patch/449686/
| > 
| > The issue seem to be related to avr32 removal.
| 
| Strange enough I got everything built even from the same git snapshot
| as
| it was in autobuilder.
| 
| That means I cannot really test proposed patch - it won't fix
| anything
| for me because everything works already.
| 
| The only reason I may think of for that difference in behavior is
| host
| gcc version. I'm on 64-bit Fedora 21 with gcc 4.9.2:
|  --->8---
|  gcc version 4.9.2 20150212 (Red Hat 4.9.2-6) (GCC)
|  --->8---
| 
| Romain, Thomas what host gcc you/autobuilder use?

Same at home 64-bit Fedora 21 with gcc 4.9.2 :)
As I said in the patch comment, I can't reproduce the issue...

Best regards,
Romain

| 
| -Alexey
| 
#7 lore raw

PATCH 1/1 ARC: bump tools to 2014.12 release

Hi Thomas,

Quoted 13 lines · Romain Naour

Hi Alexey,

----- Mail original -----

| De: "Alexey Brodkin" <[email protected]>
| ?: "romain naour" <[email protected]>, "thomas petazzoni" <[email protected]>
| Cc: buildroot at uclibc.org
| Envoy?: Vendredi 13 Mars 2015 10:00:34
| Objet: Re: [Buildroot] [PATCH] ARC: bump tools to 2014.12 release
| 
| On Fri, 2015-03-13 at 09:19 +0100, Romain Naour wrote:
| > Hi Alexey,
| > Can you have a look at this patch:
| > https://patchwork.ozlabs.org/patch/449686/

Do you think the patch Romain mentioned may fix
http://autobuild.buildroot.org/results/d62/d62f472d70f60281dfe94cffc46a3e0566fd4a3d ?

-Alexey

#8 lore raw

PATCH 1/1 ARC: bump tools to 2014.12 release

Dear Alexey Brodkin,

Quoted 15 lines · Alexey Brodkin
Quoted 13 lines · Romain Naour

Hi Alexey,

----- Mail original -----

| De: "Alexey Brodkin" <[email protected]>
| ?: "romain naour" <[email protected]>, "thomas petazzoni" <[email protected]>
| Cc: buildroot at uclibc.org
| Envoy?: Vendredi 13 Mars 2015 10:00:34
| Objet: Re: [Buildroot] [PATCH] ARC: bump tools to 2014.12 release
| 
| On Fri, 2015-03-13 at 09:19 +0100, Romain Naour wrote:
| > Hi Alexey,
| > Can you have a look at this patch:
| > https://patchwork.ozlabs.org/patch/449686/
Quoted 2 lines

Do you think the patch Romain mentioned may fix
http://autobuild.buildroot.org/results/d62/d62f472d70f60281dfe94cffc46a3e0566fd4a3d ?

Yes, most likely it will fix this problem.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
#9 lore raw

PATCH 1/1 ARC: bump tools to 2014.12 release

Hi Thomas,

Quoted 19 lines · Thomas Petazzoni

Dear Alexey Brodkin,

Quoted 15 lines · Alexey Brodkin
Quoted 13 lines · Romain Naour

Hi Alexey,

----- Mail original -----

| De: "Alexey Brodkin" <[email protected]>
| ?: "romain naour" <[email protected]>, "thomas petazzoni" <[email protected]>
| Cc: buildroot at uclibc.org
| Envoy?: Vendredi 13 Mars 2015 10:00:34
| Objet: Re: [Buildroot] [PATCH] ARC: bump tools to 2014.12 release
| 
| On Fri, 2015-03-13 at 09:19 +0100, Romain Naour wrote:
| > Hi Alexey,
| > Can you have a look at this patch:
| > https://patchwork.ozlabs.org/patch/449686/
Quoted 4 lines
Quoted 2 lines

Do you think the patch Romain mentioned may fix
http://autobuild.buildroot.org/results/d62/d62f472d70f60281dfe94cffc46a3e0566fd4a3d ?

Yes, most likely it will fix this problem.

Are you going to apply it or we'll need to find somebody to try it first?

-Alexey

#10 lore raw

PATCH 1/1 ARC: bump tools to 2014.12 release

Hi Alexey,

Le 24/03/2015 17:03, Alexey Brodkin a ?crit :

Quoted 22 lines

Hi Thomas,

Quoted 19 lines · Thomas Petazzoni

Dear Alexey Brodkin,

Quoted 15 lines · Alexey Brodkin
Quoted 13 lines · Romain Naour

Hi Alexey,

----- Mail original -----

| De: "Alexey Brodkin" <[email protected]>
| ?: "romain naour" <[email protected]>, "thomas petazzoni" <[email protected]>
| Cc: buildroot at uclibc.org
| Envoy?: Vendredi 13 Mars 2015 10:00:34
| Objet: Re: [Buildroot] [PATCH] ARC: bump tools to 2014.12 release
| 
| On Fri, 2015-03-13 at 09:19 +0100, Romain Naour wrote:
| > Hi Alexey,
| > Can you have a look at this patch:
| > https://patchwork.ozlabs.org/patch/449686/
Quoted 8 lines
Quoted 4 lines
Quoted 2 lines

Do you think the patch Romain mentioned may fix
http://autobuild.buildroot.org/results/d62/d62f472d70f60281dfe94cffc46a3e0566fd4a3d ?

Yes, most likely it will fix this problem.

Are you going to apply it or we'll need to find somebody to try it first?

I'm back on this issue and I found a way to reproduce it.

As you can see in the build.log, libiberty.a is located in host/usr/lib/libiberty.a But on my machine libiberty.a is located in host/usr/lib64/libiberty.a

This is really weird because autobuilders are also 64bits machine (x86_64-unknown-linux-gnu) and there is no reason that libiberty.a is not in the same place...
So, I moved libiberty.a from host/usr/lib64/libiberty.a to host/usr/lib/libiberty.a and now I have the very same build failure.

When gdb is build with shared libraries, it try to link with it own libiberty.a (host-gdb-7.8.2/bfd/../libiberty/pic -liberty) but it take the one from host/usr/lib/libiberty.a because -LPath/to/host/usr/lib/libiberty.a is before -LPath/to/build/host-gdb-7.8.2/bfd/../libiberty/pic.

See the complete gcc line (verbose):
/bin/sh ./libtool --tag=CC --mode=link /usr/bin/gcc -W -Wall
-Wstrict-prototypes -Wmissing-prototypes -Wshadow -O2 -I/home/naourr/git/buildroot/test/gdb/host/usr/include -release `cat
libtool-soversion` -L/home/naourr/git/buildroot/test/gdb/host/lib -L/home/naourr/git/buildroot/test/gdb/host/usr/lib -Wl,-rpath,/home/naourr/git/buildroot/test/gdb/host/usr/lib -o libbfd.la -rpath /home/naourr/git/buildroot/test/gdb/host/usr/x86_64-unknown-linux-gnu/arm-buildroot-linux-uclibcgnueabi/lib archive.lo archures.lo bfd.lo bfdio.lo bfdwin.lo cache.lo coffgen.lo corefile.lo
format.lo init.lo libbfd.lo opncls.lo reloc.lo section.lo syms.lo targets.lo
hash.lo linker.lo srec.lo binary.lo tekhex.lo ihex.lo stabs.lo stab-syms.lo
merge.lo dwarf2.lo simple.lo compress.lo verilog.lo cat ofiles -L/home/naourr/git/buildroot/test/gdb/build/host-gdb-7.8.2/bfd/../libiberty/pic
-liberty -Wl,-lc,--as-needed,-lm,--no-as-needed -lz

This line come from gdb/bfd/Makefile.in: l1239:
libbfd.la: $(libbfd_la_OBJECTS) $(libbfd_la_DEPENDENCIES)

	$(libbfd_la_LINK) $(am_libbfd_la_rpath) $(libbfd_la_OBJECTS)

$(libbfd_la_LIBADD) $(LIBS)

One solution is to disable shared libraries (preferred).

The second is to tweak LDFLAGS in gdb.mk like this:

HOST_GDB_CONF_OPTS = \
	[...]
	LDFLAGS="-L$(@D)/bfd/../libiberty/pic $(HOST_LDFLAGS)"

I don't see a better fix for gdb.

Best regards,
Romain

#11 lore raw

PATCH 1/1 ARC: bump tools to 2014.12 release

Hi Alexey,

Le 05/04/2015 23:53, Romain Naour a ?crit :

Quoted 36 lines

Hi Alexey,

Le 24/03/2015 17:03, Alexey Brodkin a ?crit :

Quoted 22 lines

Hi Thomas,

Quoted 19 lines · Thomas Petazzoni

Dear Alexey Brodkin,

Quoted 15 lines · Alexey Brodkin
Quoted 13 lines · Romain Naour

Hi Alexey,

----- Mail original -----

| De: "Alexey Brodkin" <[email protected]>
| ?: "romain naour" <[email protected]>, "thomas petazzoni" <[email protected]>
| Cc: buildroot at uclibc.org
| Envoy?: Vendredi 13 Mars 2015 10:00:34
| Objet: Re: [Buildroot] [PATCH] ARC: bump tools to 2014.12 release
| 
| On Fri, 2015-03-13 at 09:19 +0100, Romain Naour wrote:
| > Hi Alexey,
| > Can you have a look at this patch:
| > https://patchwork.ozlabs.org/patch/449686/
Quoted 8 lines
Quoted 4 lines
Quoted 2 lines

Do you think the patch Romain mentioned may fix
http://autobuild.buildroot.org/results/d62/d62f472d70f60281dfe94cffc46a3e0566fd4a3d ?

Yes, most likely it will fix this problem.

Are you going to apply it or we'll need to find somebody to try it first?

I'm back on this issue and I found a way to reproduce it.

Quoted 8 lines

As you can see in the build.log, libiberty.a is located in host/usr/lib/libiberty.a But on my machine libiberty.a is located in host/usr/lib64/libiberty.a

This is really weird because autobuilders are also 64bits machine (x86_64-unknown-linux-gnu) and there is no reason that libiberty.a is not in the same place...
So, I moved libiberty.a from host/usr/lib64/libiberty.a to host/usr/lib/libiberty.a and now I have the very same build failure.

This issue reminds me the issue related to config.site import while cross-compiling...
See:
http://git.buildroot.net/buildroot/commit/?id=f8d4fe376cf65cc4e41423804869a79c2570f674

I think the CONFIG_SITE=/dev/null setting is not taken into account somewhere (binutils or gcc) and the /usr/share/config.site file is sourced. So the libdir value is changed lib64 and that is why libiberty.a is installed to host/usr/lib64.

I'm able to reproduce easily the issue on a debian 64bits and not on a Fedora 21 64bits.

Best regards,
Romain

Quoted 42 lines

When gdb is build with shared libraries, it try to link with it own libiberty.a (host-gdb-7.8.2/bfd/../libiberty/pic -liberty) but it take the one from host/usr/lib/libiberty.a because -LPath/to/host/usr/lib/libiberty.a is before -LPath/to/build/host-gdb-7.8.2/bfd/../libiberty/pic.

See the complete gcc line (verbose):
/bin/sh ./libtool --tag=CC --mode=link /usr/bin/gcc -W -Wall

(quoted diff) +0 -2
-Wstrict-prototypes -Wmissing-prototypes -Wshadow -O2
-I/home/naourr/git/buildroot/test/gdb/host/usr/include  -release `cat

libtool-soversion` -L/home/naourr/git/buildroot/test/gdb/host/lib

(quoted diff) +0 -2
-L/home/naourr/git/buildroot/test/gdb/host/usr/lib
-Wl,-rpath,/home/naourr/git/buildroot/test/gdb/host/usr/lib -o libbfd.la -rpath

/home/naourr/git/buildroot/test/gdb/host/usr/x86_64-unknown-linux-gnu/arm-buildroot-linux-uclibcgnueabi/lib archive.lo archures.lo bfd.lo bfdio.lo bfdwin.lo cache.lo coffgen.lo corefile.lo
format.lo init.lo libbfd.lo opncls.lo reloc.lo section.lo syms.lo targets.lo
hash.lo linker.lo srec.lo binary.lo tekhex.lo ihex.lo stabs.lo stab-syms.lo
merge.lo dwarf2.lo simple.lo compress.lo verilog.lo cat ofiles

(quoted diff) +0 -2
-L/home/naourr/git/buildroot/test/gdb/build/host-gdb-7.8.2/bfd/../libiberty/pic
-liberty -Wl,-lc,--as-needed,-lm,--no-as-needed  -lz

This line come from gdb/bfd/Makefile.in: l1239:
libbfd.la: $(libbfd_la_OBJECTS) $(libbfd_la_DEPENDENCIES)

	$(libbfd_la_LINK) $(am_libbfd_la_rpath) $(libbfd_la_OBJECTS)

$(libbfd_la_LIBADD) $(LIBS)

One solution is to disable shared libraries (preferred).

The second is to tweak LDFLAGS in gdb.mk like this:

HOST_GDB_CONF_OPTS = \
	[...]
	LDFLAGS="-L$(@D)/bfd/../libiberty/pic $(HOST_LDFLAGS)"

I don't see a better fix for gdb.

Best regards,
Romain
_______________________________________________
buildroot mailing list
buildroot at busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot