Make digital mic on the AMD Renoir machines work under gnome desktop

Bug #1889217 reported by Hui Wang
14
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OEM Priority Project
Fix Released
Undecided
Unassigned
alsa-lib (Ubuntu)
Fix Released
High
Hui Wang
Focal
Fix Released
High
Hui Wang
Groovy
Fix Released
High
Hui Wang
alsa-ucm-conf (Ubuntu)
Fix Released
High
Hui Wang
Focal
Fix Released
High
Hui Wang
Groovy
Fix Released
High
Hui Wang
pulseaudio (Ubuntu)
Fix Released
High
Hui Wang
Focal
Fix Released
High
Hui Wang
Groovy
Fix Released
High
Hui Wang

Bug Description

[Impact]
On the LENOVO AMD Renoir machines, there is a digital mic directly
connected to the APU instead of the codec, so there are two separate
sound cards in the system, one is for analogue codec driven by hda
driver, the other is for the dmic driven by ASoC acp driver.

The current audio stack (pulseaudio + alsa-ucm-conf + alsa-lib) doesn't
support this design yet, it could support all audio devices on the codec
well, but it doesn't support that dmic well. In the gnome-control-center,
the dmic becomes two input devices: analog input and multichannel input,
and users can only record sound from analog input, the multichannel input
can't function at all. Besides this issue, there is another issue, after
users plug an external mic, the external mic can't replace the dmic
automatically, this gives users a bad experience since this behaviour is
different from the other audio designs.

[Fix for pulseaudio]
backport 3 patches:
2 of them from upstream
device-port: queue CARD CHANGE event before update default sink
alsa: adjust ucm sink/source priority according to ports priority
1 of them from a merge request:
https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/merge_requests/305

[Fix for alsa-ucm-conf]
backport 14 patches:
11 of them from upstream:
ucm-fix-wrong-If-in-sequence-in-HiFi-dual.conf.patch
ucm2-add-initial-ucm.conf-for-the-latest-alsa-lib.patch
sof-hda-dsp-don-t-fail-if-Auto-Mute-control-is-not-p.patch
ucm.conf-add-support-for-the-kernel-module-name-tree.patch
sof-hda-dsp-make-Headphone-Playback-Switch-condition.patch
sof-soundwire-initial-UCM2-version.patch
sof-soundwire-cleanups-recommended-by-the-ucm-valida.patch
sof-soundwire-rewrite-for-syntax-3.patch
HDA-Intel-add-support-for-AMD-acp-microphone-devices.patch
Fix-invalid-Regex-Type-in-various-Condition-blocks.patch
hda-hdmi-add-HDMI4-HDMI5-HDMI6-devices.patch
3 of them from a merge request:
https://github.com/alsa-project/alsa-ucm-conf/pull/41

