[Bug 1750947] Re: pulseaudio print lots of error when selecting unavailable profile
Hui Wang
hui.wang at canonical.com
Thu Mar 22 01:29:22 UTC 2018
@Robie,
Updated [Regression Potential] as shown below:
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 choose 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.
** Description changed:
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]
- no regression possibility, since this commit is already in artful and
- bionic.
+ 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.
https://cgit.freedesktop.org/pulseaudio/pulseaudio/commit/?id=a222a07920731f3c4967faccab7469af50b428a4
After printing out the error logs, the kernel crashes and system hangs.
--
You received this bug notification because you are a member of Ubuntu
Sponsors Team, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1750947
Title:
pulseaudio print lots of error when selecting unavailable profile
Status in HWE Next:
New
Status in pulseaudio package in Ubuntu:
Fix Released
Status in pulseaudio source package in Xenial:
New
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.
https://cgit.freedesktop.org/pulseaudio/pulseaudio/commit/?id=a222a07920731f3c4967faccab7469af50b428a4
After printing out the error logs, the kernel crashes and system
hangs.
To manage notifications about this bug go to:
https://bugs.launchpad.net/hwe-next/+bug/1750947/+subscriptions
More information about the Ubuntu-sponsors
mailing list