Re: [pulseaudio-discuss] Odd app behavior with PULSE_PROP="filter.want=echo-cancel"

2018-02-13 Thread Raman Gupta
In my testing, Zoiper's echo cancellor does not work as well as PA's.

On Tue, Feb 13, 2018 at 10:24 AM, Russell Treleaven
<rtrelea...@bunnykick.ca> wrote:
> Also zoiper will have its own echo cancellor.
> Why not use that?
>
> On Tue, Feb 13, 2018 at 10:21 AM, Georg Chini <ge...@chini.tk> wrote:
>>
>> On 12.02.2018 20:15, Raman Gupta wrote:
>>>
>>> When I start Zoiper with PULSE_PROP="filter.want=echo-cancel", it
>>> demonstrates some odd behavior. It ends up creating many Echo-Cancel
>>> Sink and Source streams, and I am unable to select my USB headset as
>>> the source of any of them.
>>>
>>> I think, based on the discussion around a recent crasher bug I
>>> reported, Zoiper is trying to be too clever, without being clever
>>> enough, and is actually trying to open up various streams to all of
>>> the Pulse Input and Output devices. Is this just a Zoiper issue, or is
>>> there something I can do on the Pulse side to solve this?
>>>
>>> I think one option is load the echo-cancellation module once in
>>> /etc/pulse/default.pa and then use the volume control to manually hook
>>> up Zoiper to that module.
>>>
>>> Regards,
>>> Raman
>>> ___
>>
>> Hi,
>>
>> can't you tell zoiper which devices to use? And can you provide
>> a log (beginning when you start zoiper) so that we can see what
>> actually happens?
>>
>> Regards
>>  Georg
>>
>> ___
>> pulseaudio-discuss mailing list
>> pulseaudio-discuss@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
>
>
>
>
> --
> Sincerely,
>
> Russell Treleaven
> sip:rtrelea...@sip.bunnykick.ca
>
>
> ___
> pulseaudio-discuss mailing list
> pulseaudio-discuss@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
>
___
pulseaudio-discuss mailing list
pulseaudio-discuss@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] Odd app behavior with PULSE_PROP="filter.want=echo-cancel"

2018-02-13 Thread Raman Gupta
On Tue, Feb 13, 2018 at 10:21 AM, Georg Chini <ge...@chini.tk> wrote:
> On 12.02.2018 20:15, Raman Gupta wrote:
>>
>> When I start Zoiper with PULSE_PROP="filter.want=echo-cancel", it
>> demonstrates some odd behavior. It ends up creating many Echo-Cancel
>> Sink and Source streams, and I am unable to select my USB headset as
>> the source of any of them.
>>
>> I think, based on the discussion around a recent crasher bug I
>> reported, Zoiper is trying to be too clever, without being clever
>> enough, and is actually trying to open up various streams to all of
>> the Pulse Input and Output devices. Is this just a Zoiper issue, or is
>> there something I can do on the Pulse side to solve this?
>>
>> I think one option is load the echo-cancellation module once in
>> /etc/pulse/default.pa and then use the volume control to manually hook
>> up Zoiper to that module.
>>
>> Regards,
>> Raman
>> ___
>
> Hi,
>
> can't you tell zoiper which devices to use? And can you provide
> a log (beginning when you start zoiper) so that we can see what
> actually happens?
>
> Regards
>  Georg

It looks like going back to a global echo cancellor works better -- PA
allows me to do the expected thing which is to have one echo cancellor
source/sink which I can hook up to a device source/sink. And then hook
up Zoiper to that echo cancellor source/sink.

The reason I don't give Zoiper specific devices to access is that I
don't want to have to configure every application individually -- to
me, this ability is one of the main benefits of PulseAudio.

I'll try and capture a log later when I won't disrupt a bunch of work
by killing the PA daemon and causing my mic to fail in every app that
uses PA (it sucks that this is a problem too).

Regards,
Raman
___
pulseaudio-discuss mailing list
pulseaudio-discuss@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] Odd app behavior with PULSE_PROP="filter.want=echo-cancel"

2018-02-12 Thread Raman Gupta
When I start Zoiper with PULSE_PROP="filter.want=echo-cancel", it
demonstrates some odd behavior. It ends up creating many Echo-Cancel
Sink and Source streams, and I am unable to select my USB headset as
the source of any of them.

I think, based on the discussion around a recent crasher bug I
reported, Zoiper is trying to be too clever, without being clever
enough, and is actually trying to open up various streams to all of
the Pulse Input and Output devices. Is this just a Zoiper issue, or is
there something I can do on the Pulse side to solve this?

I think one option is load the echo-cancellation module once in
/etc/pulse/default.pa and then use the volume control to manually hook
up Zoiper to that module.

Regards,
Raman
___
pulseaudio-discuss mailing list
pulseaudio-discuss@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] [PATCH 0/2] module-echo-cancel fixes

2017-11-09 Thread Raman Gupta
On 09/11/17 07:12 AM, Tanu Kaskinen wrote:
> Tanu Kaskinen (2):
>   echo-cancel: ignore remaining canceller messages after the module has
> been unloaded
>   echo-cancel: fix a memory leak
> 
>  src/modules/echo-cancel/module-echo-cancel.c | 22 ++
>  1 file changed, 22 insertions(+)