[Fix for alsa-lib]
backport 47 patches from upstream:
Enabled-extended-namehints-in-alsa.conf.patch
conf-add-snd_config_is_array-function.patch
topology-use-snd_config_is_array-function.patch
ucm-merge-the-array-items-from-the-condition-blocks.patch
ucm-parse-SectionOnce-section-in-the-master-UCM-conf.patch
ucm-execute-SectionDefaults-lately-when-the-first-ve.patch
ucm-handle-set-_once-command.patch
ucm-handle-set-_defaults-command.patch
ucm-initialize-mgr-once_list.patch
ucm-fix-SectionOnce-comment.patch
ucm-fix-compilation-error-in-set_defaults_user.patch
ucm-rename-SectionOnce-to-BootSequence.patch
ucm-rename-_once-command-to-_boot-command.patch
ucm-configuration-implement-in-place-Include.patch
ucm-configuration-substitute-ConfDir-and-ConfTopDir.patch
ucm-config-substitute-File-string-to-allow-variables.patch
ucm-configuration-allow-to-define-the-configuration-.patch
ucm-configuration-add-DefineRegex.patch
ucm-substitute-arguments-in-sequences.patch
ucm-allow-syntax-version-3.patch
ucm-config-change-the-in-place-include-evaluation-or.patch
ucm-allow-to-specify-the-toplevel-directory-using-as.patch
ucm-substitute-also-value-strings.patch
ucm-handle-strict-prefix-correctly-for-the-UCM-card-.patch
ucm-String-condition-implement-Empty.patch
ucm-Define-DefineRegex-is-supported-in-Syntax-3.patch
ucm-substitute-OpenName.patch
ucm-substitute-CardNumber.patch
ucm-implement-the-toplevel-ucm-configuration-file-pa.patch
ucm-substitute-device-modifier-names-too.patch
ucm-substitute-device-strings-in-the-device-lists.patch
ucm-substitute-component-sequence-string.patch
ucm-substitute-verb-name-and-file-field.patch
ucm-substitute-Comment-in-Transition-and-Device.patch
ucm-substitute-RenameDevice-and-DeleteDevice-lists.patch
ucm-substitute-arguments-in-sequences-only-for-synta.patch
ucm-shuffle-code-in-compound_merge.patch
ucm-implement-CardIdByName-substitution.patch
ucm-allow-to-ignore-errors-for-the-value-substitutio.patch
ucm-allow-to-use-the-defined-variables-in-the-substi.patch
ucm-implement-CardNumberByName-substitution.patch
ucm-fix-the-possible-buffer-overflow-substitution.patch
ucm-simplify-get_by_card-in-parser.c.patch
ucm-implement-AlwaysTrue-Condition.Type.patch
ucm-Allow-empty-strings-in-var-.-substitutions.patch
ucm-substitution-remove-duplicate-allow_empty-assign.patch
ucm-fix-parse_get_safe_name-safe-name-must-be-checke.patch
ucm-substitute-the-merged-tree-completely.patch
add Breaks alsa-ucm-conf (<= 1.2.2-1ubuntu0.1) in the d/control
add snd_config_is_array@ALSA_0.9 1.2.2-2.1ubuntu1 in the d/libasound2.symbols

[Test Case]
On the AMD Renoir machines:
* Boot the system with these updated packages (already backported kernel
drivers to Focal and oem-5.6 kernels), open the gnome-control-center
 - we could see the digital mic in the input device tab, and only one
input device
 - we could record sound through this dmic
 - we could play sound through the speaker
 - plug a headset, we could see the output device become headphones and the input device become headset mic
   - we could play sound to headphone and record sound from headset mic
 - plug an HDMI monitor, there is HDMI audio choice
   - play sound to HDMI audio, we could hear the sound from the speaker on the monitor
 - Suspend and resume, redo the previous steps, all audio devices still work well.

On other machines:
Boot the system with all updated packages or partial updated packages,
check all input devices and output devices, they worked the same as
before. this SRU doesn't bring any impact on other machines.

[Regression Risk]
For machines without using ucm, like traditional PC and Laptops with
HDA audio, it is possible that the pulseaudio can't parse the audio
devices well that there is only a dummy audio device on the
gnome-control-center, but the possibility is very low since this SRU
didn't touch the non-ucm part, and we already tested on a Dell HDA
audio machine, the audio function worked as well as before.

For machines using ucm, the regression possibility is the pulseaudio
fails to parse the needed ucm, then there is no dmic or there is only
a dummy audio device in the gnome-contro-center. This possibility is
very low too, We have tested this SRU on Dell and Lenovo dmic machines,
they worked as well as before. And for the LENOVO dual codec machine,
we don't have the hw at the moment, but changes for this machine are
all coming from upstream, and Cert team will verify the -proposed on
this machine since they have the machine. For other ucm machines, the
regression possibility is very low since this SRU doesn't touch their
ucm at all.

This SRU doesn't introduce any external API changes

Hui Wang (hui.wang)
Changed in alsa-lib (Ubuntu):
importance: Undecided → High
Changed in alsa-ucm-conf (Ubuntu):
importance: Undecided → High
Changed in pulseaudio (Ubuntu):
importance: Undecided → High
summary: - Make digital mic on the AMD renoir machines work under gnome environment
+ Make digital mic on the AMD renoir machines work under gnome desktop
Changed in alsa-lib (Ubuntu Focal):
importance: Undecided → High
Changed in alsa-ucm-conf (Ubuntu Focal):
importance: Undecided → High
Changed in pulseaudio (Ubuntu Focal):
importance: Undecided → High
tags: added: renoir
tags: added: focal groovy
Hui Wang (hui.wang)
description: updated
Changed in pulseaudio (Ubuntu Focal):
assignee: nobody → Hui Wang (hui.wang)
Changed in alsa-ucm-conf (Ubuntu Focal):
assignee: nobody → Hui Wang (hui.wang)
Changed in alsa-ucm-conf (Ubuntu Groovy):
assignee: nobody → Hui Wang (hui.wang)
Changed in alsa-lib (Ubuntu Focal):
assignee: nobody → Hui Wang (hui.wang)
Changed in alsa-lib (Ubuntu Groovy):
assignee: nobody → Hui Wang (hui.wang)
Hui Wang (hui.wang)
description: updated
Hui Wang (hui.wang)
description: updated
Hui Wang (hui.wang)
description: updated
description: updated
Revision history for this message
Sebastien Bacher (seb128) wrote :

