Thread (16 messages) 16 messages, 5 authors, 2021-06-28

Re: [OE-core] RFC: pulseaudio vs. meson vs. neon

From: Andrea Adami <hidden>
Date: 2021-06-28 12:45:12

On Mon, Jun 28, 2021 at 2:36 PM Alexander Kanavin
[off-list ref] wrote:
What I am saying is, can you discuss this with upstream first please?
Sorry, no, I'm not in the position to do that.
Besides, I am dispatching my armv4/armv5te Zaurus hardware so I will
not be able to test anymore.

My point here is, for this pulseaudio 14.x release (in Honister) we
could do an effort and patch the damn file.
I write this because this is the only failing recipe for
core-image-sato for armv4 and this arch is still supported in oe-core
(thank you so much Khem).

So I'd say as long as armv4 and armv5t are supported we need to patch
the recipe.

With next pulseaudio 15.x the older arm will maybe lose support, I
don't know now.

My 2 cents
A.A.


Alex

On Mon, 28 Jun 2021 at 14:08, Andrea Adami [off-list ref] wrote:
quoted
On Mon, Jun 28, 2021 at 1:00 PM Alexander Kanavin
[off-list ref] wrote:
quoted
Seems like an upstream problem rather than a Yocto one?
Indeed,

from
https://www.freedesktop.org/wiki/Software/PulseAudio/Notes/14.0/
"
One difference from Autotools is that the --disable-neon-opt option
for disabling NEON optimizations doesn't have a counterpart in Meson
due to how Meson's NEON detection works. NEON will always be enabled
if the compiler supports it, but even if the target machine doesn't
support NEON, that shouldn't be a problem, because PulseAudio disables
NEON at run-time if the processor doesn't support it.
"
Upstream *thinks* this is not an issue because all will be fine at runtime :)

Cheers
A.A:
quoted
Alex

On Mon, 28 Jun 2021 at 12:02, Andrea Adami [off-list ref] wrote:
quoted
Hello,

during a core-image-sato build for armv4 I got one single build
failure with pulseaudio.

This happens after commit 80bbea0 pulseaudio: switch build system from
Autotools to Meson. In the commit text it is evidenced:

"
With Meson there's no option for disabling Neon optimizations if the
compiler supports Neon. If -mfpu is set in CC or CFLAGS, it's ignored,
because the build system adds -mfpu=neon at the end of the compiler
command line, overriding any earlier -mfpu options. This shouldn't be
a problem, because PulseAudio detects at runtime whether the CPU
supports Neon instructions.
"

At the moment to build pulseaudio for armv4/5 I have reverted to using
autotools.


There was around a patch but has not been merged.
https://lists.openembedded.org/g/openembedded-core/message/146375

Maybe a v2 will be accepted?

Cheers
A.A.

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help