pulseaudio print lots of error when selecting unavailable profile
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
HWE Next |
Fix Released
|
Critical
|
Unassigned | ||
pulseaudio (Ubuntu) |
Fix Released
|
Critical
|
Unassigned | ||
Xenial |
Confirmed
|
High
|
Unassigned |
Bug Description
SRU Document:
[Impact]
A HDMI audio device usually has several output ports, each port represents a profile in pulseaudio, without this patch, the puseaudio always choose the first profile no matter it is active or not.
[Test Case]
connect each port of HDMI device, and check if the profile of that port is active or not.
[Regression Potential]
The patch will check all ports under each profile, if a profile only contains unavailable ports, this profile will be set to unavailable as well. Without this patch, all profiles are always available, then if a profile includes a unusable hdmi-output, and pulseaudio select this profile to be active (since its priority is the highest), the kernel audio driver will crash.
I think this patch will not introduce regression:
1) It is a correct logic to set a profile to be unavailable if it only contains unavailable ports.
2) pulseaudio-artful and pulseaudio-bionic already include this patch, they work very well
3) I tested this patch on 1 lenovo laptop, 1 lenovo desktop, 1 dell laptop and 1 dell desktop, all worked well as before
4) tested this patch on two dell machines (LOAD5-DVT2-A2 and Dawson-JC-C without analogue audio) which have unusable hdmi-output profile on them, the kernel driver did not crash anymore and audio function worked very well.
[Other Info]
Only pulseaudio-xenial has this problem.
we need to backport this commit to pulseaudio-xenial.
After printing out the error logs, the kernel crashes and system hangs.
Changed in pulseaudio (Ubuntu): | |
importance: | Undecided → Critical |
tags: | added: originate-from-1737727 somerville |
Changed in hwe-next: | |
importance: | Undecided → Critical |
description: | updated |
description: | updated |
tags: |
added: verification-done removed: verification-needed |
Changed in pulseaudio (Ubuntu Xenial): | |
status: | Fix Committed → Fix Released |
Changed in hwe-next: | |
status: | New → Fix Released |
Changed in pulseaudio (Ubuntu Xenial): | |
status: | Fix Released → Confirmed |
Changed in pulseaudio (Ubuntu Xenial): | |
status: | Confirmed → Fix Released |
Changed in pulseaudio (Ubuntu Xenial): | |
importance: | Undecided → High |
If this is a crash then it's very likely we already have a bug report for it.
Can you find a stack trace of the crash?