daemon: make sure pa has its own session and process group, but is not its leader so that we cannot acquire a tty ever

bluetooth: do not hand out access to devices that are not fully configured yet

configure.ac: enable check for CoreAudio

daemon: realpath segfault fix

Lennart,

Apparently I was debugging this at the same time as you. I can't figure out
why my Fedora 11 install with glibc-2.10 has a glibc realpath that doesn't
match the gnu documentation and returns null. But it does.

Your commit aa8ce5bb9b159abb2ffb0f43996340566fc2e9c6 almost fixed my
problem, but it needs a tweak.

Thanks,
David Yoder

Mac OS X: add semaphore implementation

On Wed, Sep 16, 2009 at 11:57:04PM +0200, Lennart Poettering wrote:
> On Wed, 16.09.09 15:15, Daniel Mack (daniel@caiaq.de) wrote:
>
> > +    s = pa_xnew(pa_semaphore, 1);
> > +    MPCreateSemaphore(UINT_MAX, value, &(s->sema));
> > +    pa_assert(s->sema != 0);
>
> Hmm, I'd prefer if the ret val of MPCreateSemaphore() would be checked
> here.
>
> Also I find it a bit weird checking for s->sema, though not
> initializing it to 0 in the beginning. If the call actually failed,
> then the assert will check uninitialized memory. Also, comparing
> pointers with 0 sucks. That should be NULL.
>
> Given that this can not realisitically fail, only in OOM or OOM-like
> situations in which case we abort anyway it mght be enough just writing:
>
> pa_assert_se(MPCreateSemaphore(UINT_MAX, value, &s->sema) == 0);
>
> (Assuming that success is signalled by retval == 0 on MacOSX)
>
> > +void pa_semaphore_free(pa_semaphore *s) {
> > +    pa_assert(s);
> > +    MPDeleteSemaphore(s->sema);
>
> Same here.
>
> > +    pa_xfree(s);
> > +}
> > +
> > +void pa_semaphore_post(pa_semaphore *s) {
> > +    pa_assert(s);
> > +    MPSignalSemaphore(s->sema);
>
> And here.
>
> > +}
> > +
> > +void pa_semaphore_wait(pa_semaphore *s) {
> > +    pa_assert(s);
> > +    /* should probably check return value (-ve is error), noErr is ok. */
> > +    MPWaitOnSemaphore(s->sema, kDurationForever);
>
> And here.

Ok, done. See the patch below.

Daniel

>From 26df2fbae6d9215a3ae084876fb5f79e4d9cf4f0 Mon Sep 17 00:00:00 2001
From: Kim Lester <kim@dfusion.com.au>
Date: Wed, 16 Sep 2009 09:23:39 +0800
Subject: [PATCH] Mac OS X: add semaphore implementation

Wrap clock_gettime and friends

On Wed, Sep 16, 2009 at 11:48:58PM +0200, Lennart Poettering wrote:
> On Wed, 16.09.09 15:15, Daniel Mack (daniel@caiaq.de) wrote:
>
> > From: Kim Lester <kim@dfusion.com.au>
> >
> > OS X does not define clockid_t or clock_gettime() and friends.
> > Add a wrapper to fix this.
>
> Hmpf. I am not particularly happy with this. This adds a lot of
> unnecessary compat code. We don't actually need implementations of
> clock_getres(). All we need is some kind of check whether system
> timers are accurate or whether they are rounded up to scheduling
> slices. On Linux we do that check with clock_getres(), but all the
> information it returns is actually not intertesting at all. We just
> check if this is below some trheshold, that's all.
>
> clock_settime() we don't use at all! We shouldn't carry compat code
> for that.
>
> And clock_gettime we don't really need either. We need some kind of
> accurate system timers (preferably monotonic), and on Linux we use
> clock_gettime() for that. But we already have a fallback there for
> gettimeofday().
>
> Or in other words, the current APIs pa_rtclock_get(),
> pa_rtclock_hrtimer() is supposed to be the abstract API that has
> different backends on different systems. I'd very much prefer if any
> MacOS specific code would simply be plugged in there instead of
> creating various new abstraction interfaces!

Ok - what about the version below? I don't particularily like the

Daniel

>From 9f0a051953ec354ccdb8aa44a9845c408b26ae0b Mon Sep 17 00:00:00 2001
From: Kim Lester <kim@dfusion.com.au>
Date: Wed, 16 Sep 2009 14:40:01 +0800
Subject: [PATCH] Implement pa_rtclock_get() and pa_rtclock_hrtimer() for Darwin

OS X does not define clockid_t or clock_gettime() and friends.
Add wrappers to fix this. Based on a patch from Kim Lester
<kim@dfusion.com.au>.

X11: attach X11 properties to Screen, not Display

git: ignore kde related files

svolume: fix MMX error

We need to sign extend the lower part of the multiplication before adding it to
the higher part. Makes -1 * 0xffff work again.

sink: simplify silence checks

daemon: don't crash if pa_realpath() fails

socket-util: drop redundant casts

socket-util: allocate at least sizeof(sockaddr_storage) space

use pa_fopen_cloexec() where applicable

tdb: use O_CLOEXEC if available

core-util: introduce pa_fopen_cloexec()

core-util: make sure to enable FD_CLOEXEC unconditionally to cope with kernels that silently accept but ignore O_CLOEXEC

use cloexec wrappers wherever applicable

core-util: introduce FD_CLOEXEC wrappers for open/socket/pipe/accept

Merge remote branch 'origin/master'

simd: update test cases

smoother: add comments about optimization recommendations from Jason Newton

libpulse: introduce PA_STREAM_RELATIVE_VOLUME

protocol-native: if a client set volume/mute/device store it since it is user input

libpulse: explain semantics of pa_stream_connect_playback() in more detail

protocol-native: declare that user configured volumes are always absolute

pactl: format cookie a little bit nicer

pactl: include information about client context in pactl stat output

clients: drop definition of BUFSIZE which is unused

libpulse: introduce pa_context_get_tile_size() call

memblock: decrease tile size to 64k again

module-equalizer-sink: drop source executable permissions configure.ac: add enable/disable + summary line for fftw

module-equalizer-sink:
    *fixed SSE2 optimized dsp logic (default if available)
    *cleaned up whitespace formatting (again)

module-equalizer-sink: fixed equalizer state save/restore

module-equalizer-sink: drop source executable permissions configure.ac: add enable/disable + summary line for fftw

device-manager: Fix compiler warning.

Do not use a variable called 'index' as it clashes with string.h

module-equalizer-sink:
    *fixed SSE2 optimized dsp logic (default if available)
    *cleaned up whitespace formatting (again)

module-equalizer-sink: fixed equalizer state save/restore

libpulse: add missing includes

buil-sys: fix build w/o DBus

The current git head does not build without DBus libraries installed.
Does the patch below look suitable?

Thanks,
Daniel

>From f69145fc603c56cef02134ceeba10e1727fa217e Mon Sep 17 00:00:00 2001
From: Daniel Mack <daniel@caiaq.de>
Date: Thu, 8 Oct 2009 14:41:21 +0800
Subject: [PATCH] Makefile.am: fix builds without DBus

Signed-off-by: Daniel Mack <daniel@caiaq.de>

device-manager: Rather than flagging the device as available, just include the sink/source index with PA_INVALID_INDEX meaning unavailable

Fix build when using -fweb, accept both register and memory constraints.

This was reported as Gentoo bug #287391 by Torsten Kaiser, and the fix was
suggested by Mike Frysinger.

Add check for FFTW, and add option to disable it at build-time.

This way there will be a message at configure if FFTW is not found, and
this gets in-line with the rest of the modules' dependencies.

Fix out-of-tree builds when dbus module is enabled.

Merge remote branch 'coling/history'

Merge remote branch 'phish3/master'

Merge remote branch 'tanuk/dbus-work'

position-event-sounds: never position test sounds in space

alsa: disable period event only with tsched=1

loopback: Setting latency of module-loopback

I checked the source code, and latency_msec is missing from the list
of valid module arguments. Attaching a patch to add it.

bluetooth: don't set auto_connect flag when discovering bt devices

device-manager: Play nice with module-stream-restore.

If m-s-r sets the device we let it do so. Otherwise we handle the routing. We run before
module-intended-roles as the priority list will likely be configured appropriately
to do the same job, albeit with manual setup.

device-manager: Make use of PA_IDXSET_FOREACH when applying entries.

device-manager: Keep track as to whether or not the user specifically renamed the device.

If the user has not (via our protocol extension) renamed a device, but it happens to now have
a different name (e.g. module-combine automatically updating the description for us or udev-db
getting better etc.) then make sure we update our cache with this updated version.

If the user has set a name, enforce it's use, even if the description is updated by some other
means (e.g. the user manually editing the proplist or another module doing it for them).

device-manager: No need to check the version after calling read_entry()

Merge branch 'master' into dbus-work

Conflicts:
src/daemon/daemon-conf.c

module-equalizer-sink: disable active profile name restoration as something in pack/unpack is funky and I don't have time for a
proper fix

module-equalizer-sink: *added client initiated sync support for filter state *added note of possible unstable behavior with next-power-of-2 sample rate calculation

module-equalizer-sink: resyncing with head and fix invalid writes
    * pa_log->debug for default equalizer notification
    * partially fixed infinite rewind bug
    * set max_request to window_size first iteration
    * swap order inside ROUND_UP calls
    * resync pa_sink_input_new changes
    * change pa_sample_clamp parameters to be correct to fix invalid writes
    * reenable proper reset logic + proper request size

device-manager: Fix typo in module loading script.

device-manager: Add some scripts that are only run under KDE to load/initialise module-device-manager with routing turned on.

device-manager: Misc fixes to co-exist with other stream management/routing modules.

 * Do not read or set the save_sink/save_source flags. This seems to be for module-stream-restore only...
 * Even if a sink is already set by an earlier module, still move it to the sink we dictate.

device-manager: Misc fixes.

 * Fix a s/sink/source/ copy paste issue when dumping the database.
 * Only show priority list when routing is enabled (as the list is not updated if not)
 * Fix a memory access issue when finding the highest priority sinks/sources
 * key name->device name efficiency fix.
 * Silence noisy debug on reorder - it seems to work :)
 * Reroute after reordering.
 * Initialise preferred lists to PA_INVALID_INDEX

