All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/1] Fix for lttng-ust not compiling on beagleboard
@ 2011-08-23 18:36 Saul Wold
  2011-08-23 18:36 ` [PATCH 1/1] beagleboard.conf: set DEFAULT_TUNE to cortexa8 Saul Wold
  0 siblings, 1 reply; 6+ messages in thread
From: Saul Wold @ 2011-08-23 18:36 UTC (permalink / raw)
  To: yocto

Thanks to Darren for helping with this one.

Sau!


The following changes since commit b2266beeb357bae938830f559845f5f3deb4f916:

  usermanual: The git fetcher defaults to the git protocol (or file) (2011-08-23 10:00:35 -0700)

are available in the git repository at:
  git://git.yoctoproject.org/poky-contrib sgw/fix
  http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=sgw/fix

Saul Wold (1):
  beagleboard.conf: set DEFAULT_TUNE to cortexa8

 meta-yocto/conf/machine/beagleboard.conf |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

-- 
1.7.6



^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 1/1] beagleboard.conf: set DEFAULT_TUNE to cortexa8
  2011-08-23 18:36 [PATCH 0/1] Fix for lttng-ust not compiling on beagleboard Saul Wold
@ 2011-08-23 18:36 ` Saul Wold
  2011-08-23 19:06   ` Darren Hart
  2011-08-23 19:35   ` Tom Rini
  0 siblings, 2 replies; 6+ messages in thread
From: Saul Wold @ 2011-08-23 18:36 UTC (permalink / raw)
  To: yocto

    [YOCTO #1381]

    lttng-ust generates an ICE when building for armv7, so change it to armv5
    | vfprintf.c:956:1: error: unrecognizable insn:
    | (insn 3968 3967 3969 145 (set (subreg:SI (reg/v:DI 160 [ _umax ]) 0)
    |         (sign_extend:SI (mem:QI (plus:SI (mult:SI (reg/v:SI 166 [ nextarg ])
    |                         (const_int 8 [0x8]))
    |                     (reg/f:SI 370 [ argtable.7 ])) [0 *D.6937_569+0 S1
    A32]))) vfprintf.c:555 -1
    |      (nil))
    | vfprintf.c:956:1: internal compiler error: in extract_insn, at recog.c:2109

Signed-off-by: Saul Wold <sgw@linux.intel.com>
---
 meta-yocto/conf/machine/beagleboard.conf |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/meta-yocto/conf/machine/beagleboard.conf b/meta-yocto/conf/machine/beagleboard.conf
index 0b3cebc..9ed8c59 100644
--- a/meta-yocto/conf/machine/beagleboard.conf
+++ b/meta-yocto/conf/machine/beagleboard.conf
@@ -18,6 +18,8 @@ MACHINE_EXTRA_RRECOMMENDS += "beagleboard-audio"
 # Allow for MMC booting (required by the NAND-less Beagleboard XM)
 EXTRA_IMAGEDEPENDS += "u-boot x-load"
 
+DEFAULT_TUNE = "cortexa8"
+
 include conf/machine/include/tune-cortexa8.inc
 
 IMAGE_FSTYPES += "tar.bz2 jffs2"
-- 
1.7.6



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/1] beagleboard.conf: set DEFAULT_TUNE to cortexa8
  2011-08-23 18:36 ` [PATCH 1/1] beagleboard.conf: set DEFAULT_TUNE to cortexa8 Saul Wold
@ 2011-08-23 19:06   ` Darren Hart
  2011-08-23 19:35   ` Tom Rini
  1 sibling, 0 replies; 6+ messages in thread
From: Darren Hart @ 2011-08-23 19:06 UTC (permalink / raw)
  To: Saul Wold; +Cc: yocto



On 08/23/2011 11:36 AM, Saul Wold wrote:
>     [YOCTO #1381]
> 
>     lttng-ust generates an ICE when building for armv7, so change it to armv5
>     | vfprintf.c:956:1: error: unrecognizable insn:
>     | (insn 3968 3967 3969 145 (set (subreg:SI (reg/v:DI 160 [ _umax ]) 0)
>     |         (sign_extend:SI (mem:QI (plus:SI (mult:SI (reg/v:SI 166 [ nextarg ])
>     |                         (const_int 8 [0x8]))
>     |                     (reg/f:SI 370 [ argtable.7 ])) [0 *D.6937_569+0 S1
>     A32]))) vfprintf.c:555 -1
>     |      (nil))
>     | vfprintf.c:956:1: internal compiler error: in extract_insn, at recog.c:2109
> 
> Signed-off-by: Saul Wold <sgw@linux.intel.com>
> ---
>  meta-yocto/conf/machine/beagleboard.conf |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/meta-yocto/conf/machine/beagleboard.conf b/meta-yocto/conf/machine/beagleboard.conf
> index 0b3cebc..9ed8c59 100644
> --- a/meta-yocto/conf/machine/beagleboard.conf
> +++ b/meta-yocto/conf/machine/beagleboard.conf
> @@ -18,6 +18,8 @@ MACHINE_EXTRA_RRECOMMENDS += "beagleboard-audio"
>  # Allow for MMC booting (required by the NAND-less Beagleboard XM)
>  EXTRA_IMAGEDEPENDS += "u-boot x-load"
>  
> +DEFAULT_TUNE = "cortexa8"
> +

Sorry, I didn't mean for this to be a final solution, just a means to
determine the problem. Removing neon eliminates the vector floating
point, which we do not want to do. The correct fix for this is probably
in gcc. I'd rather not build lttng for beagleboard than change the tune
default to DEFAULT_TUNE to "cortexa8".

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/1] beagleboard.conf: set DEFAULT_TUNE to cortexa8
  2011-08-23 18:36 ` [PATCH 1/1] beagleboard.conf: set DEFAULT_TUNE to cortexa8 Saul Wold
  2011-08-23 19:06   ` Darren Hart
@ 2011-08-23 19:35   ` Tom Rini
  2011-08-23 21:26     ` Saul Wold
  1 sibling, 1 reply; 6+ messages in thread
From: Tom Rini @ 2011-08-23 19:35 UTC (permalink / raw)
  To: Saul Wold; +Cc: yocto

On Tue, Aug 23, 2011 at 1:36 PM, Saul Wold <sgw@linux.intel.com> wrote:
>    [YOCTO #1381]
>
>    lttng-ust generates an ICE when building for armv7, so change it to armv5
>    | vfprintf.c:956:1: error: unrecognizable insn:
>    | (insn 3968 3967 3969 145 (set (subreg:SI (reg/v:DI 160 [ _umax ]) 0)
>    |         (sign_extend:SI (mem:QI (plus:SI (mult:SI (reg/v:SI 166 [ nextarg ])
>    |                         (const_int 8 [0x8]))
>    |                     (reg/f:SI 370 [ argtable.7 ])) [0 *D.6937_569+0 S1
>    A32]))) vfprintf.c:555 -1
>    |      (nil))
>    | vfprintf.c:956:1: internal compiler error: in extract_insn, at recog.c:2109

Which version of lttng-ust is this exactly?  In addition to what
Darren is saying about how this should be papered-over at the recipe
level until gcc itself is fixed...

-- 
Tom


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/1] beagleboard.conf: set DEFAULT_TUNE to cortexa8
  2011-08-23 19:35   ` Tom Rini
