alsa-lib potentially racy on ARMv8

Bug #1088526 reported by Riku Voipio
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Linaro AArch64 cross-distro work
Fix Released
Low
Riku Voipio

Bug Description

Alsa-lib is a low level library used for sound output. While it compiles on ARMv8, there might be some major problems run-time:

../../include/iatomic.h:1109:2: warning: #warning "Atomic operations are not supported on this architecture." [-Wcpp]

Looking at iatomic.h, without arch-specific atomics, alsa-lib will have empty rmb() amd wmb() macros and similar naive atomic_add/atomic_set macros, meaning that alsa will be badly racy on ARMv8.

Fixing this should be average amount of work, but validating it is tricky as fast model has no sound output.

Revision history for this message
Riku Voipio (riku-voipio) wrote :

I can't set the "importance" of bugs in linaro-aarch64 project, so recording here to not to forget: this bug doesn't block anyone, but must be fixed before any distribution releases. Medium importance, perhaps.

Changed in linaro-aarch64:
importance: Undecided → Medium
Revision history for this message
Riku Voipio (riku-voipio) wrote :

I've submitted a patch around this:

http://mailman.alsa-project.org/pipermail/alsa-devel/2014-February/072176.html

It turns out most of these atomics arent' really used, unless you use the pcm meter plugin (that allows you to make meters and scopes out alsa streams).

Changed in linaro-aarch64:
status: New → In Progress
importance: Medium → Low
Revision history for this message
Riku Voipio (riku-voipio) wrote :
Changed in linaro-aarch64:
assignee: nobody → Riku Voipio (riku-voipio)
status: In Progress → Fix Committed
Revision history for this message
Riku Voipio (riku-voipio) wrote :

Released in alsa-lib 1.0.28

Changed in linaro-aarch64:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.