I am seeing the following when trying to build pulseaudio on a Beagleboard running a current ALIP daily:
... CC libpulsecore_1.0_la-svolume_ arm.lo {standard input}: Assembler messages: {standard input}:82: Error: thumb conditional instruction should be in IT block -- `addcs r0,r8' {standard input}:83: Error: thumb conditional instruction should be in IT block -- `movcs r6,r0' {standard input}:98: Error: thumb conditional instruction should be in IT block -- `addcs r0,r8' {standard input}:99: Error: thumb conditional instruction should be in IT block -- `movcs r6,r0' {standard input}:119: Error: thumb conditional instruction should be in IT block -- `addcs r0,r8' {standard input}:120: Error: thumb conditional instruction should be in IT block -- `movcs r6,r0' ...
I am using the normal pulseaudio build (bootstrap.sh, configure, make). The build worked fine on a Pandaboard with Ubuntu 10.10. It looks like something is not being detected correctly via bootstrap/configure on ALIP.
I have searched and seen commits in archive for adding "-Wa, -mimplicit-it=thumb". I have added this to CFLAGS without success.
Anyone else seen any thing like this? Any thoughts on what else I could try?
Kurt Taylor (irc krtaylor)
On Mon, Feb 07, 2011, Kurt Taylor wrote:
I am seeing the following when trying to build pulseaudio on a Beagleboard running a current ALIP daily: ... CC libpulsecore_1.0_la-svolume_ arm.lo {standard input}: Assembler messages: {standard input}:82: Error: thumb conditional instruction should be in IT block -- `addcs r0,r8' {standard input}:83: Error: thumb conditional instruction should be in IT block -- `movcs r6,r0' {standard input}:98: Error: thumb conditional instruction should be in IT block -- `addcs r0,r8' {standard input}:99: Error: thumb conditional instruction should be in IT block -- `movcs r6,r0' {standard input}:119: Error: thumb conditional instruction should be in IT block -- `addcs r0,r8' {standard input}:120: Error: thumb conditional instruction should be in IT block -- `movcs r6,r0' ...
I am using the normal pulseaudio build (bootstrap.sh, configure, make). The build worked fine on a Pandaboard with Ubuntu 10.10. It looks like something is not being detected correctly via bootstrap/configure on ALIP.
I have searched and seen commits in archive for adding "-Wa, -mimplicit-it=thumb". I have added this to CFLAGS without success.
The package does a regular configure and then: make -C src libpulsecore_0.9.22_la-svolume_arm.lo CFLAGS+=-Wa,-mimplicit-it=thumb on ARM.
This is really ugly though; I guess the asm snippets in src/pulsecore/svolume_arm.c should be ported, this is explained in: https://wiki.ubuntu.com/ARM/Thumb2PortingHowto#Types of Assembly Language
Bye
Hi Kurt,
the problem seems to be in function pa_volume_s16ne_arm.which seems to be arm optimised version of pa_volume_s16ne_c.For the timebeing for your experimentation you can also avoid using arm optimised pa_volume_s16ne_arm.and let pulse audio use pa_volume_s16ne_c.
hope this helps.
Thanks Sachin
On Tue, Feb 8, 2011 at 2:09 AM, Loïc Minier loic.minier@linaro.org wrote:
On Mon, Feb 07, 2011, Kurt Taylor wrote:
I am seeing the following when trying to build pulseaudio on a
Beagleboard
running a current ALIP daily: ... CC libpulsecore_1.0_la-svolume_ arm.lo {standard input}: Assembler messages: {standard input}:82: Error: thumb conditional instruction should be in IT block -- `addcs r0,r8' {standard input}:83: Error: thumb conditional instruction should be in IT block -- `movcs r6,r0' {standard input}:98: Error: thumb conditional instruction should be in IT block -- `addcs r0,r8' {standard input}:99: Error: thumb conditional instruction should be in IT block -- `movcs r6,r0' {standard input}:119: Error: thumb conditional instruction should be in
IT
block -- `addcs r0,r8' {standard input}:120: Error: thumb conditional instruction should be in
IT
block -- `movcs r6,r0' ...
I am using the normal pulseaudio build (bootstrap.sh, configure, make).
The
build worked fine on a Pandaboard with Ubuntu 10.10. It looks like
something
is not being detected correctly via bootstrap/configure on ALIP.
I have searched and seen commits in archive for adding "-Wa, -mimplicit-it=thumb". I have added this to CFLAGS without success.
The package does a regular configure and then: make -C src libpulsecore_0.9.22_la-svolume_arm.lo CFLAGS+=-Wa,-mimplicit-it=thumb on ARM.
This is really ugly though; I guess the asm snippets in src/pulsecore/svolume_arm.c should be ported, this is explained in: https://wiki.ubuntu.com/ARM/Thumb2PortingHowto#Types of Assembly Language
Bye
Loïc Minier
linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev
Thanks Sachin, actually, no, it is from pulsecore/svolume_arm.c, unfortunately arm assembler isn't my specialty (yet).
Ubuntu gets around this by building with -mimplicit-it=thumb. Since I have temporarily given up on the panda (re: 708883), I now have the code working now on beagle using this method, although I agree with Loic that it should be fixed.
Once I get some of the arm buffer cpu wakes experimentation done and published, I'll return to this.
Kurt Taylor (irc krtaylor)
On 9 February 2011 04:00, Sachin Gupta sachin.gupta@linaro.org wrote:
Hi Kurt,
the problem seems to be in function pa_volume_s16ne_arm.which seems to be arm optimised version of pa_volume_s16ne_c.For the timebeing for your experimentation you can also avoid using arm optimised pa_volume_s16ne_arm.and let pulse audio use pa_volume_s16ne_c.
hope this helps.
Thanks Sachin
On Tue, Feb 8, 2011 at 2:09 AM, Loïc Minier loic.minier@linaro.orgwrote:
On Mon, Feb 07, 2011, Kurt Taylor wrote:
I am seeing the following when trying to build pulseaudio on a
Beagleboard
running a current ALIP daily: ... CC libpulsecore_1.0_la-svolume_ arm.lo {standard input}: Assembler messages: {standard input}:82: Error: thumb conditional instruction should be in
IT
block -- `addcs r0,r8' {standard input}:83: Error: thumb conditional instruction should be in
IT
block -- `movcs r6,r0' {standard input}:98: Error: thumb conditional instruction should be in
IT
block -- `addcs r0,r8' {standard input}:99: Error: thumb conditional instruction should be in
IT
block -- `movcs r6,r0' {standard input}:119: Error: thumb conditional instruction should be in
IT
block -- `addcs r0,r8' {standard input}:120: Error: thumb conditional instruction should be in
IT
block -- `movcs r6,r0' ...
I am using the normal pulseaudio build (bootstrap.sh, configure, make).
The
build worked fine on a Pandaboard with Ubuntu 10.10. It looks like
something
is not being detected correctly via bootstrap/configure on ALIP.
I have searched and seen commits in archive for adding "-Wa, -mimplicit-it=thumb". I have added this to CFLAGS without success.
The package does a regular configure and then: make -C src libpulsecore_0.9.22_la-svolume_arm.lo CFLAGS+=-Wa,-mimplicit-it=thumb on ARM.
This is really ugly though; I guess the asm snippets in src/pulsecore/svolume_arm.c should be ported, this is explained in: https://wiki.ubuntu.com/ARM/Thumb2PortingHowto#Types of Assembly Language
Bye
Loïc Minier
linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev
Same function, yes, actually, that is an interesting idea, I will try that.
Kurt Taylor (irc krtaylor)
On 9 February 2011 17:03, Kurt Taylor kurt.taylor@linaro.org wrote:
Thanks Sachin, actually, no, it is from pulsecore/svolume_arm.c, unfortunately arm assembler isn't my specialty (yet).
Ubuntu gets around this by building with -mimplicit-it=thumb. Since I have temporarily given up on the panda (re: 708883), I now have the code working now on beagle using this method, although I agree with Loic that it should be fixed.
Once I get some of the arm buffer cpu wakes experimentation done and published, I'll return to this.
Kurt Taylor (irc krtaylor)
On 9 February 2011 04:00, Sachin Gupta sachin.gupta@linaro.org wrote:
Hi Kurt,
the problem seems to be in function pa_volume_s16ne_arm.which seems to be arm optimised version of pa_volume_s16ne_c.For the timebeing for your experimentation you can also avoid using arm optimised pa_volume_s16ne_arm.and let pulse audio use pa_volume_s16ne_c.
hope this helps.
Thanks Sachin
On Tue, Feb 8, 2011 at 2:09 AM, Loïc Minier loic.minier@linaro.orgwrote:
On Mon, Feb 07, 2011, Kurt Taylor wrote:
I am seeing the following when trying to build pulseaudio on a
Beagleboard
running a current ALIP daily: ... CC libpulsecore_1.0_la-svolume_ arm.lo {standard input}: Assembler messages: {standard input}:82: Error: thumb conditional instruction should be in
IT
block -- `addcs r0,r8' {standard input}:83: Error: thumb conditional instruction should be in
IT
block -- `movcs r6,r0' {standard input}:98: Error: thumb conditional instruction should be in
IT
block -- `addcs r0,r8' {standard input}:99: Error: thumb conditional instruction should be in
IT
block -- `movcs r6,r0' {standard input}:119: Error: thumb conditional instruction should be in
IT
block -- `addcs r0,r8' {standard input}:120: Error: thumb conditional instruction should be in
IT
block -- `movcs r6,r0' ...
I am using the normal pulseaudio build (bootstrap.sh, configure, make).
The
build worked fine on a Pandaboard with Ubuntu 10.10. It looks like
something
is not being detected correctly via bootstrap/configure on ALIP.
I have searched and seen commits in archive for adding "-Wa, -mimplicit-it=thumb". I have added this to CFLAGS without success.
The package does a regular configure and then: make -C src libpulsecore_0.9.22_la-svolume_arm.lo CFLAGS+=-Wa,-mimplicit-it=thumb on ARM.
This is really ugly though; I guess the asm snippets in src/pulsecore/svolume_arm.c should be ported, this is explained in: https://wiki.ubuntu.com/ARM/Thumb2PortingHowto#Types of Assembly Language
Bye
Loïc Minier
linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev
On Mon, Feb 7, 2011 at 9:39 PM, Loïc Minier loic.minier@linaro.org wrote:
On Mon, Feb 07, 2011, Kurt Taylor wrote:
I am seeing the following when trying to build pulseaudio on a Beagleboard running a current ALIP daily: ... CC libpulsecore_1.0_la-svolume_ arm.lo {standard input}: Assembler messages: {standard input}:82: Error: thumb conditional instruction should be in IT block -- `addcs r0,r8' {standard input}:83: Error: thumb conditional instruction should be in IT block -- `movcs r6,r0' {standard input}:98: Error: thumb conditional instruction should be in IT block -- `addcs r0,r8' {standard input}:99: Error: thumb conditional instruction should be in IT block -- `movcs r6,r0' {standard input}:119: Error: thumb conditional instruction should be in IT block -- `addcs r0,r8' {standard input}:120: Error: thumb conditional instruction should be in IT block -- `movcs r6,r0' ...
I am using the normal pulseaudio build (bootstrap.sh, configure, make). The build worked fine on a Pandaboard with Ubuntu 10.10. It looks like something is not being detected correctly via bootstrap/configure on ALIP.
I have searched and seen commits in archive for adding "-Wa, -mimplicit-it=thumb". I have added this to CFLAGS without success.
The package does a regular configure and then: make -C src libpulsecore_0.9.22_la-svolume_arm.lo CFLAGS+=-Wa,-mimplicit-it=thumb on ARM.
This is really ugly though; I guess the asm snippets in src/pulsecore/svolume_arm.c should be ported, this is explained in: https://wiki.ubuntu.com/ARM/Thumb2PortingHowto#Types of Assembly Language
I agree with Loic that we should fix the asm and push that upstream (or maybe see if upstream trunk already has a fix for that). Michael H. and Dave M. would probably be able give some guidance here if the document Loic pointed to does not help.
-
Alexander Sack -
Kurt Taylor -
Loïc Minier -
Sachin Gupta