[pulseaudio-discuss] [PATCH] reserve-monitor: Don't trigger on our own events

Tanu Kaskinen tanuk at iki.fi
Fri Jan 11 11:18:03 PST 2013


On Fri, 2013-01-11 at 14:04 +0100, David Henningsson wrote:
> This fixes a bug where pulseaudio would give up the device (due to
> a request from JACK), but then immediately grab it again because
> the monitor callback fired, telling that the device is now available.
> 
> (Note: the protocol does not specify a timeout, i e if pulseaudio
> is requested to give its device up but JACK does not grab the dbus name,
> at what point is PulseAudio allowed to re-grab it?)
> 
> Signed-off-by: David Henningsson <david.henningsson at canonical.com>
> ---
>  src/modules/reserve-monitor.c |   30 +++++++++++++++++++-----------
>  1 file changed, 19 insertions(+), 11 deletions(-)
> 
> Will commit this to stable-3.x and master in a few days if there are
> no objections.
> 
> @Lennart, would you mind committing this to the upstream reserve.git repo as well?

This seems pretty equivalent to a patch[1] that I sent earlier, with the
difference that with your patch change_cb() is called also in "busy ->
busy" transitions (i.e. when the bus name changes owner, and neither old
or new owner is pulseaudio).

[1] http://thread.gmane.org/gmane.comp.audio.pulseaudio.general/15053

-- 
Tanu



More information about the pulseaudio-discuss mailing list