daemon: Drop --module-idle-time from docs

This option does not exist any more.

daemon: Fix *-idle-time arguments

--exit-idle-time and --scache-idle-time were marked as having an
optional argument when the argument is actually mandatory. This causes a
crash when using this argument.

Thanks to Matthijs Kooijman (blathijs on IRC) for pointing this out.

pacmd: dynamically allocate ibuf and obuf

Use pa_pipe_buf to determine the minimum size for ibuf and obuf, taking into
account the two descriptors that use each of them.

See bug #42715

mutex: handle gracefully if a PTHREAD_PRIO_INHERIT protocol cannot be set

This adds an additional check for unavailable PTHREAD_PRIO_INHERIT to the
fallback work done in ca717643ee768307475fc36ea29d920a13db0a8e

See bug #42715

module-rtp-recv: fail when SO_TIMESTAMP is not defined

SO_TIMESTAMP is not POSIX and not available in any platform, so just fail
if the current platform does not have it.

See bug #42715

rtp: use the right type when checking cmsg_type

Use SCM_* instead of SO_* when checking the type of each cmsghdr.

See bug #42715

pipe: use pa_pipe_buf instead of the macro PIPE_BUF

This helps when porting to platforms that lack PIPE_BUF.

See bug #42715

doc: Clarify pa_stream_get_latency() return value

Clarifies that the latency is returned via an inout parameter and the
return value is an error code or 0 on success.

build-sys: Make esound bits optional

This is really quite irrelevant on most modern systems today, and more
so for embedded systems.

core: comment typo

core: fix typo in logging

core: sample_spec.rate is in Hz, not kHz; change logging output

core: fix potential memory leak

build-sys: Fix po/ build with --disable-nls

We can't call IT_PROG_INTLTOOL because it requires intltool even with
--disable-nls, but it provides USE_NLS which is required for the po/
build. This might take a while to get fixed upstream, so working around
this in our build system for now.

build-sys: Fix map-file check

configure doesn't provide an abs_top_srcdir early enough, so we just use
srcdir for that. There was also a problem with the path we were setting
in VERSIONING_LDFLAGS that is fixed now.

svolume_{mmx, sse}, sconv_sse: Fix compilation errors with X32 toolchain

From d8b81d5393df36085009bf9f69d41fa85e2ae58a Mon Sep 17 00:00:00 2001
From: Nitin A Kamble <nitin.a.kamble@intel.com>
Date: Sat, 10 Dec 2011 09:09:06 +0100

Make assembly syntax compatible to the X32 toolchain and fix the
following kind of compilations errors with X32 gcc.

| pulsecore/svolume_mmx.c: Assembler messages:
| pulsecore/svolume_mmx.c:107: Error: `(%esi,%rdi,4)' is not a valid base/index expression
| pulsecore/svolume_mmx.c:135: Error: `(%esi,%rdi,4)' is not a valid base/index expression
| pulsecore/svolume_mmx.c:161: Error: `(%esi,%rdi,4)' is not a valid base/index expression
| pulsecore/svolume_mmx.c:162: Error: `8(%esi,%rdi,4)' is not a valid base/index expression
| pulsecore/svolume_mmx.c:180: Error: `(%esi,%rdi,4)' is not a valid base/index expression
| pulsecore/svolume_mmx.c:210: Error: `(%esi,%rdi,4)' is not a valid base/index expression
| pulsecore/svolume_mmx.c:244: Error: `(%esi,%rdi,4)' is not a valid base/index expression
| pulsecore/svolume_mmx.c:245: Error: `8(%esi,%rdi,4)' is not a valid base/index expression
| make[3]: *** [libpulsecore_1.1_la-svolume_mmx.lo] Error 1

Originally these assembly lines were written for x86_64 ABI, now they
are also compatible with X32 ABI [3][4].

The patch was submitted to the OpenEmbedded-Core list [1][2].

[1] http://lists.linuxtogo.org/pipermail/openembedded-core/2011-December/014189.html
[2] http://git.yoctoproject.org/cgit.cgi/poky-contrib/commit/?h=nitin/x32&id=2d8eec54f755c51f2eff600390f5a4b3cc2a7662
[3] https://wiki.yoctoproject.org/wiki/X32_abi
[4] http://en.wikipedia.org/wiki/X32_ABI

