module: Fix crash in pa_module_unload_all()

Previously, if there were no modules loaded when the daemon exited,
pa_module_unload_all() would crash due to giving zero count to
pa_xnew().

Thanks to Pierre Ossman for the patch.

Bug-Tizen: TZPC-4359

set conflicting modules with murphy behind conditionals

these modules should not be loaded with module-murphy-ivi

multilib support: re-add the baselibs.conf to packaging

The baselibs.conf is required to support multilib packages; currently,
pulseaudio-32bit and libpulse-32bit are required for the Steam
client and various Steam games.

Change-Id: Ia489433b3d08363320bf7546aebe229d96c5cfcc
Signed-off-by: Patrick McCarty <patrick.mccarty@linux.intel.com>

remove pulseaudio.service which launch pa in system mode

Change-Id: I48038cd35b05d395cfaf5f9a167ecb2261fe253c

check module-fileter-xx.so exists to avoid module load failure

Change-Id: Id6384e358ed3bac62d14874e8fe22e8792a637d9

Remove the toplevel manifest

There is already a manifest in packaging/, so the toplevel one should be
removed.

Change-Id: I38603c91e195b93bae16a561193a4b8a6d2e026d
Signed-off-by: Patrick McCarty <patrick.mccarty@linux.intel.com>

resetting manifest requested domain to floor

Change-Id: Idd9eed5f535b8cfad5610f9b4f39e0422d6e86e5

release: updated changelog.

release: added packaging for gerrit.

Change-Id: Iae038b2a9b8034d37a8bd88f71ff645d66445266

packaging: added obs export script

packaging: adding pulseaudio manifest for smack

packaging: added pulseaudio.spec.in

packaging: added pulseaudio.service

combine: fix crash in output freeing - upstream patch

Change-Id: I036147804f985721fe1d06b2f8eb423f11b34848

Add bt_profile_set patch which fixed bt a2dp&hsp profile setting issues in mobile

Change-Id: I3a83fa96cd5ed9d22dbb7630cc1cc79414a8845e

use udev only for usb devices - samsung

cope with possible infinite waiting in startup - samsung

set alsa suspend timeout to zero - samsung

create pa_ready file - samsung

add bluetooth a2dp aptx codec support - samsung

Change-Id: I14ed8c1dfc1031afcd8428b8951aad63a132ea8f

add policy module - samsung

Change-Id: I50921a731805f180cff2feffff240c0808124685

add support for dlog - samsung

Change-Id: Id795c76941710aaf7c984f375f0f0e51297ada50

Enhance for echo cancel - samsung

Change-Id: I625ca676228b58b5d53f72b9d14b3e811d212d1e

Add preload fileter for resample - samsung

Change-Id: Ia57b4eb1848d53bbfa15c04b25f4821554aff28f

add support for samsung power management - samsung

Change-Id: Id6efaab5a9888fc5eec2883c6be35090f6df375c

changes to pa simple api - samsung

Change-Id: I997c02217a8dc14524480164aa0baeea901c7b4e

adjust default bluetooth profile to off

add conditional murphy module loading to default.pa.in

Change-Id: I8199f4530ef5f10cf65488969b3a8998e05cb65c

fix draining not to lose prebuffered samples

add internal corking state for sink-input

node-manager: adding node support for pactl

node manager: adding external node manager API

make pa_thread_mq_done() safe for subsequent calls

jack detection fix for wired headset

build-sys: install files for a module development.

volume ramp: add client api support for volume ramping

volume ramp: adding volume ramping to sink-input

volume ramp: add volume ramping to sink

volume ramp: additions to the low level infra

tests: Make loopback latency test more accurate

This makes sure that we always take the timing at the point when we
write out the pulse, making the overall latency measurement more
accurate.

tests: Factor out loopback setup code

This moves over setup code for the loopback latency test into a private
library so that we can easily write more tests using the same framework.

tests: Adjust latency test calibration

This makes the test more robust by:

1. Decreasing the '1' threshold during calibration - the RMS value for
   the sine wave will be 0.5, so the previous code was making us take
   the ALSA mixer past 0dB.