@Hui, thanks for the work. That's going to be a complicated SRU, just a small comment, the 'regression potential' section is supposed to describe what sort of problem could happen following the change, it's meant to give an hint of what to focus on in the testing

see https://wiki.ubuntu.com/StableReleaseUpdates#SRU_Bug_Template for detailed explanations

'low' (and the rest of the section focussing on why it's low) is sort of missing the point, could you rework it? I expect in that case we would need to test also on non AMD config that they are not regressing?

Revision history for this message
Hui Wang (hui.wang) wrote :

@Seb,

Ok, we are going to do the large scale test for this SRU, and after the test, I will rewrite the [Regression Risk].

Once the debdiff is uploaded, it is ready to review.

thx.

Hui Wang (hui.wang)
description: updated
description: updated
summary: - Make digital mic on the AMD renoir machines work under gnome desktop
+ Make digital mic on the AMD Renoir machines work under gnome desktop
description: updated
description: updated
Revision history for this message
Hui Wang (hui.wang) wrote :

@seb,

I plan to upload the debdiff for focal, but how to handle the groovy? we will definitely upgrade those packages to higher version in the groovy sooner or later, do I need to upload debdiff for groovy as well?

tags: added: originate-from-1874593 sutton
Revision history for this message
Sebastien Bacher (seb128) wrote :

@Hui, we will get the new version but unsure when so if it's not too much extra work it would be b etter to generate debdiffs for G

Revision history for this message
Hui Wang (hui.wang) wrote :

OK, got it, will upload for G.
thx.

Revision history for this message
Hui Wang (hui.wang) wrote :

This is the alsa-lib debdiff for Groovy.

thx.

Revision history for this message
Hui Wang (hui.wang) wrote :

This is alsa-ucm-conf debdiff for Groovy.

thx.

Revision history for this message
Hui Wang (hui.wang) wrote :

This is the pulseaudio debdiff for Groovy.

thx.

Hui Wang (hui.wang)
description: updated
Revision history for this message
Hui Wang (hui.wang) wrote :

This is alsa-lib debdiff for Focal.

thx.

Revision history for this message
Hui Wang (hui.wang) wrote :

This is alsa-ucm-conf debdiff for Focal.

thx.

Revision history for this message
Hui Wang (hui.wang) wrote :

This is pulseaudio debdiff for Focal.

thx.

Changed in alsa-lib (Ubuntu Groovy):
status: New → Fix Committed
Changed in alsa-ucm-conf (Ubuntu Groovy):
status: New → Fix Committed
Changed in pulseaudio (Ubuntu Groovy):
status: New → Fix Committed
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks, I've uploaded the changes to groovy now. It would be nice having some confirmation it works there before SRUing to focal. One point for improvement also, the changelog have no human description of the changes which is suboptimal, especially in case of a SRU

Revision history for this message
Sebastien Bacher (seb128) wrote :

I've also commited to the pulseaudio Vcs, the alsa packages don't have a Vcs

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package alsa-ucm-conf - 1.2.2-1ubuntu2

---------------
alsa-ucm-conf (1.2.2-1ubuntu2) groovy; urgency=medium

  * d/p/0003-ucm-fix-wrong-If-in-sequence-in-HiFi-dual.conf.patch
  * d/p/0004-ucm2-add-initial-ucm.conf-for-the-latest-alsa-lib.patch
  * d/p/0005-sof-hda-dsp-don-t-fail-if-Auto-Mute-control-is-not-p.patch
  * d/p/0006-ucm.conf-add-support-for-the-kernel-module-name-tree.patch
  * d/p/0007-sof-hda-dsp-make-Headphone-Playback-Switch-condition.patch
  * d/p/0008-sof-soundwire-initial-UCM2-version.patch
  * d/p/0009-sof-soundwire-cleanups-recommended-by-the-ucm-valida.patch
  * d/p/0010-sof-soundwire-rewrite-for-syntax-3.patch
  * d/p/0011-HDA-Intel-add-support-for-AMD-acp-microphone-devices.patch
  * d/p/0012-hda-hdmi-add-HDMI4-HDMI5-HDMI6-devices.patch
  * d/p/0013-amd-renoir-acp-use-the-machine-driver-s-name-for-top.patch
  * d/p/0014-amd-renoir-acp-add-syntax-in-the-Linked.patch
  * d/p/0015-HDA-Intel-only-bind-the-acp-dmic-to-the-soundcard-wi.patch
  * d/p/0016-Fix-invalid-Regex-Type-in-various-Condition-blocks.patch
    Make digital mic on the AMD Renoir machines work under gnome desktop
    (LP: #1889217)

 -- Hui Wang <email address hidden> Thu, 13 Aug 2020 16:18:58 +0200

Changed in alsa-ucm-conf (Ubuntu Groovy):
status: Fix Committed → Fix Released
Revision history for this message
Hui Wang (hui.wang) wrote :

Thanks Seb.

And I verified that these 3 groovy packages worked very well on 4 machines I have.

Install the groovy daily built image on the lenovo amd renoir laptop, enable the -proposed, install the pulseaudio_13.99.1-1ubuntu9, alsa-lib_1.2.2-2.3ubuntu2 and alsa-ucm-conf_1.2.2-1ubuntu2, reboot. open the gnome-sound-setting, there is digital mic in the input device list, and It could record the sound. run suspend and resume, recheck the digital mic, it still could record the sound. check speaker and hdmi audio, all worked as before, plug a headset, sound could output to headphone and I could use headset-mic to record the sound too.

Then I did the same test on a Dell dmic machine, a lenovo I+N hda audio machine and a dell intel hda audio machine, the audio all worked as well as before. Didn't find any regression on the new packages.

thx.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package pulseaudio - 1:13.99.1-1ubuntu9

---------------
pulseaudio (1:13.99.1-1ubuntu9) groovy; urgency=medium

  * d/p/0034-alsa-adjust-ucm-sink-source-priority-according-to-po.patch
  * d/p/0035-ucm-add-possibility-to-skip-the-UCM-card-completely-.patch
  * d/p/0036-device-port-queue-CARD-CHANGE-event-before-update-de.patch
    Make digital mic on the AMD Renoir machines work under gnome desktop
    (LP: #1889217)

 -- Hui Wang <email address hidden> Sat, 08 Aug 2020 13:32:49 +0800

Changed in pulseaudio (Ubuntu Groovy):
status: Fix Committed → Fix Released
Revision history for this message
Hui Wang (hui.wang) wrote :

@Seb,

Pushed the changes to pulseaudio Vcs ubuntu-focal branch. Is there anything need to change before uploading to focal building queue?

thx.

Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (3.3 KiB)

This bug was fixed in the package alsa-lib - 1.2.2-2.3ubuntu2

---------------
alsa-lib (1.2.2-2.3ubuntu2) groovy; urgency=medium

  * d/p/0001-conf-add-snd_config_is_array-function.patch
  * d/p/0001-Enabled-extended-namehints-in-alsa.conf.patch
  * d/p/0002-topology-use-snd_config_is_array-function.patch
  * d/p/0003-ucm-merge-the-array-items-from-the-condition-blocks.patch
  * d/p/0004-ucm-parse-SectionOnce-section-in-the-master-UCM-conf.patch
  * d/p/0005-ucm-execute-SectionDefaults-lately-when-the-first-ve.patch
  * d/p/0006-ucm-handle-set-_once-command.patch
  * d/p/0007-ucm-handle-set-_defaults-command.patch
  * d/p/0008-ucm-initialize-mgr-once_list.patch
  * d/p/0009-ucm-fix-SectionOnce-comment.patch
  * d/p/0010-ucm-fix-compilation-error-in-set_defaults_user.patch
  * d/p/0011-ucm-rename-SectionOnce-to-BootSequence.patch
  * d/p/0012-ucm-rename-_once-command-to-_boot-command.patch
  * d/p/0013-ucm-configuration-implement-in-place-Include.patch
  * d/p/0014-ucm-configuration-substitute-ConfDir-and-ConfTopDir.patch
  * d/p/0015-ucm-config-substitute-File-string-to-allow-variables.patch
  * d/p/0016-ucm-configuration-allow-to-define-the-configuration-.patch
  * d/p/0017-ucm-configuration-add-DefineRegex.patch
  * d/p/0018-ucm-substitute-arguments-in-sequences.patch
  * d/p/0019-ucm-allow-syntax-version-3.patch
  * d/p/0020-ucm-config-change-the-in-place-include-evaluation-or.patch
  * d/p/0021-ucm-allow-to-specify-the-toplevel-directory-using-as.patch
  * d/p/0022-ucm-substitute-also-value-strings.patch
  * d/p/0023-ucm-handle-strict-prefix-correctly-for-the-UCM-card-.patch
  * d/p/0024-ucm-String-condition-implement-Empty.patch
  * d/p/0025-ucm-Define-DefineRegex-is-supported-in-Syntax-3.patch
  * d/p/0026-ucm-substitute-OpenName.patch
  * d/p/0027-ucm-substitute-CardNumber.patch
  * d/p/0028-ucm-implement-the-toplevel-ucm-configuration-file-pa.patch
  * d/p/0029-ucm-substitute-device-modifier-names-too.patch
  * d/p/0030-ucm-substitute-device-strings-in-the-device-lists.patch
  * d/p/0031-ucm-substitute-component-sequence-string.patch
  * d/p/0032-ucm-substitute-verb-name-and-file-field.patch
  * d/p/0033-ucm-substitute-Comment-in-Transition-and-Device.patch
  * d/p/0034-ucm-substitute-RenameDevice-and-DeleteDevice-lists.patch
  * d/p/0035-ucm-substitute-arguments-in-sequences-only-for-synta.patch
  * d/p/0036-ucm-shuffle-code-in-compound_merge.patch
  * d/p/0037-ucm-implement-CardIdByName-substitution.patch
  * d/p/0038-ucm-allow-to-ignore-errors-for-the-value-substitutio.patch
  * d/p/0039-ucm-allow-to-use-the-defined-variables-in-the-substi.patch
  * d/p/0040-ucm-implement-CardNumberByName-substitution.patch
  * d/p/0041-ucm-fix-the-possible-buffer-overflow-substitution.patch
  * d/p/0042-ucm-simplify-get_by_card-in-parser.c.patch
  * d/p/0043-ucm-implement-AlwaysTrue-Condition.Type.patch
  * d/p/0044-ucm-Allow-empty-strings-in-var-.-substitutions.patch
  * d/p/0045-ucm-substitution-remove-duplicate-allow_empty-assign.patch
  * d/p/0046-ucm-fix-parse_get_safe_name-safe-name-must-be-checke.patch
  * d/p/0047-ucm-substitute-the-merged-tree-completely.patch
  * add Breaks alsa-ucm-conf (<= 1.2.2-1ubuntu1) in the d/control
  * add snd_...

Read more...

Changed in alsa-lib (Ubuntu Groovy):
status: Fix Committed → Fix Released
Changed in alsa-lib (Ubuntu Focal):
status: New → Fix Committed
Changed in alsa-ucm-conf (Ubuntu Focal):
status: New → Fix Committed
Changed in pulseaudio (Ubuntu Focal):
status: New → Fix Committed
Revision history for this message
Łukasz Zemczak (sil2100) wrote : Please test proposed package

Hello Hui, or anyone else affected,

Accepted alsa-lib into focal-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/alsa-lib/1.2.2-2.1ubuntu2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-focal to verification-done-focal. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-focal. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

tags: added: verification-needed verification-needed-focal
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Hello Hui, or anyone else affected,

Accepted alsa-ucm-conf into focal-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/alsa-ucm-conf/1.2.2-1ubuntu0.2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-focal to verification-done-focal. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-focal. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Hello Hui, or anyone else affected,

Accepted pulseaudio into focal-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/pulseaudio/1:13.99.1-1ubuntu3.6 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-focal to verification-done-focal. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-focal. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Revision history for this message
Ubuntu SRU Bot (ubuntu-sru-bot) wrote : Autopkgtest regression report (alsa-lib/1.2.2-2.1ubuntu2)

All autopkgtests for the newly accepted alsa-lib (1.2.2-2.1ubuntu2) for focal have finished running.
The following regressions have been reported in tests triggered by the package:

asterisk/1:16.2.1~dfsg-2ubuntu1 (armhf)
gsequencer/3.1.3-1 (s390x)

Please visit the excuses page listed below and investigate the failures, proceeding afterwards as per the StableReleaseUpdates policy regarding autopkgtest regressions [1].

https://people.canonical.com/~ubuntu-archive/proposed-migration/focal/update_excuses.html#alsa-lib

[1] https://wiki.ubuntu.com/StableReleaseUpdates#Autopkgtest_Regressions

Thank you!

Revision history for this message
Hui Wang (hui.wang) wrote :

For the gsequencer/3.1.3-1 (s390x):

The last step fails, but it has nothing to do with alsalib, this is the log without regression reported:
ags-integration-unit-test PASS
ags-integration-functional-test FLAKY non-zero exit status 2
Creating nova instance adt-focal-s390x-gsequencer-20200820-095922 from image adt/ubuntu-focal-s390x-server-20200820.img (UUID 6a71e79c-0324-4927-a49e-11d2b937df21)...

This is the log with regression reported:
ags-integration-unit-test PASS
ags-integration-functional-test FLAKY non-zero exit status 2
Creating nova instance adt-focal-s390x-gsequencer-20200820-100757 from image adt/ubuntu-focal-s390x-server-20200820.img (UUID 6a71e79c-0324-4927-a49e-11d2b937df21)...
Unexpected error:
Traceback (most recent call last):
  File "/home/ubuntu/autopkgtest/lib/VirtSubproc.py", line 739, in mainloop
    command()
  File "/home/ubuntu/autopkgtest/lib/VirtSubproc.py", line 668, in command
    r = f(c, ce)
  File "/home/ubuntu/autopkgtest/lib/VirtSubproc.py", line 116, in cmd_close
    cleanup()
  File "/home/ubuntu/autopkgtest/lib/VirtSubproc.py", line 694, in cleanup
    caller.hook_cleanup()
  File "/home/ubuntu/autopkgtest/virt/autopkgtest-virt-ssh", line 500, in hook_cleanup
    execute_setup_script('cleanup')
  File "/home/ubuntu/autopkgtest/virt/autopkgtest-virt-ssh", line 205, in execute_setup_script
    stderr=subprocess.PIPE)
  File "/home/ubuntu/autopkgtest/lib/VirtSubproc.py", line 144, in execute_timeout
    (out, err) = sp.communicate(instr)
  File "/usr/lib/python3.5/subprocess.py", line 1072, in communicate
    stdout, stderr = self._communicate(input, endtime, timeout)
  File "/usr/lib/python3.5/subprocess.py", line 1715, in _communicate
    ready = selector.select(timeout)
  File "/usr/lib/python3.5/selectors.py", line 376, in select
    fd_event_list = self._poll.poll(timeout)
  File "/home/ubuntu/autopkgtest/lib/VirtSubproc.py", line 64, in alarm_handler
    raise Timeout()
VirtSubproc.Timeout
autopkgtest [11:15:29]: ERROR: autopkgtest
: error cleaning up:

autopkgtest [11:15:29]: ERROR: testbed failure: cannot send to testbed: [Errno 32] Broken pipe

It is an error on creating nova instance, it has nothing to do with alsa-lib.

Revision history for this message
Shengyao Xue (xueshengyao) wrote :

 -proposed package tested OK

installed a latest Ubuntu 20.04 image on a Lenovo ThinkPad T14s(AMD Renoir) and installed the packages libasound2(1.2.2-2.1ubuntu2) and pulseaudio(1.13.99.1-1ubuntu3.6) from focal-proposed.

the audio functions works fine.

tags: added: verification-done-focal
removed: verification-needed verification-needed-focal
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (3.5 KiB)

This bug was fixed in the package alsa-lib - 1.2.2-2.1ubuntu2

---------------
alsa-lib (1.2.2-2.1ubuntu2) focal; urgency=medium

  * d/p/0001-conf-add-snd_config_is_array-function.patch
  * d/p/0001-Enabled-extended-namehints-in-alsa.conf.patch
  * d/p/0002-topology-use-snd_config_is_array-function.patch
  * d/p/0003-ucm-merge-the-array-items-from-the-condition-blocks.patch
  * d/p/0004-ucm-parse-SectionOnce-section-in-the-master-UCM-conf.patch
  * d/p/0005-ucm-execute-SectionDefaults-lately-when-the-first-ve.patch
  * d/p/0006-ucm-handle-set-_once-command.patch
  * d/p/0007-ucm-handle-set-_defaults-command.patch
  * d/p/0008-ucm-initialize-mgr-once_list.patch
  * d/p/0009-ucm-fix-SectionOnce-comment.patch
  * d/p/0010-ucm-fix-compilation-error-in-set_defaults_user.patch
  * d/p/0011-ucm-rename-SectionOnce-to-BootSequence.patch
  * d/p/0012-ucm-rename-_once-command-to-_boot-command.patch
  * d/p/0013-ucm-configuration-implement-in-place-Include.patch
  * d/p/0014-ucm-configuration-substitute-ConfDir-and-ConfTopDir.patch
  * d/p/0015-ucm-config-substitute-File-string-to-allow-variables.patch
  * d/p/0016-ucm-configuration-allow-to-define-the-configuration-.patch
  * d/p/0017-ucm-configuration-add-DefineRegex.patch
  * d/p/0018-ucm-substitute-arguments-in-sequences.patch
  * d/p/0019-ucm-allow-syntax-version-3.patch
  * d/p/0020-ucm-config-change-the-in-place-include-evaluation-or.patch
  * d/p/0021-ucm-allow-to-specify-the-toplevel-directory-using-as.patch
  * d/p/0022-ucm-substitute-also-value-strings.patch
  * d/p/0023-ucm-handle-strict-prefix-correctly-for-the-UCM-card-.patch
  * d/p/0024-ucm-String-condition-implement-Empty.patch
  * d/p/0025-ucm-Define-DefineRegex-is-supported-in-Syntax-3.patch
  * d/p/0026-ucm-substitute-OpenName.patch
  * d/p/0027-ucm-substitute-CardNumber.patch
  * d/p/0028-ucm-implement-the-toplevel-ucm-configuration-file-pa.patch
  * d/p/0029-ucm-substitute-device-modifier-names-too.patch
  * d/p/0030-ucm-substitute-device-strings-in-the-device-lists.patch
  * d/p/0031-ucm-substitute-component-sequence-string.patch
  * d/p/0032-ucm-substitute-verb-name-and-file-field.patch
  * d/p/0033-ucm-substitute-Comment-in-Transition-and-Device.patch
  * d/p/0034-ucm-substitute-RenameDevice-and-DeleteDevice-lists.patch
  * d/p/0035-ucm-substitute-arguments-in-sequences-only-for-synta.patch
  * d/p/0036-ucm-shuffle-code-in-compound_merge.patch
  * d/p/0037-ucm-implement-CardIdByName-substitution.patch
  * d/p/0038-ucm-allow-to-ignore-errors-for-the-value-substitutio.patch
  * d/p/0039-ucm-allow-to-use-the-defined-variables-in-the-substi.patch
  * d/p/0040-ucm-implement-CardNumberByName-substitution.patch
  * d/p/0041-ucm-fix-the-possible-buffer-overflow-substitution.patch
  * d/p/0042-ucm-simplify-get_by_card-in-parser.c.patch
  * d/p/0043-ucm-implement-AlwaysTrue-Condition.Type.patch
  * d/p/0044-ucm-Allow-empty-strings-in-var-.-substitutions.patch
  * d/p/0045-ucm-substitution-remove-duplicate-allow_empty-assign.patch
  * d/p/0046-ucm-fix-parse_get_safe_name-safe-name-must-be-checke.patch
  * d/p/0047-ucm-substitute-the-merged-tree-completely.patch
  * add Breaks alsa-ucm-conf (<= 1.2.2-1ubuntu0.1) in the d/control
  * add snd...

Read more...

Changed in alsa-lib (Ubuntu Focal):
status: Fix Committed → Fix Released
Revision history for this message
Łukasz Zemczak (sil2100) wrote : Update Released

The verification of the Stable Release Update for alsa-lib has completed successfully and the package is now being released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package alsa-ucm-conf - 1.2.2-1ubuntu0.2

---------------
alsa-ucm-conf (1.2.2-1ubuntu0.2) focal; urgency=medium

  * d/p/0003-ucm-fix-wrong-If-in-sequence-in-HiFi-dual.conf.patch
  * d/p/0004-ucm2-add-initial-ucm.conf-for-the-latest-alsa-lib.patch
  * d/p/0005-sof-hda-dsp-don-t-fail-if-Auto-Mute-control-is-not-p.patch
  * d/p/0006-ucm.conf-add-support-for-the-kernel-module-name-tree.patch
  * d/p/0007-sof-hda-dsp-make-Headphone-Playback-Switch-condition.patch
  * d/p/0008-sof-soundwire-initial-UCM2-version.patch
  * d/p/0009-sof-soundwire-cleanups-recommended-by-the-ucm-valida.patch
  * d/p/0010-sof-soundwire-rewrite-for-syntax-3.patch
  * d/p/0011-HDA-Intel-add-support-for-AMD-acp-microphone-devices.patch
  * d/p/0012-hda-hdmi-add-HDMI4-HDMI5-HDMI6-devices.patch
  * d/p/0013-amd-renoir-acp-use-the-machine-driver-s-name-for-top.patch
  * d/p/0014-amd-renoir-acp-add-syntax-in-the-Linked.patch
  * d/p/0015-HDA-Intel-only-bind-the-acp-dmic-to-the-soundcard-wi.patch
  * d/p/0016-Fix-invalid-Regex-Type-in-various-Condition-blocks.patch
    enable sound on AMD Renoir machines (LP: #1889217)

  [ Kai-Heng Feng ]
  * d/p/0017-Add-support-for-Lenovo-ThinkStation-P620-Main-Audio.patch
    Add proper stream and jack assignment to Lenovo ThinkStation P620
    (LP: #1891461)

 -- Hui Wang <email address hidden> Mon, 10 Aug 2020 15:05:02 +0800

Changed in alsa-ucm-conf (Ubuntu Focal):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package pulseaudio - 1:13.99.1-1ubuntu3.6

---------------
pulseaudio (1:13.99.1-1ubuntu3.6) focal; urgency=medium

  [ Kai-Heng Feng ]
  * d/p/0001-alsa-mixer-Recognize-USB-audio-jack-mixer.patch:
    - recognize USB jack mixer to support jack detection (LP: #1891461)
  * d/p/0002-module-alsa-card-Set-a-minimum-profile-priority-if-i.patch:
    - resolve an issue when headset gets unplugged, HDMI audio is choosen
      instead of internal speaker (LP: #1891461)

  [ Hui Wang ]
  * d/p/0034-alsa-adjust-ucm-sink-source-priority-according-to-po.patch
  * d/p/0035-ucm-add-possibility-to-skip-the-UCM-card-completely-.patch
  * d/p/0036-device-port-queue-CARD-CHANGE-event-before-update-de.patch
    - Make the AMD acp sound card skipped in the PA, then the ucm could
      link it to other sound cards, it is easy for gnome to support AMD
      Renoir sound driver after this change. (LP: #1889217)

 -- Kai-Heng Feng <email address hidden> Thu, 13 Aug 2020 18:53:08 +0800

Changed in pulseaudio (Ubuntu Focal):
status: Fix Committed → Fix Released
Revision history for this message
Shengyao Xue (xueshengyao) wrote :

I tested the packages below on 2 ThinkPads(one is Intel Only, another is Intel + Nvidia), the sound functions works fine.

- libasound2, version 1.2.2-2.1ubuntu2
- libasound2-data, version 1.2.2-2.1ubuntu2
- alsa-ucm-conf, version 1.2.2-1ubuntu0.2
- pulseaudio, version 1:13.99.1-1ubuntu3.6
- pulseaudio-module-bluetooth, version 1:13.99.1-1ubuntu3.6
- pulseaudio-utils, version 1:13.99.1-1ubuntu3.6

Changed in oem-priority:
status: New → Fix Released
tags: added: oem-priority
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.