x11: Fix build without NLS support

tests: Fix resampler-test compilation without NLS support

echo-cancel: Better handling of error conditions in test

echo-cancel: Begin log message with uppercase letter

echo-cancel: Set file mode to binary in test code

protocol-native: Fix 'auth-group-enabled' modarg

This is actually implemented in module-protocol-stub as
'auth-group-enable'. An unfortunate typo because the other argument is
spelt as 'enabled', but it's better to be slightly inconsistent than to
change the public interface.

resampler: Move some peak resampler asserts around

This moves a couple of asserts from peak_resample() to peaks_init()
since they're resampler parameters that shouldn't change after
initialisation.

resampler: Remove invalid channel asserts in peak and trivial

The incoming channel count will be fixed up by the remapping code before
the resampler is invoked.

utils: Typo fixes in qpaeq

Pointed out by John Frankish <j-frankish@slb.com> in:

https://bugs.freedesktop.org/show_bug.cgi?id=43733

echo-cance: Make Adrian canceller optional

makes the Adrian echo canceller implementation optional at compile time

this patch supersedes an earlier patch proposal and addresses the following
comments:
* separate patch from speex dependency rework (Arun)
* check that at least one EC implementation is available (Arun)
* properly align yes/no in configure summary for Adrian (Frederic)

build-sys: Make speex library optional

make speex library dependency optional, this affects the resampler
and the echo canceller module

this patch supersedes an earlier patch proposal and addresses the following
comments:
* fix order of pa_echo_canceller_method_t enum and ec_table (Frederic)
* the default resampler is speex if available as before, otherwise ffmpeg (Arun)
* does not touch the Adrian EC implementation (see separate patch) (Arun)

utils: Fixes for building without NLS

iochannel: Handle missing un.h correctly

mime: Move assert to correct position

build-sys: Use absolute path for map-file while linking

This makes the link process more robust in some stranger configurations.

build-sys: Fix building without NLS

stream: Fix typos and formatting in stream.h doxygen documentation

mainloop: Fix typos and rewording of thread-mainloop.h doxygen documentation

simple: Fix typos in simple.h doxygen documentation

stream: Fix 'e g' as 'e.g.\ '

error: Fix spelling of 'initialization' in errortab for PA_ERR_MODINITFAILED

Also update the related translations.

proplist: Consistently use parameter name p for a pa_proplist* in prototype

utf8: Fix doxygen file description for utf8.h

proplist: Fix typos in doxygen documentation

proplist: Add doxygen file description for proplist.h

introspect: Fix unresolved doxygen link pa_port_available_t

format: Add description for format.h

format: Add periods to fix brief documentation for pa_format_info_is_pcm()

format: Fix unknown doxygen command \second

def: Document that pa_buffer_attr.maxlength is in bytes.

Fixes https://bugs.freedesktop.org/show_bug.cgi?id=42919

stream: Fix comments

pstream: Fix spelling of 'receive'.

These functions are not exported in the map file and thus do
not form part of the API and can be corrected without any problem.

alsa-mixer: Remove unused pa_alsa_path_set_probe() declaration.

device-port: Add a property list to ports.

They're not used for anything yet, but hopefully soon they'll be.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

pactl: Show port availability

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

Fixup a few things in the new path probing

 * If mapping_probe_paths() fails to open the mixer, all paths are now
removed from the mapping's path sets.
 * pa_path_set.probed isn't really used for anything (removed).
 * If profile probing is configured to be skipped, mapping_paths_probe()
should still be called.

Thanks to Tanu for spotting.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

alsa: add card ports and path probe cache

To be able to add ports to all profiles, we need to probe all
profiles at startup. To speed this up, we now have a cache of
probes paths which is owned by the profile set. Since paths
are now owned by the profile set, the path set must now have
a hashmap of paths instead of a linked list.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

alsa: Ignore the virtual "thinkpad-acpi" card

The thinkpad ACPI driver sometimes creates a virtual sound card,
which at best exposes a volume control. Save some startup time, and
unnecessary error messages in the log, by ignoring it.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

