On Sat, 29 Apr 2017, at 10:30 PM, Tanu Kaskinen wrote: > On Sat, 2017-04-29 at 17:42 +0200, Takashi Iwai wrote: > > On Sat, 29 Apr 2017 14:45:41 +0200, > > Arun Raghavan wrote: > > > > > > (looping in Takashi for his thoughts, since we'd discussed this a long > > > time ago) > > > > > > On Fri, 28 Apr 2017, at 12:38 AM, Tanu Kaskinen wrote: > > > > On Wed, 2017-04-26 at 14:19 +0530, Arun Raghavan wrote: > > > > > On Mon, 24 Apr 2017, at 08:36 PM, Tanu Kaskinen wrote: > > > > > > Previously, if front:x didn't work, we would try to use hw:x for analog > > > > > > stereo output. There's no guarantee that hw:x is an analog output, > > > > > > however. For example, the Intel HDMI LPE driver uses hw:x for HDMI > > > > > > output, and PulseAudio incorrectly created analog profiles for that > > > > > > card, because front:x doesn't work but hw:x does. > > > > > > > > > > > > This patch changes things so that the analog stereo mapping doesn't any > > > > > > more use hw:x as a fallback. A separate "unknown stereo" fallback > > > > > > mapping is added to handle the rare case where hw:x is the only PCM > > > > > > device that works. > > > > > > > > > > > > BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=100488 > > > > > > --- > > > > > > > > > > I recall that front: actually adds a softvol on top of hw: (in some > > > > > cases?), and we actually wanted to move to hw: always. > > > > > > > > Using hw: always seems like a bad idea (for the reasons described in > > > > this patch). > > > > > > > > > Do we know if that's still true? > > > > > > > > At least I don't know. > > > > The softvol is skipped by passing SND_PCM_NO_SOFTVOL bit flag at > > opening a stream even if softvol is included in the configuration. > > IIRC, this was introduced exactly for PA :) > > I looked at some old discussions[1][2] to find out why we're still not > using the NO_SOFTVOL flag. The problem seems to be that using the flag > isn't enough to guarantee that there aren't any softvol elements in the > mixer. It seems that if any program ever has used "front:" without the > NO_SOFTVOL flag, the softvol PCM element will be created and will > remain forever on that system. If the mixer element has been created > once, then starting to use NO_SOFTVOL causes a regression, because > PulseAudio will see and use the PCM mixer element, but the mixer > element won't be hooked to the actual softvol plugin. That is, the > mixer element won't have any effect on the volume. > > In [1] Takashi says that PulseAudio can check if the PCM element is > user-created, but in [2] Jaroslav says that it's not possible. > > I didn't see anyone suggesting using "hw:" instead of "front:", and I > don't see how that would help anyway. Arun, do you think this patch is > ok to be applied? Yes, please go ahead. Would be nice to put out a call for testing on this h/w with the first RC release notes. -- Arun