@ 2011-08-23 21:26     ` Saul Wold
  2011-08-23 21:57       ` Martin Jansa
  0 siblings, 1 reply; 6+ messages in thread
From: Saul Wold @ 2011-08-23 21:26 UTC (permalink / raw)
  To: Tom Rini; +Cc: yocto

On 08/23/2011 12:35 PM, Tom Rini wrote:
> On Tue, Aug 23, 2011 at 1:36 PM, Saul Wold<sgw@linux.intel.com>  wrote:
>>     [YOCTO #1381]
>>
>>     lttng-ust generates an ICE when building for armv7, so change it to armv5
>>     | vfprintf.c:956:1: error: unrecognizable insn:
>>     | (insn 3968 3967 3969 145 (set (subreg:SI (reg/v:DI 160 [ _umax ]) 0)
>>     |         (sign_extend:SI (mem:QI (plus:SI (mult:SI (reg/v:SI 166 [ nextarg ])
>>     |                         (const_int 8 [0x8]))
>>     |                     (reg/f:SI 370 [ argtable.7 ])) [0 *D.6937_569+0 S1
>>     A32]))) vfprintf.c:555 -1
>>     |      (nil))
>>     | vfprintf.c:956:1: internal compiler error: in extract_insn, at recog.c:2109
>
> Which version of lttng-ust is this exactly?  In addition to what
> Darren is saying about how this should be papered-over at the recipe
> level until gcc itself is fixed...
>
Sorry, this was my mis-understanding of Darren's comment.

Version of lttng-ust is 0.15.

Initially I tried to do something with tcmode-default, setting lttng-ust 
similar to meta-xlib which also had a ICE issue.

TARGET_CC_ARCH_arm_pn-mesa-xlib := 
"${@'${TARGET_CC_ARCH}'.replace('armv7-a','armv5')}"

That caused the compiler to complain about a mode issue, probably due to 
mixing of tune parameters:

| arm-poky-linux-gnueabi-libtool: compile:  ccache 
arm-poky-linux-gnueabi-gcc
-march=armv5 -fno-tree-vectorize -mthumb-interwork -mfloat-abi=softfp
-mfpu=neon -mtune=cortex-a8
--sysroot=/vol/1/sgw/autobuilder/yab/yocto-slave/external/build/build/tmp/sysroots/beagleboard
-DHAVE_CONFIG_H -I. -I.. -I../include/ust -I../include -I../libustcomm
-DUST_COMPONENT=libust -fno-strict-aliasing -Wall -pipe -g
-feliminate-unused-debug-types -MT libust_la-marker-control.lo -MD -MP -MF
.deps/libust_la-marker-control.Tpo -c marker-control.c -o
libust_la-marker-control.o >/dev/null 2>&1
| mv -f .deps/libust_la-tracercore.Tpo .deps/libust_la-tracercore.Plo
| {standard input}: Assembler messages:
| {standard input}:190: Error: selected processor does not support ARM 
mode `dmb'

Yes, we are trying to solve Compiler issue, when I first started working 
on the last week there was not a compiler bug filed, but now there seems 
to be one!

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43137

Which looks like it is resolved also!

I will move this to Nitin to see if he can incorporate the patch into gcc.

Sau!



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/1] beagleboard.conf: set DEFAULT_TUNE to cortexa8
  2011-08-23 21:26     ` Saul Wold
@ 2011-08-23 21:57       ` Martin Jansa
  0 siblings, 0 replies; 6+ messages in thread
From: Martin Jansa @ 2011-08-23 21:57 UTC (permalink / raw)
  To: Saul Wold; +Cc: yocto

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

On Tue, Aug 23, 2011 at 02:26:06PM -0700, Saul Wold wrote:
> On 08/23/2011 12:35 PM, Tom Rini wrote:
> > On Tue, Aug 23, 2011 at 1:36 PM, Saul Wold<sgw@linux.intel.com>  wrote:
> >>     [YOCTO #1381]
> >>
> >>     lttng-ust generates an ICE when building for armv7, so change it to armv5
> >>     | vfprintf.c:956:1: error: unrecognizable insn:
> >>     | (insn 3968 3967 3969 145 (set (subreg:SI (reg/v:DI 160 [ _umax ]) 0)
> >>     |         (sign_extend:SI (mem:QI (plus:SI (mult:SI (reg/v:SI 166 [ nextarg ])
> >>     |                         (const_int 8 [0x8]))
> >>     |                     (reg/f:SI 370 [ argtable.7 ])) [0 *D.6937_569+0 S1
> >>     A32]))) vfprintf.c:555 -1
> >>     |      (nil))
> >>     | vfprintf.c:956:1: internal compiler error: in extract_insn, at recog.c:2109
> >
> > Which version of lttng-ust is this exactly?  In addition to what
> > Darren is saying about how this should be papered-over at the recipe
> > level until gcc itself is fixed...
> >
> Sorry, this was my mis-understanding of Darren's comment.
> 
> Version of lttng-ust is 0.15.
> 
> Initially I tried to do something with tcmode-default, setting lttng-ust 
> similar to meta-xlib which also had a ICE issue.
> 
> TARGET_CC_ARCH_arm_pn-mesa-xlib := 
> "${@'${TARGET_CC_ARCH}'.replace('armv7-a','armv5')}"
> 
> That caused the compiler to complain about a mode issue, probably due to 
> mixing of tune parameters:
> 
> | arm-poky-linux-gnueabi-libtool: compile:  ccache 
> arm-poky-linux-gnueabi-gcc
> -march=armv5 -fno-tree-vectorize -mthumb-interwork -mfloat-abi=softfp
> -mfpu=neon -mtune=cortex-a8
> --sysroot=/vol/1/sgw/autobuilder/yab/yocto-slave/external/build/build/tmp/sysroots/beagleboard
> -DHAVE_CONFIG_H -I. -I.. -I../include/ust -I../include -I../libustcomm
> -DUST_COMPONENT=libust -fno-strict-aliasing -Wall -pipe -g
> -feliminate-unused-debug-types -MT libust_la-marker-control.lo -MD -MP -MF
> .deps/libust_la-marker-control.Tpo -c marker-control.c -o
> libust_la-marker-control.o >/dev/null 2>&1
> | mv -f .deps/libust_la-tracercore.Tpo .deps/libust_la-tracercore.Plo
> | {standard input}: Assembler messages:
> | {standard input}:190: Error: selected processor does not support ARM 
> mode `dmb'
> 
> Yes, we are trying to solve Compiler issue, when I first started working 
> on the last week there was not a compiler bug filed, but now there seems 
> to be one!
> 
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43137
> 
> Which looks like it is resolved also!

This is better bug report:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50099

Regards,

> I will move this to Nitin to see if he can incorporate the patch into gcc.
> 
> Sau!
> 
> _______________________________________________
> yocto mailing list
> yocto@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/yocto

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2011-08-23 21:58 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-08-23 18:36 [PATCH 0/1] Fix for lttng-ust not compiling on beagleboard Saul Wold
2011-08-23 18:36 ` [PATCH 1/1] beagleboard.conf: set DEFAULT_TUNE to cortexa8 Saul Wold
2011-08-23 19:06   ` Darren Hart
2011-08-23 19:35   ` Tom Rini
2011-08-23 21:26     ` Saul Wold
2011-08-23 21:57       ` Martin Jansa

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.