device-manager: Fix the writing of the database when priority doesn't change.

device-manager: Add extra debug messages

device-manager: Change the prefer/defer options to a single 'reorder' command.

We put in the devices from the wire into a hashmap and then add all like type device in the database
and then order them based on priority (with the ones specified on the wire always being in that order at
the top of the list.

device-manager: Don't notify clients on every subscription (it happens all the time).

Also compare the entries fully before saving.

device-manager: Save icon and report current availability over protocol.

This also ensures we let clients know whenver a sink changes in some capacity.
Also correct some debug code.

device-manager: Expose the priority lists in the protocol extension.

Also leave space for 'icon' and 'available' details too, althought currently this info is dummy.

device-manager: Add a function to dump the database which we do whenever we save it (and on startup)

device-manager: Reroute streams when they change allowing the media.role to be updated mid-stream.

We do not handle the _EVENT_NEW subscription here as the PA_CORE_HOOK_SINK_INPUT_NEW/PA_CORE_HOOK_SOURCE_OUTPUT_NEW
hook should handle the initial routing.

device-manager: Refactor the routing method to allow the routing of a single stream

device-manager: More sensible names for internal functions

device-manager: Reroute the streams on startup and update our cache on enable.

device-manager: Keep a cache of the highest priority devices for each role.

Rather than querying our database on every new stream, we keep a cache and only update it when a sink/source is added/removed.

device-manager: Fix the database write mode

device-manager: Update exports

device-manager: Some efficiency and safety tweaks

device-manager: Allow the routing component to be turned on via a module argument as well as via protocol extn.

device-manager: Remove unused variables

device-manager: Set the most appropriate sink/source when new streams are created

device-manager: Add routing functions that are triggered when sinks/soruces are added/removed.

device-manager: Add a function to get a list of the highest priority device indexes for each role.

device-manager: Remove unneeded logic for checking for and (un)loading module-stream-restore. We can co-exist

device-manager: Rough framework (slots etc.) for handling routing.

This is incomplete, it just adds the slots in question and assigns noops to them.
Some minor cleanup of types.

Due to the priority of the hooks, it seems we can actually coexist with module-stream restore so
the code to detect and unload it will be removed shortly.

device-manager: debug and comments

device-manager: Fix the freeing of the datum on prefer/defer.

Also fix a log typo

device-manager: When a new device is encountered, initialise the priority list to an appropriate value

device-manager: Let subscribed clients know when something changes.

device-manager: Change the write function to a rename function.

The structure itself will contain various bits of info so exposing this fully to the client is a bad idea.
By keeping to a rename operation we keep what we do store abstracted from the clients.

Also fix some doxy comments.

device-manager: Provide a method for prefering/defering a device.

This allows clients to edit the priroity order. What is not yet in place is the initialisation of that priority list
when new devices are detected or the cleaning (remove holes) when devices are removed.

In order to keep the storage transparent I will likely remove the write functionality and replace it with a
simple rename method.

I also still need to expose the priority itself when reading the data.

device-manager: Fix copy+paste code that looped over the tagstruct when not necessary

device-manager: Provide a way for clients to enable/disable role-based device-priority routing.

The routing logic itself does not yet exist, but the command currently will unload/load module-stream-restore as approriate.
(module-stream-restore would conflict with the role-based priority-routing).

device-manager: Update docs version -> 0.9.19 (predicted)

device-manager: Fix copy+paste leftover

device-manager: Only store and save details for non-monitor sources

stream-restore: Preventative initialistion to NULL

There is not technically a bug here due to the early return and the knowledge that one of the if blocks
will definitely be run, but this makes sure we don't call free on uninitialised data or do
anything else suitibly daft. Also helps when you copy the code and change it slightly and don't realise
you've left things open...

device-restore: Fix the application of an entry to allow changing the name of devices.

This fixes a few bugs in the copy+pasted implementation of apply_entry()/get_name().

device-manager: Fix tagstruct description extraction (copy+paste blunder)

device-manager: Link native protocol library.