2. Using the difference rather than absolute value for 0->1 transitions,
   so that we're somewhat independent noise in our calculations.

tests: Make loopback latency test run with rt priority

tests: Add a latency measurement test

This test is intended to measure real latency by playing a sample to a
sink and capturing that over a loopback interface. The loopback can
either be physical (cable running from headphone out to line in) or
virtual (monitor source or module loopback).

Also included in this is calibration code to make sure that volumes are
sufficiently adjusted to be able to detect the played back signal (and
that there aren't false positives due to line noise).

One of the objectives of all this is to later factor out the setup code
to allow us to easily write more loopback tests for various
functionality (volumes, resampling, mixing, etc.).

bluetooth: Fix input port description

zeroconf: Make Avahi usage in m-z-publish async

This pushes all avahi-client code to a threaded mainloop from the PA
mainloop context. We need to do this because avahi-client makes blocking
D-Bus calls, and we don't want to block the mainloop for that long.

The only exception to this now that I don't see a workaround for is
during module unload time. However, this shouldn't be a huge problem
since in most cases, this will only happen at server shutdown time.

The bulk of the change is partitioning the data so that PA core objects
only (well, mostly) get accessed in the PA mainloop and Avahi calls
happen only in the Avahi threaded mainloop.

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

mainloop: Add API to set thread name for threaded mainloop

pulsecore: Fix assert in pa_msgobject creation

Allows for creation of derived types that don't have any other fields.

shell-completion: Add list-cards to the pacmd completion.

The list-cards command was missing for both of the shell completions.
This fixes it.

rescue-streams: Fix wording of module description

Signed-off-by: Peter Meerwald <p.meerwald@bct-electronic.com>

intended-roles: Fix typo in module description

Signed-off-by: Peter Meerwald <p.meerwald@bct-electronic.com>

pactl: Use colon to separate priority in profiles output

for example:
  Profiles:
input:analog-stereo: Analog Stereo Input (sinks: 0, sources: 1, priority. 60)
output:analog-stereo: Analog Stereo Output (sinks: 1, sources: 0, priority. 6000)

it should be "priority: xxx", not "priority. xxx"

Signed-off-by: Peter Meerwald <p.meerwald@bct-electronic.com>

thread-mainloop: Fix bug in example code

Checking the operation state caused a deadlock, because the state
won't change before my_drain_callback() returns, and it doesn't
return before my_drain_stream_func() calls
pa_threaded_mainloop_accept().

cli: Fix plural in caption of source-outputs list

$ pacmd list-source-outputs
>>> 4 source outputs(s) available.

outputs(s) makes no sense

Signed-off-by: Peter Meerwald <p.meerwald@bct-electronic.com>

bluetooth: Support transport auto-release

With BlueZ 5, if the remote device suspends the audio, the transport
state will change to "idle" and the endpoint is not required to release
the transport, since this could introduce race conditions. Therefore,
ignore the call to pa_bluetooth_transport_release() if the transport is
not acquired any more.

bluetooth: Update to new BlueZ 5 transport acquire/release API

The new D-Bus API doesn't support access rights, which weren't used by
PulseAudio anyway, but it does solve a race condition: now optional
acquires can be implemented by bluetooth-util atomically using the D-Bus
TryAcquire() method.

bluetooth: Support media transport's State property

BlueZ 5 exposes a 'State' property in the media transport interface.
With regard to PA, this replaces the profile-specific interfaces, since
they were being used to know if the audio was streaming or not.

bluetooth: Parse media transport's properties

Add the code to parse the properties of the media transport object when
a PropertiesChanged signal is received.

Note that the transport might have an owner other than BlueZ, and thus
the property changes would be emitted from arbitrary senders. For
performance reasons, the installed match considers the interface name
where the property has changed.

It could be possible to install and remove the D-Bus matches dynamically
when a new owner is registered/unregistered, but filtering based on the
interface name seems good enough already.

bluetooth: Support Properties.PropertiesChanged signal

Install matches for signal Properties.PropertiesChanged and process the
properties corresponding to the tracked devices.

bluetooth: Support ObjectManager interface add/remove

Install matches for signals ObjectManager.InterfacesAdded and
ObjectManager.InterfacesRemoved, and process the devices that are
registered and unregistered dynamically.

core: Add an "internal" suspend cause

This lets us suspend devices from within the core for short periods
without having to overload one of the existing suspend causes.

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

pactl: Flush stdout buffer when printing subscribe events.

"pactl subscribe" is running continuously, and without flushing its output is
not usable for "process-on-arrival" per-line tasks, such as grepping. This
patch should fix this. For example, now:
pactl subscribe | grep 'server'
should print only server events as they arrive.

bluetooth: BlueZ 5 interface rename to org.bluez.MediaTransport1

Use the new interface name if BlueZ 5 has been detected.

bluetooth: BlueZ 5 interface rename to org.bluez.Media1

Use the new interface name if BlueZ 5 has been detected.

bluetooth: BlueZ 5 interface rename to org.bluez.MediaEndpoint1

Use the new interface name if BlueZ 5 has been detected.

bluetooth: Fix error checking style

bluetooth: Parse the tree returned by ObjectManager

Parse the result of ObjectManager.GetManagedObjects(), which includes
all objects registered, their interfaces and the corresponding
properties per interface.

bluetooth: Detect BlueZ 5

Check the existence of ObjectManager to detect the version of the
running daemon. If the interface exists, it should be BlueZ 5.

module: Assign the index before calling init()

Any code that runs inside the init() callback sees an invalid module
index. Sometimes init() does things that cause hooks to be fired. This
means that any code that uses hooks may see an invalid module index.
Fix this by assigning the module index before init() is called.

There are no known issues in the upstream code base where an invalid
module index would be used, but an out-of-tree module
(module-murphy-ivi) had a problem with this.

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

bluetooth: Remove the 'bluez.name' property

The 'Name' property of the Device interface became optional in BlueZ 5
and may not be present anymore (that happens when testing against the
PTS 4.7.0), so it's better not to expose it to clients so they don't
rely on its existence.

bluetooth: Use 'Alias' value as the device description

The 'Alias' property should be preffered over the 'Name' property,
according to the BlueZ API documentation.

bluetooth: Add 'bluez.alias' property

.gitignore: Ignore src/*.gcno files created when building with --enable-gcov

Signed-off-by: Peter Meerwald <p.meerwald@bct-electronic.com>

switch-on-port-available: Silence gcc warning.

This silences this gcc warning:
module-switch-on-port-available.c:111:12: warning:
    'good' may be used uninitialized in this function

module: initialize module index to invalid value.

m->init() was called while m->index was uninitialized, which was bad
style.

fdsem: Remember pa_write() type in pa_fdsem_post()

pa_write() knows two types of operation:
calling send() and calling write()

there is a flag (a pointer to an int) passed to pa_write()
which can remember which write type was successful

if the pointer is NULL or the int is 0, send() is tried first,
with a fallback to write() if send() resulted in ENOTSOCK

pa_fdsem_post() calls pa_write() with a NULL pointer;
unfortunately (at least with HAVE_SYS_EVENTFD_H #define'd) send()
always fails here and write() is called -- causing an extra syscall
quite frequently

strace:
send(17, "\1\0\0\0\0\0\0\0", 8, MSG_NOSIGNAL) = -1 ENOTSOCK (Socket operation on non-socket)
write(17, "\1\0\0\0\0\0\0\0", 8) = 8

the patch adds a write_type field to pa_fdsem to the successful
pa_write() type can be remembered and unnecessary send() calls are
avoided

Signed-off-by: Peter Meerwald <p.meerwald@bct-electronic.com>

pasuspender: Resume before exiting in case of SIGINT or fork() failure.

Pressing Ctrl-C in a terminal while pasuspender is running
causes the sinks and sources to stay suspended after
pasuspender has exited, which is very annoying. This patch
fixes that problem, and also a similar problem with fork()
failures.

alsa: Don't use pa_strna() for port names

The name variable is never NULL, so there's no point in using
pa_strna().

core-util: Don't accept random words in pa_parse_boolean()

The old code accepted any word that started with "y", "Y",
"n", "N", "t", "T", "f" or "F". Fix this by having
a whitelist of full strings instead of checking just the
first letter.

build: make ARM NEON check in configure.ac more strict

the check for NEON so far only checked if -mfpu=neon is understood by the compiler,
however, this is not enough:

(i) #include <arm_neon.h> should be checked
(ii) -mfpu=neon must be passed before CFLAGS because eventually the per-library CFLAGS
for NEON code in src/Makefile.am are passed to the compiler before the global CFLAGS

in case the build environment passes CFLAGS to configure and we try to set conflicting
CFLAGS option, the former take precedence; CFLAGS cannot be overridden

this does not fix
http://lists.freedesktop.org/archives/pulseaudio-discuss/2012-December/015570.html
but at least makes the build fail in configure (and not while compiling stuff)
and gives better diagnostics

Signed-off-by: Peter Meerwald <p.meerwald@bct-electronic.com>

device-port: Introduce pa_device_port_new_data

Port creation is now slightly different. It is now similar to how
other objects are created (e.g. sinks/sources/cards).

This should become more useful in the future when we move more stuff to
the ports.

Functionally nothing has changed.

.gitignore: Update m4/ ignore list

build-sys: Use upstream gettext instead glib one

bluetooth: Improve code and log readability

This commit makes the code cleaner, avoiding unnecessary line breaks. It
also changes the debug message elements order, to make it look more
natural ("path, interface, member" instead of "interface, path,
member").

bluetooth: Remove unnecessary braces

device-port: Make it impossible to have dual-direction ports

alsa-mixer: Remove the "name" option from the "General" section of path configuration files

This means that the path names will always correspond to the
path configuration file names, so they will automatically be
unique (in the scope of one card).

alsa-mixer: Introduce "description-key" option for paths

Previously the path description was looked up based on the
path name only. Since there can be multiple paths that use
the same description, it had to be possible to have multiple
paths with the same name.

Having the same name with multiple paths makes identifying
the paths more complex than necessary, so the plan is to
make it impossible to have paths with the same name. This
patch prepares for that by retaining the possibility to
still have the same description with multiple paths. Instead
of the path name, the path description is looked up by using
the "path description key" if it is set (path name is still
used as a fallback lookup key).

switch-on-port-available: Fix error reporting style

switch-on-port-available: Remove a redundant function argument

switch-on-port-available: Prepare for dual-direction ports going away

As an extra, I broke try_to_switch_profile() into smaller
functions, because the two levels of loops with continue
statements inside both were a bit hard to follow.

build-sys: Bump soname

Add a NEWS file

Short version based on Tanu's longer notes posted on list (and now on
the wiki, linked from this file.

osx: Fix latency querying

get_latency_us() used an uninitialized variable, and an incorrect
scope for some of the AudioObjectGetPropertyData() calls. As a result,
audio would randomly not work at all.

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

build-sys: Bump soname

We dropped pa_format_info_free2 as an exposed symbol, but this was never
exposed via headers, so not counting this as ABI change.

rtp: Reset timestamp with correct size if not found

This fixes a compiler warning when building under Ubuntu.

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

module-device-manager: Fix description restore

e->description is a pointer, not a fixed char array. Hence it
makes no sense to use strncmp.

This fixes a compiler warning when compiling under Ubuntu.

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

core-util: Add pa_safe_streq

Like pa_streq, but does not blow up on NULL pointers.

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

sink-input,source-output: Deal with FIX* flags and extended API

The sample spec fixup when FIX* flags are set was not being propagated
to the pa_format_info, causing the two to be out of sync when FIX* was
used.

bluetooth: Fix missing port->profile association

Commit 17b3cb954b179392e80b0a46d8f2ba4693aec386 merged Bluetooth ports
into two ports (one for input, one for output) but the association
between ports and profiles was lost.

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