echo-cancel: Fix memory leak in test program

memory for pa_echo_canceller is allocated in init_common() a second time,
hence the allocated memory in main() is leaked

resampler: Some minor tweaks

that were forgotten in the previous peaks resampler commits.

echo-cancel: Fix warning for undefined HAVE_WEBRTC

tests: Don't link tests that only use public api to libpulsecommon

resamplers: Improve performance of peaks resampler

This is mainly achieved by special-casing the common 1ch float case,
which is used by applications such as pavucontrol.

resamplers: Rework the peaks resampler

The algorithm had been implemented the same way as the trivial resampler.  But
an important difference between the two is that the trivial resampler can write
an output as soon as the first corresponding input sample is seen, whereas the
peaks resampler must have read all input samples before writing an output
sample.

With this rework, the peaks resampler now outputs samples correctly when the
input data is spanning multiple memblocks.

shm: Use a goto rather than early return for consistency.

This is functionally the same but is easier to understand.

win32: Avoid some compiler warnings when cross-compiling for mingw32

Autoconf documentation says that AC_FUNC_SELECT_ARGTYPES shouldn't be used anyway.

device-port: Remove redundant include after the better circular dep fix.

x11: Drop unneeded 'struct' and use the typedef directly.

Use simple free function in pa_queue_free

Use simple free function in pa_dynarray_free

device-port: Fix the circular dependency problem more cleanly.

Declaring structs before including anything ensures that the
structs are available for everybody in case of circular includes.

bluetooth: Fix property reply handling for hfgw

resamplers: Optimize trivial resampler

This improves the performance of a typical s16 2ch resampling by 88%.

resamplers: Use better variable name in trivial resampler

build-sys: Disable fastpath asserts by default

resamplers: Use fastpath assert in trivial resampler

When the assert is disabled, the trivial resampler gets a 35% performance boost.

pulsecore: Hardcode FRAME_SIZE_MAX_ALLOW

Instead of using PA_SCACHE_ENTRY_SIZE_MAX, the size for FRAME_SIZE_MAX_ALLOW is
set directly to the same value.  This removes the need for the core-scache.h
include, which caused an unwanted dependency of libpulsecommon on libpulsecore.

Fixes https://bugs.freedesktop.org/show_bug.cgi?id=41539

Update LICENSE

Mention gdbm, lirc and fftw there.

pulsecore: Fix issue with circuilar definitions.

Without this fix, errors about previous definitions are generated in
numerous locations.

Example of error:

  CC     libpulsecore_1.98_la-auth-cookie.lo
In file included from ../../src/pulsecore/source.h:46:0,
                 from ../../src/pulsecore/sink.h:40,
                 from ../../src/pulsecore/core.h:50,
                 from ../../src/pulsecore/shared.h:25,
                 from ../../src/pulsecore/auth-cookie.c:33:
../../src/pulsecore/device-port.h:40:24: error: redefinition of typedef
'pa_core'
../../src/pulsecore/core.h:29:24: note: previous declaration of
'pa_core' was here
make[3]: *** [libpulsecore_1.98_la-auth-cookie.lo] Error 1

Overall it would be nicer if we could avoid this kind of fix, but it
would require further reorganisation that I'm not prepared to undertake
right now.

cli: Show card ports and jack detection status

Expose the new stuff through pacmd.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

Notify port available status changes, and update protocol version

The recommended way of setting available status is to call
pa_device_port_set_available, which will send a subscription event
to the relevant card. It will also fire a hook.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

Cards now has ports directly, and device port has list of profiles

This forms the base for being able to expose all ports of all
profiles (even inactive ones) to clients.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

Turn device ports into reference counted objects

Since both cards and sinks can hold references to a port, it makes
sense to reference count them. Although no current implementation
actually has sinks with ports but without a card, it felt wrong
to make it harder to make such an implementation in the future.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

Introduce "available" concept for ports, and communicate that to clients. Bump protocol version to 24.

Note: There is still no notification when status availability changes.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

filters: Handle echo-cancel streams better