Tanu, is it possible to add unit tests that will help ensure this
isn't broken again in the future? This is the second or third time
I've had an issue related to this.

Regards,
Raman
___
pulseaudio-discuss mailing list
pulseaudio-discuss@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] [PATCH 0/7] Fix for an echo-cancel related crash

2016-04-09 Thread Raman Gupta
On 03/22/2016 09:41 AM, Tanu Kaskinen wrote:
> This is my second attempt at fixing bug 93443[1]. The first attempt[2]
> had a nasty side effect of killing streams connected to any filter
> sink while the filter sink was removed. This second attempt is an
> entirely different approach.

As per my comment on bug 93443, I tested the first three patches of
this series on PulseAudio 7.1 (Fedora 23), and I can verify that an
echo-cancelled stream does not get killed any longer.

Regards,
Raman
___
pulseaudio-discuss mailing list
pulseaudio-discuss@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] [PATCH 0/7] Fix for an echo-cancel related crash

2016-03-23 Thread Raman Gupta
Is there a Fedora copr repo where I can get the patched version for
testing?

If not, can I apply this patch to v7.1?

Thanks!
Raman

On 03/22/2016 09:41 AM, Tanu Kaskinen wrote:
> This is my second attempt at fixing bug 93443[1]. The first attempt[2]
> had a nasty side effect of killing streams connected to any filter
> sink while the filter sink was removed. This second attempt is an
> entirely different approach.
> 
> The first patch is the "main" fix, but I think the second patch must
> be applied together with it, because after implementing the first fix,
> the result was that module-device-manager went into some infinite
> stream moving loop that was only stopped by a crash in
> module-echo-cancel (that crash is a separate issue, which I'll try to
> fix next).
> 
> When trying to reproduce the use case of bug 93443, I found that
> module-device-manager can easily break the filtering set up by
> module-filter-apply. The third patch fixes that.
> 
> The last four patches refactor stream proplist updating. The third
> patch has code that sets a property, and getting the property update
> logging just the way I want was frustrating, so I changed the core
> property updating code to take care of the logging.
> 
> [1] https://bugs.freedesktop.org/show_bug.cgi?id=93443
> [2] http://thread.gmane.org/gmane.comp.audio.pulseaudio.general/24735
> 
> Tanu Kaskinen (7):
>   echo-cancel: rework move handling
>   don't move streams to devices that are going away
>   device-manager, filter-apply: don't reroute streams that have a filter
>   sink-input, source-output: rework property setting
>   sink-input, source-output: remove set_name()
>   loopback: refactor proplist updating
>   filter-apply: simplify proplist updating
> 
>  src/modules/echo-cancel/module-echo-cancel.c |  28 ++
>  src/modules/module-device-manager.c  |  14 +++
>  src/modules/module-filter-apply.c|  11 +-
>  src/modules/module-loopback.c|  28 +++---
>  src/pulsecore/protocol-native.c  |   4 +-
>  src/pulsecore/sink-input.c   | 145 
> +--
>  src/pulsecore/sink-input.h   |   4 +-
>  src/pulsecore/sink.c |   7 +-
>  src/pulsecore/sink.h |   6 ++
>  src/pulsecore/source-output.c| 145 
> +--
>  src/pulsecore/source-output.h|   4 +-
>  src/pulsecore/source.c   |   5 +
>  src/pulsecore/source.h   |   6 ++
>  13 files changed, 304 insertions(+), 103 deletions(-)
> 
___
pulseaudio-discuss mailing list
pulseaudio-discuss@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] Filters, autoloading, profile changes and crashes

2015-06-09 Thread Raman Gupta
Arun,

I am the OP for bug #90416. I can't comment on the
details/implementation of your patch, but I will note that the patch
series does indeed fix the problem. Very cool!

Regards,
Raman Gupta

On 06/09/2015 03:08 AM, a...@accosted.net wrote:
 This fixes bug #90416. Details of the problem and the fix are on the bug
 and patch #3. This does feel a bit sub-optimal to me, but I couldn't
 think of a better way to maintain consistency of state. If anyone has
 better ideas, I'm all ears.
 
 Cheers,
 Arun
 
 ___
 pulseaudio-discuss mailing list
 pulseaudio-discuss@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
 
___
pulseaudio-discuss mailing list
pulseaudio-discuss@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] Filters, autoloading, profile changes and crashes

2015-06-09 Thread Raman Gupta
On 06/09/2015 01:21 PM, Raman Gupta wrote:
 Arun,
 
 I am the OP for bug #90416. I can't comment on the
 details/implementation of your patch, but I will note that the patch
 series does indeed fix the problem. Very cool!

One minor note: after applying this patch, when the card profile is
switched, the PulseAudio Volume Control application shows Unknown
Input on the echo-cancelled stream in the Recording tab. Restarting
the volume control app resets it. Kmix does not show the same behavior.

Regards,
Raman

___
pulseaudio-discuss mailing list
pulseaudio-discuss@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss