Re: [pulseaudio-discuss] Bluetooth ports

2012-11-22 Thread David Henningsson
On 11/23/2012 05:24 AM, Arun Raghavan wrote: David, On Thu, 2012-11-22 at 16:55 +0100, David Henningsson wrote: [...] ...now, I'm testing PulseAudio 2.99.2 and things do not show up as I expect them to. In PulseAudio 2.x, there were no bluetooth ports at all, in 2.99, "pactl list-cards" looks l

Re: [pulseaudio-discuss] pulseaudio daemon not responging

2012-11-22 Thread Arun Raghavan
On Thu, 2012-11-22 at 23:07 +0200, Stefan Stefanov wrote: > Can anyone explain me what is wrong with libpthread-stub.so? > > As I understand, sem_wait() works as intended according to the source code: > static int __sem_wait_stub(sem_t *_sem) > { > unsigned int *sem = (unsigned int *) _sem; >

Re: [pulseaudio-discuss] Bluetooth ports

2012-11-22 Thread Arun Raghavan
David, On Thu, 2012-11-22 at 16:55 +0100, David Henningsson wrote: [...] > ...now, I'm testing PulseAudio 2.99.2 and things do not show up as I > expect them to. In PulseAudio 2.x, there were no bluetooth ports at all, > in 2.99, "pactl list-cards" looks like this [1]. I don't know if it's a >

Re: [pulseaudio-discuss] gnome-shell hangs, waiting for pulse-audio

2012-11-22 Thread Henrik /KaarPoSoft
On 11/22/12 13:39, Tanu Kaskinen wrote: Henrik, is it possible for you to try 2.99.2 too? For what it's worth, last time when I said "I can't reproduce this" I did have other windows open, but I retested with the gnome-terminal window being the only window, and I still couldn't reproduce the bu

Re: [pulseaudio-discuss] [PATCH v3 0/5] Unloading bluetooth device module instances

2012-11-22 Thread Tanu Kaskinen
On Thu, 2012-11-22 at 15:20 +0100, Mikel Astiz wrote: > From: Mikel Astiz > > This is a follow-up of Tanu's work on fixing the issue with duplicated > module-bluetooth-device instances. > > Main changes from v2: > - Audio.State is not removed since otherwise there is an issue with > module-car

Re: [pulseaudio-discuss] [PATCH v3 4/5] bluetooth: Run the discovery hook only when necessary

2012-11-22 Thread Tanu Kaskinen
On Thu, 2012-11-22 at 15:20 +0100, Mikel Astiz wrote: > From: Tanu Kaskinen > > This is a minor optimization too, but the main benefit is that it's > makes the code easier to understand (I hope), since run_callback() > won't be called at times when it's not needed. This patch doesn't seem to con

Re: [pulseaudio-discuss] Bluetooth ports

2012-11-22 Thread Tanu Kaskinen
On Thu, 2012-11-22 at 16:55 +0100, David Henningsson wrote: > First, sorry for not being into all the bluetooth details and relying on > you to sort it all out correctly... > > ...now, I'm testing PulseAudio 2.99.2 and things do not show up as I > expect them to. In PulseAudio 2.x, there were no

Re: [pulseaudio-discuss] pulseaudio daemon not responging

2012-11-22 Thread Stefan Stefanov
Ok. Thanks. 2012/11/22 Tanu Kaskinen : > On Thu, 2012-11-22 at 20:38 +0200, Stefan Stefanov wrote: >> >http://thread.gmane.org/gmane.comp.audio.pulseaudio.general/11000 >> Exactly the same problem as mine. >> >> 1. Build of pulseaudio failed without libpthread-stubs.so with "no >> such file ..." >

Re: [pulseaudio-discuss] pulseaudio daemon not responging

2012-11-22 Thread Stefan Stefanov
Can anyone explain me what is wrong with libpthread-stub.so? As I understand, sem_wait() works as intended according to the source code: static int __sem_wait_stub(sem_t *_sem) { unsigned int *sem = (unsigned int *) _sem; if (!*sem) { /* Not available, simulate a blocking sem_wait

Re: [pulseaudio-discuss] pulseaudio daemon not responging

2012-11-22 Thread Tanu Kaskinen
On Thu, 2012-11-22 at 20:38 +0200, Stefan Stefanov wrote: > >http://thread.gmane.org/gmane.comp.audio.pulseaudio.general/11000 > Exactly the same problem as mine. > > 1. Build of pulseaudio failed without libpthread-stubs.so with "no > such file ..." > 2. I noticed that in crux linux, sem_wait() r

Re: [pulseaudio-discuss] pulseaudio daemon not responging

2012-11-22 Thread Stefan Stefanov
>http://thread.gmane.org/gmane.comp.audio.pulseaudio.general/11000 Exactly the same problem as mine. 1. Build of pulseaudio failed without libpthread-stubs.so with "no such file ..." 2. I noticed that in crux linux, sem_wait() resides in libpthread_stubs.so, and it is found through call to libpthr

[pulseaudio-discuss] Bluetooth ports

2012-11-22 Thread David Henningsson
First, sorry for not being into all the bluetooth details and relying on you to sort it all out correctly... ...now, I'm testing PulseAudio 2.99.2 and things do not show up as I expect them to. In PulseAudio 2.x, there were no bluetooth ports at all, in 2.99, "pactl list-cards" looks like this

[pulseaudio-discuss] [PATCH v3 5/5] bluetooth: Unload device module when no audio profiles connected

2012-11-22 Thread Mikel Astiz
From: Tanu Kaskinen Without this patch, device modules will be left around after the device has been disconnected and when they are reconnected, the discovery module will load duplicate device module instances. BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=57239 --- src/modules/bluetoot

[pulseaudio-discuss] [PATCH v3 4/5] bluetooth: Run the discovery hook only when necessary

2012-11-22 Thread Mikel Astiz
From: Tanu Kaskinen This is a minor optimization too, but the main benefit is that it's makes the code easier to understand (I hope), since run_callback() won't be called at times when it's not needed. --- src/modules/bluetooth/bluetooth-util.c | 10 -- 1 file changed, 8 insertions(+), 2

[pulseaudio-discuss] [PATCH v3 3/5] bluetooth: Add helper pa_bluetooth_device_any_audio_connected()

2012-11-22 Thread Mikel Astiz
From: Tanu Kaskinen The new helper function makes it easier to check whether any audio profiles are connected. That information is needed by the discovery module for deciding whether a new device module should be loaded. The device module should use this information too to unload itself at the ri

[pulseaudio-discuss] [PATCH v3 2/5] bluetooth: Rename former device_use_audio()

2012-11-22 Thread Mikel Astiz
From: Mikel Astiz Use a more accurate name for the function since it doesn't just check if it is an audio device (which can be detected quite early), but it also checks if the most relevant properties (device info, etc.) have been received. Besides, add the const qualifier to the pointer since i

[pulseaudio-discuss] [PATCH v3 1/5] bluetooth: Ignore Device.Connected

2012-11-22 Thread Mikel Astiz
From: Tanu Kaskinen The Device.Connected was only used for tracking whether a device module should be loaded, but that information is already included in the individual profile state properties. The property can therefore be completely ignored without any loss in functionality. --- src/modules/b

[pulseaudio-discuss] [PATCH v3 0/5] Unloading bluetooth device module instances

2012-11-22 Thread Mikel Astiz
From: Mikel Astiz This is a follow-up of Tanu's work on fixing the issue with duplicated module-bluetooth-device instances. Main changes from v2: - Audio.State is not removed since otherwise there is an issue with module-card-restore (see Tanu's v2 cover-letter). - Instead of using an addition

Re: [pulseaudio-discuss] [PATCH pavucontrol] mainwindow: Don't clear the "updating" flag in updateCard() too early.

2012-11-22 Thread Tanu Kaskinen
On Tue, 2012-11-20 at 21:59 +, Colin Guthrie wrote: > 'Twas brillig, and Tanu Kaskinen at 19/11/12 13:29 did gyre and gimble: > > The prepareMenu() call can change the active profile selection, which > > in turn will cause a "set card profile" command to be sent to the > > server if the "updati

Re: [pulseaudio-discuss] [PATCH v2 3/5] bluetooth: Add a pa_bluetooth_discovery pointer to pa_bluetooth_device.

2012-11-22 Thread Tanu Kaskinen
On Tue, 2012-11-20 at 16:48 +0200, Tanu Kaskinen wrote: > This simplifies some function argument lists. I pushed this patch, the other patches will be reworked by Mikel. -- Tanu ___ pulseaudio-discuss mailing list pulseaudio-discuss@lists.freedesktop.

Re: [pulseaudio-discuss] pulseaudio daemon not responging

2012-11-22 Thread Tanu Kaskinen
On Thu, 2012-11-22 at 15:15 +0200, Stefan Stefanov wrote: > #2 0xb7b1f665 in sem_wait () from /usr/lib/libpthread-stubs.so.0 I think libpthread-stubs.so.0 isn't a very good implementation for sem_wait()... I don't have any straightforward advice for what to do, but maybe you'll find this thread

Re: [pulseaudio-discuss] pulseaudio daemon not responging

2012-11-22 Thread Stefan Stefanov
To be more precise here is gdb log generated by using Tanu's advise: >gdb pulseaudio >(gdb) run -vvv > >Ctrl-C >(gdb) thread apply all bt Thread 2 (Thread 0xb3455b40 (LWP 16627)): #0 0xb7fdd416 in __kernel_vsyscall () #1 0xb7719db9 in ppoll () from /lib/libc.so.6 #2 0xb7f5a841 in pa_rtpoll_run

Re: [pulseaudio-discuss] pulseaudio daemon not responging

2012-11-22 Thread Stefan Stefanov
Even more details: void pa_semaphore_wait(pa_semaphore *s) { int ret; pa_assert(s); do { ret = sem_wait(&s->sem); } while (ret < 0 && errno == EINTR); pa_assert(ret == 0); } Actually it hangs in sem_wait(...) API function.

Re: [pulseaudio-discuss] pulseaudio daemon not responging

2012-11-22 Thread Stefan Stefanov
Just used gdb and find the exact place of hanging. (Just to mention that now I debug the same system as mine, but in the VM. Problem with haging in VM also occurs.) Result of debugging with gdb: ---

Re: [pulseaudio-discuss] gnome-shell hangs, waiting for pulse-audio

2012-11-22 Thread Tanu Kaskinen
On Wed, 2012-11-21 at 21:07 +, Stefan P. wrote: > Hello, > > Yesterday I installed pulseaudio-2.99.2, and since then, the freeze has not > happened again! Great! Henrik, is it possible for you to try 2.99.2 too? For what it's worth, last time when I said "I can't reproduce this" I did have o

Re: [pulseaudio-discuss] pulseaudio daemon not responging

2012-11-22 Thread Tanu Kaskinen
On Thu, 2012-11-22 at 00:22 +0200, Stefan Stefanov wrote: > Ok. > Here is a report after simple debugging with "traps" like pa_log("test"); > The call stack: > > pa_alsa_sink_new -> alsa-sink.c > pa_sink_put(u->sink) -> sink.c > pa_assert_se(sink_set_state(s, PA_SINK_IDLE) == 0) -> sink.c -> i