profile/ivi/pulseaudio.git
13 years agocore: Add a pa_format_info structure
Arun Raghavan [Fri, 25 Feb 2011 07:05:14 +0000 (12:35 +0530)]
core: Add a pa_format_info structure

This will be used to represent the format of data provided by the client
for both compressed and PCM formats in a new extended API.

13 years agosample: Use PA_SAMPLE_INVALID instead of numeric value
Arun Raghavan [Thu, 24 Feb 2011 09:16:01 +0000 (14:46 +0530)]
sample: Use PA_SAMPLE_INVALID instead of numeric value

13 years agosink: Trivial typo fix
Arun Raghavan [Fri, 25 Feb 2011 07:14:53 +0000 (12:44 +0530)]
sink: Trivial typo fix

13 years agosink-input: Don't resample passthrough inputs
Pierre-Louis Bossart [Tue, 8 Mar 2011 09:42:56 +0000 (15:12 +0530)]
sink-input: Don't resample passthrough inputs

13 years agofilter-apply: Mark modules as being autoloaded
Arun Raghavan [Mon, 2 May 2011 04:38:27 +0000 (10:08 +0530)]
filter-apply: Mark modules as being autoloaded

(Based on Colin's review) We mark modules as being autoloaded so that
they can handle this as a special case if needed (which is required by
module-echo-cancel for now). This inverts how things were done and makes
using these modules manually less error-prone.

13 years agoMake connect-stress test compile for win32
Maarten Bosmans [Sat, 30 Apr 2011 19:46:30 +0000 (21:46 +0200)]
Make connect-stress test compile for win32

13 years agobuild-system: Small fixes
Maarten Bosmans [Sat, 30 Apr 2011 13:25:55 +0000 (15:25 +0200)]
build-system: Small fixes

13 years agobuild-system: Rearrange database selection
Maarten Bosmans [Sat, 30 Apr 2011 13:25:54 +0000 (15:25 +0200)]
build-system: Rearrange database selection

13 years agobuild-system: Replace some more conditionals with AS_IF
Maarten Bosmans [Sat, 30 Apr 2011 13:25:53 +0000 (15:25 +0200)]
build-system: Replace some more conditionals with AS_IF

13 years agobuild-system: Move dependency error messages to outer scope
Maarten Bosmans [Sat, 30 Apr 2011 13:25:52 +0000 (15:25 +0200)]
build-system: Move dependency error messages to outer scope

13 years agobuild-system: Move AC_DEFINE to separate line with AS_IF
Maarten Bosmans [Sat, 30 Apr 2011 13:25:51 +0000 (15:25 +0200)]
build-system: Move AC_DEFINE to separate line with AS_IF

13 years agobuild-system: Use AS_IF macro for configure output
Maarten Bosmans [Sat, 30 Apr 2011 13:25:50 +0000 (15:25 +0200)]
build-system: Use AS_IF macro for configure output

And add some HAVE_[feature] variables for clarity.

13 years agobuild-system: Simplify AC_ARG_ENABLE usage
Maarten Bosmans [Sat, 30 Apr 2011 13:25:49 +0000 (15:25 +0200)]
build-system: Simplify AC_ARG_ENABLE usage

Make use of the enable_[feature] variable automatically defined
by AC_ARG_ENABLE instead of defining our own variable.

13 years agocombine: Rename module-combine to module-combine-sink.
Colin Guthrie [Fri, 29 Apr 2011 15:40:50 +0000 (16:40 +0100)]
combine: Rename module-combine to module-combine-sink.

This also includes a compatibility module that will ultimately be removed in
a future release.

13 years agobluetooth: Only close SCO if status has changed
Luiz Augusto von Dentz [Fri, 29 Apr 2011 14:48:07 +0000 (17:48 +0300)]
bluetooth: Only close SCO if status has changed

In case the io thread is starting there is no point on closing SCO just
to resume it latter when the source/sink changes to opened state.

13 years agobluetooth: Fix using pointer-pointer when appending an array as variant
Luiz Augusto von Dentz [Fri, 29 Apr 2011 14:48:06 +0000 (17:48 +0300)]
bluetooth: Fix using pointer-pointer when appending an array as variant

pa_dbus_append_basic_array_variant_dict_entry should take the direct
pointer to the array.

13 years agobluetooth: Fix not updating sample spec when using Media API
Luiz Augusto von Dentz [Fri, 29 Apr 2011 14:48:05 +0000 (17:48 +0300)]
bluetooth: Fix not updating sample spec when using Media API

When using transport configured via Media API sample spec needs to be
updated since codec configuration may affect it when e.g. headset
configure a different frequency or number of channels from default.

13 years agox11: More XCB fixes.
Arnaud Fontaine [Tue, 29 Mar 2011 20:57:28 +0000 (22:56 +0159)]
x11: More XCB fixes.

Commit 65ef80b fixed building with xcb-util >= 0.3.8, but the reply is never
checked (possible SIGSEGV if the reply is NULL) nor freed (memory leak at each
call of the functions).

Also, remove include and dependencies on xcb-atom, as it was only meaningful
for xcb_atom_get() and STRING, and depend instead on xcb >= 1.6 for
XCB_ATOM_STRING.

13 years agodbus: Fix the order of freeing stuff when unloading module-dbus-protocol.
Tanu Kaskinen [Fri, 29 Apr 2011 10:58:43 +0000 (13:58 +0300)]
dbus: Fix the order of freeing stuff when unloading module-dbus-protocol.

13 years agodbus: Fix connection idxset freeing when unloading the module.
Tanu Kaskinen [Fri, 29 Apr 2011 10:58:42 +0000 (13:58 +0300)]
dbus: Fix connection idxset freeing when unloading the module.

If u->connections isn't empty when module-dbus-protocol is
unloaded, then connection_free() is called for the
remaining connections when the idxset is freed.
connection_free() tries to remove the connection from the
idxset, but that fails, because the item has already been
removed from the idxset in this scenario.

The problem is solved by not trying to remove the connection
from the idxset in connection_free(). Instead, whoever wants
to delete connections, has to remove the connection from the
idxset in addition to calling connection_free().

13 years agomatch: Support for both merging and replacing proplist updates.
Tanu Kaskinen [Fri, 29 Apr 2011 10:45:29 +0000 (13:45 +0300)]
match: Support for both merging and replacing proplist updates.

This patch adds a new update mode specifier that can be optionally
given in match rules after the regexp. Property list updates triggered
by the rule will honour the given mode. The two allowed modes are 'merge'
and 'replace', corresponding to PA_UPDATE_MERGE and PA_UPDATE_REPLACE
respectively. If omitted, the mode defaults to PA_UPDATE_MERGE, ie. to
the original behavior.

For example, to force 'media.role' to be overwritten with 'bar' for
streams matching foo you can use an entry like this:

foo replace "bar"

This will really overwrite media.role to bar even if it has already been
set to something else by the application.

Thanks to Krisztian Litkey for the original patch and the description
above. In addition to implementing the new feature, this patch fixes
a number of bugs in the parsing code.

13 years agocore: Drop empty gaps in the memblockq when playing data from it.
Antti-Ville Jansson [Wed, 20 Apr 2011 12:56:29 +0000 (15:56 +0300)]
core: Drop empty gaps in the memblockq when playing data from it.

It's possible that the memblockq of a sink input is rewound to a negative read
index if the sink input is moved between sinks shortly after its creation. When
this happens, pa_memblockq_peek() returns a memchunk whose 'memblock' field is
NULL and whose 'length' field indicates the length of the gap caused by the
negative read index. This will trigger an assert in play-memblockq.c.

If the memblockq had a silence memchunk, pa_memblockq_peek() would return
silence for the duration of the gap and the assert would be avoided. However,
this approach would prevent the sink input from being drained and is thus not
possible. Instead, we handle the aforementioned situation by dropping the gap
indicated by the 'length' field of the memchunk and by peeking the actual data
that comes after the gap.

This scenario seems to be quite rare in everyday use, but it causes a severe
bug in the handheld world. The assert can be triggered e.g. by loading two null
sinks, playing a sample from the cache to one of them and then moving the
created sink input between the two sinks. The rewinds done by the null sinks
seem to be quite long (I don't know if this is normal behaviour or something
fishy in module-null-sink).

See also:

    6bd34156b130c07b130de10111a12ef6dab18b52
    virtual-sink: Fix a crash when moving the sink to a new master right after setup.

    https://tango.0pointer.de/pipermail/pulseaudio-discuss/2011-February/009105.html

Reproduce:

This problem can be reproduced with the following script:

SAMPLE_PATH="/usr/share/sounds/alsa/"
SAMPLE="Front_Left"

pactl remove-sample $SAMPLE 2> /dev/null
pactl upload-sample $SAMPLE_PATH$SAMPLE.wav

mod1=`pactl load-module module-null-sink sink_name=null1`
mod2=`pactl load-module module-null-sink sink_name=null2`

pactl play-sample $SAMPLE null1

input=`pactl list | grep "Sink Input #" | tail -n 1 | cut -d# -f2`

echo "Sample $SAMPLE playing as Sink Input #$input"

pactl move-sink-input $input null2
pactl move-sink-input $input null1

pactl unload-module $mod1
pactl unload-module $mod2

13 years agostream-restore: Enable database dumping if DEBUG_VOLUME is defined.
Tanu Kaskinen [Wed, 27 Apr 2011 10:08:38 +0000 (13:08 +0300)]
stream-restore: Enable database dumping if DEBUG_VOLUME is defined.

13 years agomodule-null-source: New null-source module
Marc-André Lureau [Wed, 27 Apr 2011 11:50:00 +0000 (14:50 +0300)]
module-null-source: New null-source module

13 years agox11: Fix build errors with newest xcb-util.
Maciej Grela [Tue, 29 Mar 2011 20:57:28 +0000 (22:56 +0159)]
x11: Fix build errors with newest xcb-util.

The xcb_atom_get functions were removed from xcb-util. Changed these to
xcb_intern_atom/xcb_intern_atom_reply. Also, STRING is now
XCB_ATOM_STRING.

13 years agofilter-heuristics: Match roles correctly
Arun Raghavan [Fri, 22 Apr 2011 14:27:29 +0000 (19:57 +0530)]
filter-heuristics: Match roles correctly

13 years agoecho-cancel: Play nice with module-filter-*
Arun Raghavan [Fri, 22 Apr 2011 12:14:50 +0000 (17:44 +0530)]
echo-cancel: Play nice with module-filter-*

With automaticl filter loading by module-filter-apply, setting the
virtual sink/source to have the "phone" intended role will break routing
when you first connect a phone stream to an ALSA device and then turn on
your Bluetooth headset. This happens because module-intended-roles
doesn't move a stream if it is already on a device that provides the
required role.

This patch introduces a "manual_load" parameter that is meant to be used
when not using module-filter-apply for loading the AEC module. If this
parameter is set, the virtual devices are given the "phone" role, else
we count on module-filter-heuristics to do the right thing.

13 years agofilters: Handle filters on sources as well
Arun Raghavan [Fri, 22 Apr 2011 12:03:03 +0000 (17:33 +0530)]
filters: Handle filters on sources as well

This makes the core code in the filter-* modules generic enough to be
used on sources or sinks. We need special handling for modules that
introduce more than one sink (for now echo-cancel only).

13 years agofilters: Handle stream moves properly
Arun Raghavan [Thu, 21 Apr 2011 07:22:27 +0000 (12:52 +0530)]
filters: Handle stream moves properly

This makes sure that we handle streams moving between sinks properly. To
do this, we change the way the filter.* properties are handled a little
bit.

Firstly, this splits up the "filter.apply" property into two properties
- "filter.want" and "filter.apply". "filter.apply" acts as before - it
bypasses module-filter-heuristics and directly tells module-filter-apply
what filters are to be applied.

"filter.want" is used to tell module-filter-heuristics what filters the
client wants. The module then decides whether to actually apply the
filter or not (for now, this makes sure we don't apply echo-cancellation
even if requested on phone sinks (where it is assumed AEC is taken care
of or is not required).

Next, we also make sure that we track whether the client set
"filter.apply" or module-filter-heuristics did - and in the latter case,
we recalculate "filter.apply" and then have module-filter-apply apply
the filter if required. This introduces some evil in the form of causing
the move_finish callback to possibly trigger another move, but we
protect for this case (with a property) to be doubly sure of not causing
an infinite loop.

13 years agofilter-heuristics: Only apply AEC if we're not already on a phone sink
Arun Raghavan [Wed, 20 Apr 2011 12:15:26 +0000 (17:45 +0530)]
filter-heuristics: Only apply AEC if we're not already on a phone sink

This makes sure that we don't apply AEC on sinks that are already
connected to a "phone" device, the assumptiong being that anything
marked as such either doesn't have need it, or handles it itself.

13 years agotest: Make the connect-stress less likely to bail out due to >32 streams.
Colin Guthrie [Fri, 22 Apr 2011 09:25:42 +0000 (10:25 +0100)]
test: Make the connect-stress less likely to bail out due to >32 streams.

When running two connect-stress tests at the same time the liklihood of >32 streams
per sink increases. All it takes is for an event sound to fire to trigger an abort of
the test.

This leaves just a little bit of wriggle room for a couple external streams.
Of course the overall problem is still there but this just makes it
slightly less likely without really affecting the test itself.

13 years agobuild-sys: Make -isysroot and -mmacosx-version-min configurable
Daniel Mack [Fri, 22 Apr 2011 00:41:24 +0000 (02:41 +0200)]
build-sys: Make -isysroot and -mmacosx-version-min configurable

13 years agoprotocol-dbus: Fix some memory management bugs.
Tanu Kaskinen [Thu, 21 Apr 2011 05:06:53 +0000 (08:06 +0300)]
protocol-dbus: Fix some memory management bugs.

There were several memory leaks. In addition to those,
pa_dbus_protocol_add_interface() used a string from the
caller as a key to a hashmap, instead of a copy of the
string. This caused trouble when the caller freed the
string while the key was still in use in the hashmap.

13 years agomatch: Match rule earlier, in SINK_INPUT_NEW
Marc-André Lureau [Wed, 20 Apr 2011 12:22:46 +0000 (15:22 +0300)]
match: Match rule earlier, in SINK_INPUT_NEW

13 years agomatch: Don't double free in case of missing table file
Marc-André Lureau [Wed, 20 Apr 2011 11:49:18 +0000 (14:49 +0300)]
match: Don't double free in case of missing table file

13 years agomodule-coreaudio-device: Fix two build warnings
Daniel Mack [Fri, 22 Apr 2011 02:12:36 +0000 (04:12 +0200)]
module-coreaudio-device: Fix two build warnings

13 years agomodule-coreaudio-device: Set the thread name to device name
Daniel Mack [Fri, 22 Apr 2011 02:10:46 +0000 (04:10 +0200)]
module-coreaudio-device: Set the thread name to device name

This makes gdb's "info threads" better understandable

13 years agomodule-coreaudio-device: Dispatch sink/source state messages from main loop
Daniel Mack [Fri, 22 Apr 2011 02:08:45 +0000 (04:08 +0200)]
module-coreaudio-device: Dispatch sink/source state messages from main loop

This fixes a long standing race condition when tearing down streams on
Mac OS X.

13 years agothread-posix: Use pthread_(get|set)name_np() if available
Daniel Mack [Fri, 22 Apr 2011 02:28:11 +0000 (04:28 +0200)]
thread-posix: Use pthread_(get|set)name_np() if available

Newer generations of libpthread have functions to set and get the thread
names. If available, use them.

13 years agopulsecore:: Define _POSIX_C_SOURCE locally for rtclock on OSX
Daniel Mack [Fri, 22 Apr 2011 00:27:35 +0000 (02:27 +0200)]
pulsecore:: Define _POSIX_C_SOURCE locally for rtclock on OSX

Defining this macro on a global level is disadvantageous for other APIs,
and as we need it for clock_gettime() only on Mac OS X, define it
locally in pulsecore/core-rtclock.c only.

13 years agoutil: Implement pa_get_binary_name() for Mac OS X
Daniel Mack [Fri, 15 Apr 2011 17:35:25 +0000 (19:35 +0200)]
util: Implement pa_get_binary_name() for Mac OS X

13 years agomodule-coreaudio-detect: Add 'ioproc_frames' parameter
Daniel Mack [Thu, 7 Apr 2011 10:59:52 +0000 (12:59 +0200)]
module-coreaudio-detect: Add 'ioproc_frames' parameter

This value is passed on to the instances of module-coreaudio-device that
are loaded upon device detection. The value is purely optional, as the
device module will fall back to to its default if it's not given.

13 years agomemblockq: decode unset chunks as NULL chunks again
Lennart Poettering [Thu, 21 Apr 2011 19:05:51 +0000 (21:05 +0200)]
memblockq: decode unset chunks as NULL chunks again

This fixes asserts beeing hit when see requests are posted via the
native protocol.

13 years agofilter-apply: Make housekeeping optional
Arun Raghavan [Wed, 20 Apr 2011 08:15:48 +0000 (13:45 +0530)]
filter-apply: Make housekeeping optional

Adds an autoclean option (defaults to TRUE) that controls whether
module-filter-apply cleans up unused modules or not. This is useful in
cases where you know that a filter will be used often and thus can avoid
overhead from repeated module load/unload.

13 years agofilter: Move the proplist defines into the central place and document them.
Colin Guthrie [Wed, 20 Apr 2011 08:25:31 +0000 (09:25 +0100)]
filter: Move the proplist defines into the central place and document them.

13 years agotests: improve resampler test
Marc-André Lureau [Tue, 19 Apr 2011 10:29:19 +0000 (13:29 +0300)]
tests: improve resampler test

13 years agointerpol-test: remove unused include getopt.h
Marc-André Lureau [Tue, 19 Apr 2011 06:46:45 +0000 (09:46 +0300)]
interpol-test: remove unused include getopt.h

13 years agovala: More vala fixes
Alexander Kurtz [Mon, 18 Apr 2011 08:28:05 +0000 (10:28 +0200)]
vala: More vala fixes

1. Remove the "has_destroy_function=false" attribute. It was only
necessary because of a bug in vala which is fixed in 0.12. [1]

2. Add sizes to all fixed-size arrays to make vala recognize them as
such. Using symbolic constants for this is not yet supported. [2]

3. CardInfo struct: Move the brackets in the list of available profiles
to the type to make it clear that this is a dynamically-sized array. [3]

[1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=622773
[2] https://bugzilla.gnome.org/show_bug.cgi?id=647788
[3] http://0pointer.de/lennart/projects/pulseaudio/doxygen/structpa__card__info.html

13 years agovolume: Get more data from volume tests
Arun Raghavan [Sun, 17 Apr 2011 09:58:15 +0000 (15:28 +0530)]
volume: Get more data from volume tests

This makes the volume tests run in two loops and print the minimum,
maximum and standard deviation of readings from the inner loop. This
makes it easier to reason out performance drops (i.e. algorithmic
problems vs. other system issues such as processor contention).

13 years agostream-restore: Check for readability before reading volume
Arun Raghavan [Tue, 12 Apr 2011 07:41:40 +0000 (13:11 +0530)]
stream-restore: Check for readability before reading volume

This avoids an assert in pa_sink_input_get_volume() when connecting a
passthrough stream.

13 years agopa_poll(): Simplify detection of invalid fds in select() emulation mode
Daniel Mack [Wed, 13 Apr 2011 17:51:49 +0000 (19:51 +0200)]
pa_poll(): Simplify detection of invalid fds in select() emulation mode

For systems which have a fcntl() implementation, we can simplify the
code which determines whether a file selector is valid in pa_poll().

The old code, which is harder to read and more expensive, stays around
for all platforms we need to emulate poll() for using select(), and
which don't provide fcntl(). IOW, for Windows.

On Mac OS X, however, the detection for bad fds via more select() calls
doesn't work, resulting in hung main loops, so the patch fixes a real
bug there.

13 years agosink-input: Check flat volume with pa_sink_flat_volume_enabled().
Tanu Kaskinen [Fri, 15 Apr 2011 15:36:38 +0000 (18:36 +0300)]
sink-input: Check flat volume with pa_sink_flat_volume_enabled().

Checking just the flag doesn't work if the sink uses volume sharing, because
such sinks never have PA_SINK_FLAT_VOLUME set.

13 years agofilter-heuristics: New module that applies some basic heuristics regarding filters.
Colin Guthrie [Thu, 14 Apr 2011 11:05:45 +0000 (13:05 +0200)]
filter-heuristics: New module that applies some basic heuristics regarding filters.

At present the only heuristic is one to apply the echo-cancel filter
when dealing with phone streams.

13 years agofilter-apply: New module to automatically load filter sinks (and move streams) based...
Colin Guthrie [Thu, 14 Apr 2011 11:04:03 +0000 (13:04 +0200)]
filter-apply: New module to automatically load filter sinks (and move streams) based on sink-input property hints.

This module does not yet deal with modules that need matched inputs/outputs
(i.e. echo-cancel) but this will be added in due course.

13 years agoequalizer: Use sink_master as the module argument rather than just master.
Colin Guthrie [Thu, 14 Apr 2011 11:00:58 +0000 (13:00 +0200)]
equalizer: Use sink_master as the module argument rather than just 

This brings more uniformity to arguments to match module-echo-cancel
(which needs both sink and source masters, hence the disambiguation).

This will allow other modules to load filters in a more uniform way
in the future without kludges to deal with variation in arguments.

13 years agobluetooth-device: fix rounding errors caused by few bt volume steps
Juho Hämäläinen [Mon, 4 Apr 2011 12:24:17 +0000 (15:24 +0300)]
bluetooth-device: fix rounding errors caused by few bt volume steps

When volume changes in bluetooth device PulseAudio volume is rounded
one too low, so if bluetooth headset changes volume and that volume
is immediately set again for bluetooth device, bluetooth step drifts
lower all the time. Volume is incremented by one in the conversion so
that we get right bluetooth step when re-applying volume.

Signed-off-by: Juho Hämäläinen <ext-juho.hamalainen@nokia.com>
13 years agostream-restore: add version to new entry.
Harri Mähönen [Tue, 5 Apr 2011 08:58:01 +0000 (11:58 +0300)]
stream-restore: add version to new entry.

13 years agodoc: Fix typo
Colin Guthrie [Tue, 5 Apr 2011 08:53:16 +0000 (09:53 +0100)]
doc: Fix typo

13 years agoalsa: Fix log output to inform about positive base volumes correctly.
Tanu Kaskinen [Mon, 4 Apr 2011 11:51:16 +0000 (14:51 +0300)]
alsa: Fix log output to inform about positive base volumes correctly.

This fix was done for _set_port_cb() already, but the first fix didn't fix
setup_mixer(). Now that's done too.

13 years agobluetooth: drop data every 500ms on oor condition
Marc-André Lureau [Mon, 4 Apr 2011 11:33:35 +0000 (14:33 +0300)]
bluetooth: drop data every 500ms on oor condition

13 years agoswitch-on-connect: Add a new module to allow for hotplugged devices to be used by...
Michael Terry [Sun, 3 Apr 2011 12:42:44 +0000 (13:42 +0100)]
switch-on-connect: Add a new module to allow for hotplugged devices to be used by default.

This module implements a simply policy decision that any newly plugged
in devices should be used.

This is a reasonable approach and paprefs will be updated to allow for
this option to be turned on or off.

This is more or less a stop-gap solution. When priority lists are
implemented in the core, then policy modules may ultimately be
re-engineered to adjust the priority lists rather than doing any of
their own routing per-se.

13 years agotests: add a connection stress test
Daniel Mack [Thu, 31 Mar 2011 12:37:40 +0000 (14:37 +0200)]
tests: add a connection stress test

This test is based on a threaded main loop and was written to hunt an
evil race condition.

13 years agobluetooth: Fix HSP volume handling.
Tanu Kaskinen [Thu, 31 Mar 2011 12:00:52 +0000 (15:00 +0300)]
bluetooth: Fix HSP volume handling.

Previously the userdata for the volume callbacks was saved to
pa_core.shared only once when loading module-bluetooth-device, and only when
the SCO over PCM feature was used. That breaks volume handling in cases where
the HSP profile is used without the SCO over PCM setup. Now the userdata is
set always when a sink or source is created, and removed when a sink or source
is removed.

13 years agobluetooth: restore original sco_{sink, src}->set_volume when unloading
Marc-André Lureau [Mon, 28 Mar 2011 12:35:17 +0000 (15:35 +0300)]
bluetooth: restore original sco_{sink, src}->set_volume when unloading

13 years agobluetooth: fix set_volume_cb on sco over pcm
Marc-André Lureau [Mon, 28 Mar 2011 12:35:16 +0000 (15:35 +0300)]
bluetooth: fix set_volume_cb on sco over pcm

The current implementation is totally bogus, it cast the over_sink
userdata to the bluetooth-device userdata... It was failing nicely
because the previous code had a gentle safe-guard in u->profile ==
PROFILE_HSP, and u->profile was just random.

There is no easy way to associate additional data to a sink or
source. Two solutions seems possible: looking up loaded modules and
check which one was handling the sink/source, or using pa_shared. I
went for the second solution.

13 years agobluetooth: use sco_sink/source to start with right state
Marc-André Lureau [Mon, 28 Mar 2011 12:35:15 +0000 (15:35 +0300)]
bluetooth: use sco_sink/source to start with right state

Note from Tanu Kaskinen: I resolved some conflicts with newer upstream code, so
if this patch is broken, blame me..

13 years agobluetooth: Drop all "#ifdef NOKIA" directives.
Tanu Kaskinen [Mon, 28 Mar 2011 12:35:14 +0000 (15:35 +0300)]
bluetooth: Drop all "#ifdef NOKIA" directives.

The #ifdefs only added clutter. I don't see any reason to not compile the
SCO over PCM support in all the time.

13 years agovala: ChannelMap has no destroy function.
Sean McNamara [Wed, 30 Mar 2011 17:41:02 +0000 (13:41 -0400)]
vala: ChannelMap has no destroy function.

13 years agobluetooth: Pull a2dp-codecs.h from BlueZ
Arun Raghavan [Wed, 30 Mar 2011 19:26:20 +0000 (00:56 +0530)]
bluetooth: Pull a2dp-codecs.h from BlueZ

This pulls a2dp-codecs.h from BlueZ which contains the capabilities
structures for SBC and MPEG. We currently have these manually added to
ipc.h, so pulling this header makes our files identical to upstream.

13 years agoalsa-mixer: Check that the kernel driver returns consistent limits with both snd_mixe...
Tanu Kaskinen [Tue, 22 Mar 2011 10:23:19 +0000 (12:23 +0200)]
alsa-mixer: Check that the kernel driver returns consistent limits with both snd_mixer_selem_get_*_dB_range() and _ask_*_vol_dB().

The check is inspired by a driver that returned higher dB limit from
snd_mixer_selem_get_playback_dB_range() than what _ask_playback_vol_dB()
returned at maximum integer volume.

13 years agoalsa-mixer: Make sure that SND_MIXER_SCHN_UNKNOWN isn't used when indexing e->masks.
Tanu Kaskinen [Wed, 23 Mar 2011 10:03:43 +0000 (12:03 +0200)]
alsa-mixer: Make sure that SND_MIXER_SCHN_UNKNOWN isn't used when indexing e->masks.

SND_MIXER_SCHN_UNKNOWN is defined as -1, so that's not a good array index...

13 years agoalsa-mixer: Make probing elements with more than two volume channels fail.
Tanu Kaskinen [Wed, 23 Mar 2011 12:52:48 +0000 (14:52 +0200)]
alsa-mixer: Make probing elements with more than two volume channels fail.

This is just a quick hack to prevent array overflow. Correct fix would be to
implement support for more channels.

13 years agopactl: Add short output format for list action
Maarten Bosmans [Sat, 26 Mar 2011 16:15:43 +0000 (17:15 +0100)]
pactl: Add short output format for list action

13 years agopactl: Separate stat and info actions
Maarten Bosmans [Sat, 26 Mar 2011 16:15:42 +0000 (17:15 +0100)]
pactl: Separate stat and info actions

13 years agopactl: Add subcommands to the list command
Maarten Bosmans [Sat, 26 Mar 2011 16:15:41 +0000 (17:15 +0100)]
pactl: Add subcommands to the list command

13 years agopactl: Accept more volume specification formats
Maarten Bosmans [Sat, 26 Mar 2011 16:15:40 +0000 (17:15 +0100)]
pactl: Accept more volume specification formats

With this you can specify the volume with 6554, 10%, 0.001 or -60dB,
all resulting in the same volume change.

13 years agosink-input: Add volume_writable to pa_sink_input.
Tanu Kaskinen [Sun, 27 Mar 2011 20:00:26 +0000 (23:00 +0300)]
sink-input: Add volume_writable to pa_sink_input.

This is pretty cosmetic change; there's no actual functionality added.
Previously the volume_writable information was available through the
pa_sink_input_is_volume_writable() function, but I find it cleaner to have a
real variable.

The sink input introspection variable name was also changed from
read_only_volume to volume_writable for consistency.

13 years ago.gitignore: add `.tarball-version`
Paul Menzel [Tue, 29 Mar 2011 09:49:52 +0000 (11:49 +0200)]
.gitignore: add `.tarball-version`

`.tarball-version` is created by `.git-version-gen`.

Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
13 years agobluetooth: run `make update-sbc` to pull in build fix for thumb mode
Paul Menzel [Tue, 29 Mar 2011 10:14:27 +0000 (12:14 +0200)]
bluetooth: run `make update-sbc` to pull in build fix for thumb mode

This update pulls in commit c495077c [1] to fix a build error.

        commit c495077cf8a8c37afd90875ec5a5b16b294be15e
        Author: Siarhei Siamashka <siarhei.siamashka@nokia.com>
        Date:   Tue Mar 29 01:57:39 2011 +0300

            sbc: better compatibility with ARM thumb/thumb2

            ARM assembly optimizations fail to compile in thumb mode, but are fine
            for thumb2. Update ifdefs in the code to make use of ARM assembly only
            when it is safe and also make sure that no optimizations are missed
            when compiling for thumb2.

            The problem was reported by Paul Menzel:
            https://tango.0pointer.de/pipermail/pulseaudio-discuss/2011-February/009022.html

This patch is tested with OpenEmbedded using `minimal-uclibc` for `MACHINE = "at91sam9260ek"`.

Note that changes to ipc.h from 8f3ef04b had to be manually reapplied.

[1] http://git.kernel.org/?p=bluetooth/bluez.git;a=commit;h=c495077cf8a8c37afd90875ec5a5b16b294be15e

13 years agomodule-jack-sink/source: protect against null return in jack_get_ports
David Henningsson [Mon, 28 Mar 2011 13:16:12 +0000 (15:16 +0200)]
module-jack-sink/source: protect against null return in jack_get_ports

Just picking up a crash report from Ubuntu, here's the result.

--
David Henningsson, Canonical Ltd.
http://launchpad.net/~diwic

From 934c52c79bb6faed56a64d6e15f9b285f687afee Mon Sep 17 00:00:00 2001
From: David Henningsson <david.henningsson@canonical.com>
Date: Mon, 28 Mar 2011 14:30:44 +0200
Subject: [PATCH] module-jack-sink/source: protect against null return in jack_get_ports

According to jack_get_ports documentation, it seems like returning NULL
is valid, and that it should be freed using jack_free.

Reported-by: Grayson Peddie
BugLink: http://bugs.launchpad.net/bugs/733424
Signed-off-by: David Henningsson <david.henningsson@canonical.com>
13 years agocork-on-phone: Handle sink-inputs with NULL sinks
Arun Raghavan [Mon, 28 Mar 2011 11:09:53 +0000 (16:39 +0530)]
cork-on-phone: Handle sink-inputs with NULL sinks

It's possible that by the time we receive the unlink hook, the given
sink-input's sink is set to NULL. Handle this gracefully.

13 years agoalsa-mixer: Get rid of a compiler warning.
Tanu Kaskinen [Sun, 27 Mar 2011 18:35:03 +0000 (21:35 +0300)]
alsa-mixer: Get rid of a compiler warning.

On 64-bit systems LONG_MAX is greater than the largest possible value of a
uint32_t variable, which caused the compiler to warn about a comparison that is
always false. On 32-bit systems pa_atou() can return a value that will overflow
when assigned to e->volume_limit, which has type long, so the comparison was
necessary.

This dilemma is resolved by using pa_atol() instead of pa_atou().

13 years ago.gitignore: Add ChangeLog to the ignore list.
Tanu Kaskinen [Sun, 27 Mar 2011 16:14:54 +0000 (19:14 +0300)]
.gitignore: Add ChangeLog to the ignore list.

13 years agovala: Fix path error in the last commit.
Colin Guthrie [Sat, 26 Mar 2011 12:42:43 +0000 (12:42 +0000)]
vala: Fix path error in the last commit.

13 years agovala: move GLibMainLoop class into separate file to fix linker errors
Alexander Kurtz [Sat, 26 Mar 2011 11:26:46 +0000 (11:26 +0000)]
vala: move GLibMainLoop class into separate file to fix linker errors

Vala uses the name of the *.vapi file to determine the libraries to link
against. Since the pa_glib_mainloop_*() functions are in a separate
library (libpulse-mainloop-glib.so) the corresponding objects in the
Vala bindings have to be in a separate *.vapi file.

If you are compiling an app without the GLib integration you could use:
 $ valac --pkg=libpulse test.vala
but if you do use GLib you can use:
 $ valac --pkg=libpulse-mainloop-glib test.vala
(libpulse is a dep of the libpulse-mainloop-glib so no need to specify
it explicitly)

13 years agobluetooth: Fix a double-free-esque error introduced in 8f3ef04b
Colin Guthrie [Sat, 26 Mar 2011 00:12:09 +0000 (00:12 +0000)]
bluetooth: Fix a double-free-esque error introduced in 8f3ef04b

13 years agodbus: Do not refcnt the core.
Colin Guthrie [Fri, 25 Mar 2011 23:43:26 +0000 (23:43 +0000)]
dbus: Do not refcnt the core.

We should not call pa_core_ref() anywhere in the code. Doing so
will prevent proper daemon shutdown as the only call (in daemon/main.c)
to pa_core_unref() should always call free_core() and perform a normal
shutdown (i.e. unload all modules gracefully).

13 years agodaemon: Fix regression introduced in f1d1447e.
Colin Guthrie [Fri, 25 Mar 2011 23:03:31 +0000 (23:03 +0000)]
daemon: Fix regression introduced in f1d1447e.

With Tanu's patch, the server no longer starts when a server is configured.
While this is sensible in most circumstances there is a corner case where
we still want to start.

In a typical X11 login, module-x11-publish will be loaded and will thus
set the PULSE_SERVER X11 property on the root window. This then hits the
check introduced in f1d1447e and exits. If PA had previously crashed
(thus leaving behind it's X11 properties) then this means that we will not
autospawn nor even allow ourselves to be started manually until
pax11publish -r is run to clear out the X11 properties. This is obviously
not desirable.

This patch introduces a more in-depth check of the server. If it looks like
a local unix domain socket, then we do not exit straight away and instead
probe further. This should not pose any problems with e.g. remote SSH
usage as the DBus Machine ID is used in the server string.

13 years agodaemon: Fix some more error paths in the double forking.
Colin Guthrie [Fri, 25 Mar 2011 09:12:51 +0000 (09:12 +0000)]
daemon: Fix some more error paths in the double forking.

As spotted by Tanu Kaskinen:

The first process: daemon_pipe is not closed if the first fork() call
fails. Even if it doesn't fail, the first process never closes
daemon_pipe[0].

The second process: daemon_pipe[1] is not closed if anything fails
between the first and the second fork() call. Also, if the second fork
fails, then the finish section writes to daemon_pipe2[1], even though
only the third process should do that. Also, if anything fails between
the first and the second fork, then the second process never writes
anything to daemon_pipe[1]. I don't know what happens in the first
process in this case - does it get an error or does pa_loop_read() get
stuck.

The third process: No problems :)

13 years agobuild: Bump Orc dependency to 0.4.11
Arun Raghavan [Fri, 25 Mar 2011 10:46:56 +0000 (16:16 +0530)]
build: Bump Orc dependency to 0.4.11

0.4.9 errors out at compile time, and might as well bump to 0.4.11 since
that's the version being tested with and has been around for a while
now. Thanks to Paul Menzel <paulepanter@users.sourceforge.net> for
pointing this out.

13 years agovala: delegate FreeCb does not have a target.
Sean McNamara [Fri, 25 Mar 2011 05:28:10 +0000 (01:28 -0400)]
vala: delegate FreeCb does not have a target.

13 years agopo: Remove files no longer in the tree (and which didn't have any translations anyway).
Colin Guthrie [Thu, 24 Mar 2011 22:08:59 +0000 (22:08 +0000)]
po: Remove files no longer in the tree (and which didn't have any translations anyway).

File was removed in c470680e

13 years agovala: Some bugfixes for the vala bindings
Alexander Kurtz [Thu, 24 Mar 2011 21:35:17 +0000 (21:35 +0000)]
vala: Some bugfixes for the vala bindings

 1. PA uses Vala's "Posix" package (see line 23 of libpulse.vapi).
    These dependencies have to be declared in the *.deps file.

 2. Fix obvious copy/paste error.

 3. Rename the parameter to match the C function. This simplifies
    understanding what this parameter means.

 4. According to the official documentation the "dev" parameter
    for playback/capture streams on connection may be NULL and this
    is the default. Change the method definition accordingly.

13 years agodaemon: Fix regression with --start introduced with the double fork in 8e94f653
Colin Guthrie [Thu, 24 Mar 2011 21:27:55 +0000 (21:27 +0000)]
daemon: Fix regression with --start introduced with the double fork in 8e94f653

The previous commit intoduced a double fork which caused a more or less immediate
successful return prior to the hard work of actually starting a daemon.

This patch simply used pipe() to only signal our father when the daemon really
has finished starting.

13 years agolog: Correct bad function implementation
Vincent Becker [Thu, 24 Mar 2011 10:35:02 +0000 (11:35 +0100)]
log: Correct bad function implementation

Replace wrong implementation of log to file in pa_daemon_conf_set_log_level to pa_daemon_conf_set_log_target

13 years agobluetooth: Get rid of warnings about unused stuff when building against a D-Bus versi...
Tanu Kaskinen [Mon, 21 Mar 2011 13:08:35 +0000 (15:08 +0200)]
bluetooth: Get rid of warnings about unused stuff when building against a D-Bus version that doesn't have fd-passing support.

13 years agobluetooth: Don't log an error if an endpoint type is disabled.
Tanu Kaskinen [Mon, 21 Mar 2011 13:08:34 +0000 (15:08 +0200)]
bluetooth: Don't log an error if an endpoint type is disabled.

It's perfectly normal for BlueZ to disable some endpoint types, so printing a
log message at error level isn't a good idea.

For facilitating an informative message in case some endpoint type is disabled,
the send_and_add_to_pending() function interface is also changed to be more
generic (the pa_bluetooth_device pointer is replaced with a void pointer).

13 years agoMove compile-time checks around pa_run_from_build_tree to core-util
Maarten Bosmans [Tue, 22 Mar 2011 15:02:24 +0000 (16:02 +0100)]
Move compile-time checks around pa_run_from_build_tree to core-util

To make the code cleaner and have the checks all in one place.

13 years agowin32: Simplify dl_search_path code
Maarten Bosmans [Tue, 22 Mar 2011 15:02:23 +0000 (16:02 +0100)]
win32: Simplify dl_search_path code

And add #include <sys/stat.h>, needed by the code introduced in f7acd4bd.

13 years agoMerge remote-tracking branch 'zonique/osx'
Colin Guthrie [Thu, 24 Mar 2011 09:08:05 +0000 (09:08 +0000)]
Merge remote-tracking branch 'zonique/osx'