This makes handling of echo-cancel streams (and potentially other
filters that need to work with multiple streams) cleaner. We do this by
adding the ability to apply filter on a group of streams rather than
always treating streams individually.

This is required, for example, when changing the input source for an
echo-cancel source output. When this happens, we want to change the
module-echo-cancel instance to actually cancel between the new source
and old sink.

To do this when one stream in a group moves, we create a new filter
instance between the current master sink/source pair and move the
relevant streams to the virtual sink/source from this new instance.

proplist: Add internal API to get stream group

This moves out code from module-stream-restore and makes an internal API
out of it to get a "stream group" for a given sink input or source output.
This is factored out for reuse in module-filter-*.

The stream group basically provides some means of attaching a logical
identification to the stream (by role, application id, etc.).

filters: Allow a filter to have both sink and source

This sets the base for better representing module-echo-cancel. This
patch itself should change no behaviour -- it only makes way for other
changes.

filters: Fix the master source/sink when autoloaded

When autoloaded, it is expected that module-filter-apply (or whatever is
loading us) will take care of applying the filter on the correct
sink/source master. Instead of adding complexity by tracking what is
currently being filtered, we just disallow filtering anything except the
original master sink/source and let module-filter-apply or whatever is
loading us deal with dynamic sink/source changes.

echo-cancel: Use more human-friendly descriptions

This makes what devices are being cancelled clearer in the UI (at the
cost of being somewhat less clear when multiple devices of the same name
are plugged, but at least that's a much smaller set than everyone).

echo-cancel: Use speex by default if webrtc isn't available

Thanks to Peter Meerwald <pmeerw@pmeerw.net> for pointing this out.

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=42925

core: Look up /etc/machine-id if D-Bus machine-id is not found

It appears that this is currently the fallback for early boot and other
such cases where /var might not be available. Relevant upstream commit:

http://cgit.freedesktop.org/dbus/dbus/commit/?id=66e52541d5bdd4927a5c702963749760643313f4

Thanks to Samuli Suominen <ssuominen@gentoo.org> for pointing this out
in https://bugs.gentoo.org/show_bug.cgi?id=390287

oss: Use log2 function from core-util instead of own implementation

Squash unused variable compiler warning

Remove vector.h and vector-test

This code was never used and vectorization is handled by Orc anyway.

build-sys: Remove time-smoother and shm from libpulsecore

As they are already included in libpulsecommon.

rtpoll: Update comment

to reflect changes made in
32e2cd6d3216f780c4cffed0f8eb3c30f2c8d732
core: get rid of rt sig/timer handling since modern Linux' ppoll() is finally fixed for granularity

stream: Fix upload samples' cleanup

In pa_create_stream_callback, a stream is inserted into
s->context->record_streams only if it's a record stream. Otherwise it's
inserted into s->context->playback_streams. However, in stream_unlink the
stream is removed from s->context->playback_streams only if it's a playback
stream and otherwise it's removed from s->context->record_streams.

Thus, if the stream is an upload stream, we first insert it into
s->context->playback_streams in pa_create_stream_callback and then try to
remove it unsuccessfully from s->context->record_streams in stream_unlink. This
means that we are leaking hashmap entries until the context is freed,
constantly consuming more memory with applications that upload and unload a
large number of samples through one context.

Of course, this begs the question whether upload streams even belong in either
of those hashmaps. I don't want to mess around with the code too much at this
point though, so this patch should be a sufficient improvement.

alsa: Minor debug code cleanup

Remove a trap on underrun and uncomment a couple of useful debug prints
(still enabled only when DEBUG_TIMING is defined).

tests: Add playback tests to check-daemon

Adds the standard/extended API playback tests to the check-daemon set.

core: Make debugging a bit simpler

This takes out a bunch of commented debug prints and puts them in
defines. Makes it easier to turn them all on/off at a single point.

tests: Add a target for running manual tests

This adds a check-daemon make target which requires a running PulseAudio
server to test against.

echo-cancel: Make WebRTC the default canceller

Works pretty well, and isn't a CPU hog, so let's use it!

echo-cancel: Turn WebRTC analog gain control on by default

Seems to be pretty well-behaved.

echo-cancel: Hook up WebRTC analog gain control