Hello community, here is the log from the commit of package alsa for openSUSE:Factory checked in at 2017-05-06 18:25:00 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/alsa (Old) and /work/SRC/openSUSE:Factory/.alsa.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "alsa" Sat May 6 18:25:00 2017 rev:175 rq:492619 version:1.1.3 Changes: -------- --- /work/SRC/openSUSE:Factory/alsa/alsa.changes 2017-04-17 10:22:34.106017794 +0200 +++ /work/SRC/openSUSE:Factory/.alsa.new/alsa.changes 2017-05-06 18:25:02.426265571 +0200 @@ -1,0 +2,7 @@ +Wed May 3 10:07:17 CEST 2017 - [email protected] + +- Remove direct_memory_access flag from dmix definition again for + fixing a regression used with old alsa-lib binary (boo#1037021): + 0096-conf-Remove-dmix.direct_memory_access-setup.patch + +------------------------------------------------------------------- New: ---- 0096-conf-Remove-dmix.direct_memory_access-setup.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ alsa.spec ++++++ --- /var/tmp/diff_new_pack.RPlnZy/_old 2017-05-06 18:25:04.437981709 +0200 +++ /var/tmp/diff_new_pack.RPlnZy/_new 2017-05-06 18:25:04.441981144 +0200 @@ -88,6 +88,7 @@ Patch41: 0041-conf-cards-add-VC4-HDMI-card.patch Patch42: 0042-pcm-plug-save-converter-config.patch Patch43: 0043-pcm-file-delegate-htimestamping-to-slave-instead-of-.patch +Patch96: 0096-conf-Remove-dmix.direct_memory_access-setup.patch Patch97: 0097-pcm-dmix-Disable-var_periodsize-as-default.patch # rest suse patches Patch98: 0098-dmix-Workaround-for-binary-incompatibility.patch @@ -196,6 +197,7 @@ %patch41 -p1 %patch42 -p1 %patch43 -p1 +%patch96 -p1 %patch97 -p1 %patch98 -p1 ++++++ 0096-conf-Remove-dmix.direct_memory_access-setup.patch ++++++ >From f3d2f4046e8492013044a99fd4324a21dbb82938 Mon Sep 17 00:00:00 2001 From: Takashi Iwai <[email protected]> Date: Wed, 3 May 2017 09:42:02 +0200 Subject: [PATCH 1/2] conf: Remove dmix.direct_memory_access setup The commit [22eca6468b4a: pcm: dmix: Allow disabling x86 optimizations] introduced the new flag for dmix & co, direct_memory_access. However, it turned out that such an addition of the new flag in the default pcm definition causes an error when it's used with old alsa-lib codes. Although the code added here is correct, per se, and it works as expected, it's not wise to break the configuration with old stuff -- even if the usage is somehow incorrect and should be avoided. Since the usage of the new flag is only for HDMI LPE audio, and the usage of dmix itself should be limited with that hardware, this patch removes the setup so that it works with the old alsa-lib again. We may introduce the dmix behavior change in a smarter way, e.g. passing some flag from the hardware driver so that it works more generically without the manual fiddling of config files. Bugzilla: http://bugzilla.opensuse.org/show_bug.cgi?id=1037021 Signed-off-by: Takashi Iwai <[email protected]> --- src/conf/cards/HdmiLpeAudio.conf | 23 ----------------------- src/conf/pcm/dmix.conf | 15 --------------- 2 files changed, 38 deletions(-) diff --git a/src/conf/cards/HdmiLpeAudio.conf b/src/conf/cards/HdmiLpeAudio.conf index 61bdfeae2917..dae71facac6d 100644 --- a/src/conf/cards/HdmiLpeAudio.conf +++ b/src/conf/cards/HdmiLpeAudio.conf @@ -20,29 +20,6 @@ HdmiLpeAudio.pcm.front.0 { } } -# uncached memory reads have a high penalty -HdmiLpeAudio.dmix.direct_memory_access false - -# default with dmix+softvol -HdmiLpeAudio.pcm.default { - @args [ CARD ] - @args.CARD { - type string - } - type plug - slave.pcm { - type softvol - slave.pcm { - @func concat - strings [ "dmix:" $CARD ] - } - control { - name "PCM Playback Volume" - card $CARD - } - } -} - <confdir:pcm/surround40.conf> <confdir:pcm/surround21.conf> <confdir:pcm/surround41.conf> diff --git a/src/conf/pcm/dmix.conf b/src/conf/pcm/dmix.conf index 2d3b329e533a..7d0aa0158c42 100644 --- a/src/conf/pcm/dmix.conf +++ b/src/conf/pcm/dmix.conf @@ -49,21 +49,6 @@ pcm.!dmix { @func refer name defaults.pcm.ipc_perm } - direct_memory_access { - @func refer - name { - @func concat - strings [ - "cards." - { - @func card_driver - card $CARD - } - ".dmix.direct_memory_access" - ] - } - default true - } slave { pcm { type hw -- 2.12.2
