Bug 315920

Hmm, there is a capture checkbox? Weird. All stream in PA backend should be modelled as "Playback" items (even the capture ones)... wonder when this changed...

Created attachment 77652 [details]
screenshot of kmix and pavucontrol initially

Created attachment 77653 [details]
screenshot of kmix and pavucontrol after unchecking "capture"

It seems as if the capture checkbox simply tries to pull the slider down to 0% to achieve the effect. So maybe this is as simple as finding the place where the request for a certain capture amplification is lost and replaced by garbage.

Hmm, I wonder if "capture" is just a cosmetic thing and it really is a playback control and it's the "mute" ticky box. Not looked at the source for ages so hard to say without poking.

At first glance it looks like it could be some mishandling of the mute functionality at some level. At least it's the capture volume that's doing this and not the output volume :s

Created attachment 77726 [details]
fix the bug: code was referencing incorrect Volume object

I prepared the commit against KDE/4.10. Let me know and I'll commit to 4.10 and master.

While the patch seems sensible, I'm still confused as to why things were changed....

As I mentioned above, all streams and devices were originally represented by playback objects, and capture stuff was never used. This was originally because the "capture" stuff had a built in premise that only one device could be marked as capture at any given time and they had no mute button.

I want to be able to mute my Mic or even just a single stream. I would personally favour finding whatever commit changed the capture devices and streams to use capture volumes and revert it. This should both fix this issue and also give the mute buttons back.

Of course if others feel differently, then this patch looks sensible enough.

Created attachment 77979 [details]
Alternative fix for the issue.

It seems the capture volume was done in 5c57272a along with a comment to check with me if it was correct.... it isn't :) 

Sorry if I missed you asking me about that Christian :(

Anyway the applied patch appears to work also and return the ability to mute my mic. Can someone else confirm that this is OK and that there are no other unexpected side effects (i.e. if there was a valid reason to switch to capture volumes that I've missed).

(In reply to comment #8)
> Anyway the applied patch appears to work also and return the ability to mute
> my mic.

s/applied patch/attached patch/

Yes I have the same problem. I tried audacity to record but whatever the level change is, it does not effect real microphone recording level.

I used alsamixer to fix it. It seems it is maximizing the `Mic Boost`.

Pulseaudio will control all the underlying ALSA mixer elements then add it's own software amplification as requested to get the desired level. The fact it's controlling Mic Boost is simply a detail resulting from PA controlling all the mixer elements.

I'm pretty sure my patch is correct, but I'd like Christian or Matthias' feedback first!

I just tested your patch. It solves my issue. Also in my patch the "mute" 
button didn't work. In yours it does. The button just has the wrong icon. 
Maybe that was the reason to switch to a record control? In any case, a fix 
for this issue is way more important than the correct icon.

Created attachment 78055 [details]
Screenshot of capture "icon"

Hmm, what icon do you see?

For me capture "icon" is really broken... it's not even an icon, but a ticky box. The "Mute" icon on the other hand is just a speaker icon (which I guess is wrong, but I've never really thought about it before until you said it!!)

Anyway, I think this is OK to push so I'll do that now and CP it to the 4.10 branch.

Git commit 64afb1660b808ea8f1ba3d8d54a5db1cfe5a2acd by Colin Guthrie.
Committed on 12/03/2013 at 13:17.
Pushed by cguthrie into branch 'master'.

pulse: Revert the use of capture volumes added in 5c57272a.

Using capture controls does not give proper mute handling.
This is why all streams in the PA backend are treated as 'playback'
even if they are technically capture streams/devices.

This fixes two problems:
 1. You cannot mute mics or recording streams
 2. Capture volume is set to crazy high values

M  +2    -6    backends/mixer_pulse.cpp

http://commits.kde.org/kmix/64afb1660b808ea8f1ba3d8d54a5db1cfe5a2acd

Git commit 22f9ea6a1d6dbf530c8593af7b2331a83c990e37 by Colin Guthrie.
Committed on 12/03/2013 at 13:17.
Pushed by cguthrie into branch 'KDE/4.10'.

pulse: Revert the use of capture volumes added in 5c57272a.

Using capture controls does not give proper mute handling.
This is why all streams in the PA backend are treated as 'playback'
even if they are technically capture streams/devices.

This fixes two problems:
 1. You cannot mute mics or recording streams
 2. Capture volume is set to crazy high values
(cherry picked from commit 64afb1660b808ea8f1ba3d8d54a5db1cfe5a2acd)

M  +2    -6    backends/mixer_pulse.cpp

http://commits.kde.org/kmix/22f9ea6a1d6dbf530c8593af7b2331a83c990e37

On Thursday 14 March 2013 10:20:45 you wrote:
> For me capture "icon" is really broken... it's not even an icon, but a ticky
> box. The "Mute" icon on the other hand is just a speaker icon (which I
> guess is wrong, but I've never really thought about it before until you
> said it!!)

True, the capture checkbox is not really better. Maybe the icon could be a red 
circle or microphone for capture?
 
> Anyway, I think this is OK to push so I'll do that now and CP it to the 4.10
> branch.

Yes. Thanks!

By the way, currently I was not using the PulseAudio, but was having the same problem with it, too. Will the problem also solve this issue or the patch is only related to pulseaudio?

Because when I install pulseaudio, I cannot even talk via Skype, Hangout or SIP programs.

(In reply to comment #17)
> By the way, currently I was not using the PulseAudio, but was having the
> same problem with it, too. Will the problem also solve this issue or the
> patch is only related to pulseaudio?

This patch only relates to the PulseAudio backend in Kmix.

> Because when I install pulseaudio, I cannot even talk via Skype, Hangout or
> SIP programs.

Something is likely busted with your setup as I happily use Skypes and Google+ hangouts quite frequently. While I've not used it for a while, Empathy with SIP also works great for me.

(In reply to comment #18)
> (In reply to comment #17)
> > Because when I install pulseaudio, I cannot even talk via Skype, Hangout or
> > SIP programs.
> 
> Something is likely busted with your setup as I happily use Skypes and
> Google+ hangouts quite frequently. While I've not used it for a while,
> Empathy with SIP also works great for me.

Maybe it is not OK to write here, but could you attach your PulseAudio conf file here, maybe I could solve my PulseAudio problem, too?

(In reply to comment #19)
> (In reply to comment #18)
> > (In reply to comment #17)
> > > Because when I install pulseaudio, I cannot even talk via Skype, Hangout or
> > > SIP programs.
> > 
> > Something is likely busted with your setup as I happily use Skypes and
> > Google+ hangouts quite frequently. While I've not used it for a while,
> > Empathy with SIP also works great for me.
> 
> Maybe it is not OK to write here, but could you attach your PulseAudio conf
> file here, maybe I could solve my PulseAudio problem, too?

This is really something to take up with your distro. As I'm a PulseAudio developer and maintain PulseAudio in Mageia distribution, I can only really offer support if that's the setup you use. I'm not familiar with how other distributions intend the setup to work, so my advice may not apply. My setup however is basically just what we do Out of the Box on Mageia. Nothing special involved.

*** Bug 316957 has been marked as a duplicate of this bug. ***

*** Bug 317002 has been marked as a duplicate of this bug. ***

*** Bug 317024 has been marked as a duplicate of this bug. ***