summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * sink: Trivial typo fix in commentArun Raghavan2011-05-021-1/+1
| |
| * alsa-mixer: Remove passthrough profilesArun Raghavan2011-05-023-28/+1
| | | | | | | | | | These aren't used any more - we handle passthrough mode in the iec958* profiles now.
| * core: Fix some FIXMEs for the extended APIArun Raghavan2011-05-022-10/+26
| | | | | | | | | | | | This adds some checks that I'd postponed and adds a "should-be-good-enough" guess for tlength when using a compressed format.
| * sink-input: Kill passthrough streams if moving to an unsupported sinkArun Raghavan2011-05-023-0/+33
| | | | | | | | | | | | This will eventually be replaced by a hook to let clients know that the stream has moved so that they can gracefully reconnect and renegotiate a supported format.
| * sink: Fix leak in pa_sink_check_formats()Arun Raghavan2011-05-021-1/+4
| | | | | | | | We weren't freeing the sink formats idxset.
| * format: Avoid some code duplicationArun Raghavan2011-05-024-14/+12
| | | | | | | | | | | | We frequently need to free an idxset containing pa_format_infos, so define an internal free function that can be used directly with this (instead of defining it once-per-file).
| * sink-input: Don't assert on bad formatsArun Raghavan2011-05-023-17/+21
| | | | | | | | Handles bad format input more gracefully and returns an error instead.
| * sink-input: Return NOTSUPPORTED if format negotiation failsArun Raghavan2011-05-021-1/+1
| | | | | | | | This is easier for clients to grok than INVALID.
| * sink-input: Minor cleanupsArun Raghavan2011-05-021-5/+5
| | | | | | | | | | Removes a couple of warnings and simplifies the assertion logic that verifies format negotiation was successful.
| * tests: Add a trivial test for the extended APIArun Raghavan2011-05-022-0/+203
| | | | | | | | | | | | This is just sync-playback.c modified to use the extended API. We need something more sophisticated for properly testing the compressed formats, but that is a non-trivial task in itself.
| * sink: Remove PASSTHROUGH flagArun Raghavan2011-05-025-59/+40
| | | | | | | | | | | | | | This removes the passthrough flag from sinks since we will drop exclusively passthrough sinks in favour of providing a list of formats supported by each sink. We can still determine whether a sink is in passthrough mode by checking if any non-PCM streams are attached to it.
| * format: Add convenience API to check if a format is PCM or notArun Raghavan2011-05-023-0/+8
| |
| * core: Add extended stream API to support compressed formatsArun Raghavan2011-05-0225-81/+346
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is the beginning of work to support compressed formats natively in PulseAudio. This adds a pa_stream_new_extended() that takes a format structure, sends it to the server (=> protocol extension) and has the server negotiate with the appropropriate sink to figure out what format it should use. This is work in progress, and works only with PCM streams. Actual compressed format support in some sink needs to be implemented, and extensive testing is required. More details on how this is supposed to work is available at: http://pulseaudio.org/wiki/PassthroughSupport
| * sink: Extend API for compressed formats supportArun Raghavan2011-05-022-0/+58
| | | | | | | | | | | | This adds a get_formats() vfunc for sinks to provide a list of formats they can support. pa_sink_check_formats() can be used during or after routing to determine what formats from a stream the sink can support.
| * format: Add some properties and internal APIArun Raghavan2011-05-023-0/+104
| | | | | | | | | | The properties will be used by clients to set the sample format, sampling rate, etc. The functions will be used internally.
| * core: Add a pa_format_info structureArun Raghavan2011-05-028-1/+205
| | | | | | | | | | 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.
| * sample: Use PA_SAMPLE_INVALID instead of numeric valueArun Raghavan2011-05-021-1/+1
| |
| * sink: Trivial typo fixArun Raghavan2011-05-021-7/+7
| |
| * sink-input: Don't resample passthrough inputsPierre-Louis Bossart2011-05-021-12/+14
| |
| * filter-apply: Mark modules as being autoloadedArun Raghavan2011-05-025-11/+28
| | | | | | | | | | | | | | (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.
* | echo-cancel: Remove unnecessary noalign attributeArun Raghavan2011-05-111-1/+0
| | | | | | | | | | | | | | This was just introduced for debugging and should not have been in the final commit. Won't make a difference at the moment since this function is used as a pointer, but removing this in case we change this in the future.
* | echo-cancel: Handle alignment requirement manuallyArun Raghavan2011-05-112-4/+11
| | | | | | | | | | | | | | PA_ALIGNED can't always guarantee that the alignment we want (the GCC man page suggests that the linker might not be able to meet the alignment requirements we desire). Instead, we now allocate some extra memory and guaratee that the alignment we require is met.
* | rtkit: use private bus connection in order to avoid threading issues when ↵Lennart Poettering2011-05-061-1/+2
| | | | | | | | invoking pa_make_realtime()
* | filter-apply: Mark modules as being autoloadedArun Raghavan2011-05-035-11/+28
|/ | | | | | | (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.
* Make connect-stress test compile for win32Maarten Bosmans2011-04-301-3/+3
|
* build-system: Small fixesMaarten Bosmans2011-04-301-14/+10
|
* build-system: Rearrange database selectionMaarten Bosmans2011-04-301-32/+32
|
* build-system: Replace some more conditionals with AS_IFMaarten Bosmans2011-04-301-208/+103
|
* build-system: Move dependency error messages to outer scopeMaarten Bosmans2011-04-301-64/+43
|
* build-system: Move AC_DEFINE to separate line with AS_IFMaarten Bosmans2011-04-301-42/+17
|
* build-system: Use AS_IF macro for configure outputMaarten Bosmans2011-04-301-150/+48
| | | | And add some HAVE_[feature] variables for clarity.
* build-system: Simplify AC_ARG_ENABLE usageMaarten Bosmans2011-04-301-306/+109
| | | | | Make use of the enable_[feature] variable automatically defined by AC_ARG_ENABLE instead of defining our own variable.
* combine: Rename module-combine to module-combine-sink.Colin Guthrie2011-04-303-1352/+1429
| | | | | This also includes a compatibility module that will ultimately be removed in a future release.
* bluetooth: Only close SCO if status has changedLuiz Augusto von Dentz2011-04-301-5/+5
| | | | | 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.
* bluetooth: Fix using pointer-pointer when appending an array as variantLuiz Augusto von Dentz2011-04-301-4/+2
| | | | | pa_dbus_append_basic_array_variant_dict_entry should take the direct pointer to the array.
* bluetooth: Fix not updating sample spec when using Media APILuiz Augusto von Dentz2011-04-301-0/+13
| | | | | | 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.
* x11: More XCB fixes.Arnaud Fontaine2011-04-302-14/+26
| | | | | | | | | | 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.
* dbus: Fix the order of freeing stuff when unloading module-dbus-protocol.Tanu Kaskinen2011-04-291-3/+9
|
* dbus: Fix connection idxset freeing when unloading the module.Tanu Kaskinen2011-04-291-13/+8
| | | | | | | | | | | | | | 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().
* match: Support for both merging and replacing proplist updates.Tanu Kaskinen2011-04-291-37/+60
| | | | | | | | | | | | | | | | | | | | | 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.
* core: Drop empty gaps in the memblockq when playing data from it.Antti-Ville Jansson2011-04-281-5/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* stream-restore: Enable database dumping if DEBUG_VOLUME is defined.Tanu Kaskinen2011-04-281-2/+2
|
* module-null-source: New null-source moduleMarc-André Lureau2011-04-282-0/+300
|
* x11: Fix build errors with newest xcb-util.Maciej Grela2011-04-281-9/+17
| | | | | | 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.
* filter-heuristics: Match roles correctlyArun Raghavan2011-04-231-2/+23
|
* echo-cancel: Play nice with module-filter-*Arun Raghavan2011-04-231-2/+14
| | | | | | | | | | | | | | 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.
* filters: Handle filters on sources as wellArun Raghavan2011-04-232-77/+283
| | | | | | 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).
* filters: Handle stream moves properlyArun Raghavan2011-04-233-16/+79
| | | | | | | | | | | | | | | | | | | | | | | | | 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.
* filter-heuristics: Only apply AEC if we're not already on a phone sinkArun Raghavan2011-04-231-2/+5
| | | | | | 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.
* test: Make the connect-stress less likely to bail out due to >32 streams.Colin Guthrie2011-04-231-1/+9
| | | | | | | | | | 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.