@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 Desktop Packages, which is subscribed to pulseaudio in Ubuntu. 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 -- Mailing list: https://launchpad.net/~desktop-packages Post to : desktop-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~desktop-packages More help : https://help.launchpad.net/ListHelp
