prepare test6

increase timing update interval exponentially

in verbose mode log buffer attr changes

add buffer_attr callback stuff to exported symbol list

fix buffer defaults

be a bit more verbose about max_request changes

Log underruns

handle buffer_attr changed messages properly

update command name table

fix an error where a signal was accidently freed when it is tried to register it twice

Merge commit 'origin/master-tx'

Sending translation for Spanish

Sending translation for Spanish

Sending translation for Spanish

revive solaris module

On Wed, 4 Mar 2009, Lennart Poettering wrote:

[snip]
> > This patch disables link map/library versioning unless ld is GNU ld.
> > Another approach for solaris would be to use that linker's -M option,
> > but I couldn't make that work (due to undefined mainloop, browse and
> > simple symbols when linking pacat. I can post the errors if anyone is
> > intested.)
>
> The linking in PA is a bit weird since we have a cyclic dependency
> between libpulse and libpulsecommon which however is not explicit.

Could that affect the pacat link somehow?

What are the implications for client apps that link with the non-versioned
libraries I've been building on solaris?

[snip]
> >  struct userdata {
> >      pa_core *core;
> > @@ -87,15 +92,24 @@ struct userdata {
> >
> >      pa_memchunk memchunk;
> >
> > -    unsigned int page_size;
> > -
> >      uint32_t frame_size;
> > -    uint32_t buffer_size;
> > -    unsigned int written_bytes, read_bytes;
> > +    int32_t buffer_size;
> > +    volatile uint64_t written_bytes, read_bytes;
> > +    pa_mutex *written_bytes_lock;
>
> Hmm, we generally try do do things without locking in PA. This smells as
> if it was solvable using atomic ints as well.
>
> Actually, looking at this again I get the impression these mutex are
> completely unnecessary here. All functions that lock these mutexes are
> called from the IO thread so no locking should be nessary.
>
> Please don't use volatile here. I am pretty sure it is a misuse. Also
> see http://kernel.org/doc/Documentation/volatile-considered-harmful.txt
> which applies here too I think.

OK, I've removed the locks. For some reason I thought that the get_latency
function was called from two different threads.

> > +static void sink_set_volume(pa_sink *s) {
> > +    struct userdata *u;
> > +    audio_info_t info;
> > +
> > +    pa_assert_se(u = s->userdata);
> > +
> > +    if (u->fd >= 0) {
> > +        AUDIO_INITINFO(&info);
> > +
> > +        info.play.gain = pa_cvolume_avg(&s->virtual_volume) * AUDIO_MAX_GAIN / PA_VOLUME_NORM;
> > +        assert(info.play.gain <= AUDIO_MAX_GAIN);
>
> I'd prefer if you'd use pa_cvolume_max here instead of pa_cvolume_avg()
> because this makes the volume independant of the balance.
>
> > -    info.play.error = 0;
> > +        info.play.gain = pa_cvolume_avg(&s->virtual_volume) * AUDIO_MAX_GAIN / PA_VOLUME_NORM;
> > +        assert(info.play.gain <= AUDIO_MAX_GAIN);
>
> Same here. (i.e. for the source)

Done and done.

> > +            if (u->sink->thread_info.rewind_requested)
> > +                pa_sink_process_rewind(u->sink, 0);
>
> This is correct.
>
> >
> >              err = ioctl(u->fd, AUDIO_GETINFO, &info);
> >              pa_assert(err >= 0);
>
> Hmm, if at all this should be pa_assert_se(), not pa_assert() (so that
> it is not defined away by -DNDEBUG). However I'd prefer if the error
> would be could correctly. (I see that this code is not yours, but
> still...)

Done.

> > +                        case EINTR:
> > +                            break;
>
> I think you should simply try again in this case...

Done.

> > +                        case EAGAIN:
> > +                            u->buffer_size = u->buffer_size * 18 / 25;
> > +                            u->buffer_size -= u->buffer_size % u->frame_size;
> > +                            u->buffer_size = PA_MAX(u->buffer_size, (int32_t)MIN_BUFFER_SIZE);
> > +                            pa_sink_set_max_request(u->sink, u->buffer_size);
> > +                            pa_log("EAGAIN. Buffer size is now %u bytes (%llu buffered)", u->buffer_size, buffered_bytes);
> > +                            break;
>
> Hmm, care to explain this?

EAGAIN happens when the user requests a buffer size that is too large for
the STREAMS layer to accept. We end up looping with EAGAIN every time we
try to write out the rest of the buffer, which burns enough CPU time to
trip the CPU limit.

So, I reduce the buffer size with each EAGAIN. This gets us reasonably
close to the largest usable buffer size. (Perhaps there's a better way to
determine what that limit is, but I don't know how.)

> > +
> > +            pa_rtpoll_set_timer_absolute(u->rtpoll, xtime0 + pa_bytes_to_usec(buffered_bytes / 2, &u->sink->sample_spec));
> > +        } else {
> > +            pa_rtpoll_set_timer_disabled(u->rtpoll);
> >          }
>
> Hmm, you schedule audio via timers? Is that a good idea?

Perhaps not. I won't know until I test on more hardware.

But, given that we have rt priority and high resolution timers on solaris,
I think it is OK in theory...

The reason I used a timer was to minimise CPU usage and avoid the CPU
limit. Recall that getting woken up by poll is not an option for playback
unfortunately. We can arrange for a signal when the FD becomes writable,
but that throws out the whole buffer size concept, which acts to reduce
latency.

> That really only makes sense if you have to deal with large buffers and
> support rewinding.

I've implemented rewind support, but I'm still not sure that I have
understood the concept; I take it that we "rewind" (from the point-of-view
of the renderer, not the sink) so that some rendered but as yet unplayed
portion of the memblock/buffers can then be rendered again?

> Please keep in mind that the system clock and the sound card clock
> deviate. If you use the system timers to do PCM scheduling ou might need
> a pa_smoother object that is able to estimate the deviation for you.

Actually, in an earlier version I did use a smoother (after reading about
that in the wiki). But because of the non-monotonic sample counter (bug?)
I decided that it probably wasn't worth the added complexity so I removed
it. I'll put the smoother back if I can figure out the problem with the
sample counter.

>
> > +    u->frame_size = pa_frame_size(&ss);
> >
> > -    if ((fd = open(p = pa_modargs_get_value(ma, "device", DEFAULT_DEVICE), mode | O_NONBLOCK)) < 0)
> > +    u->buffer_size = 16384;
>
> It would appear more appropriate to me if the buffer size is adjusted by
> the sample spec used.

Done.

> One last thing: it would probably be a good idea to allocate a pa_card
> object and attach the sink and the source to it.

It is possible to open /dev/audio twice by loading the solaris module
twice -- once for the sink (passing record=0) and once for source (passing
playback=0), thus giving seperate threads/LWPs for source and sink. It
might be misleading to allocate two cards in that situation?

> Right now pa_cards are mostly useful for switching profiles but even if
> you do not allow switching profiles on-the-fly it is of some value to
> find out via the cards object which source belongs to which sink.
>
> Otherwise I am happy!
>
> Thanks for your patch! I'd be thankful if you could fix the issues
> pointed out and prepare another patch on top of current git!

No problem. Patch follows. It also includes a portability fix for
pa_realpath and a fix for a bug in the pa_signal_new() error path that
causes signal data be freed if you attempt to register the same signal
twice.

> I hope I answered all your questions,

Your answers were very helpful, thanks.

Finn

>
> Lennart
>
>

Specifying ALSA mixer control

On Fri, Mar 27, 2009 at 7:21 AM, Lennart Poettering <lennart@poettering.net> wrote:

>> I tried installing the latest git sources on my Ubuntu Jaunty box but
>> it just broke sound in all my applications.  For my own purposes, I'm
>> going to need to start with the Ubuntu-patched 0.9.14.  However, if
>> you are willing to accept this patch I will forward port it so that it
>> applies to the latest sources.  It's a completely harmless change, so
>> why not apply it?
>
> Yes, I am happy to apply it. Could you please update it for current git?
>

Great.  An updated patch is attached.  For symmetry, I added this
option to the alsa source module as well.

The Ubuntu folks have customized pulse so much that it is difficult
for me to get this version working on my system.  For this patch I
have only made sure that it compiles.  But it does pretty much the
same thing as the one for 0.9.14, which is working great for me.

Thanks,
Kyle

fix some typos in doxygen comments

these bugged me for a while now.

Maarten

handle failure to parse proplist in cli-command gracefully

explain ff7033c11d9248fe837204b03c8397231dc511fe

Revert "make sure we always read in all properties"

This reverts commit 1c8f968282208be391f062c1656d85e2ba5078ac.

Turns out doing this is racy. The appropriate fix seems to be
38825d79123678bf0c5d156aaea4bedb888a7fcd

allow nofail mode only when no server string was specified

minor cleanups

take org.pulseaudio.Server instead of org.pulseaudio to match the interface name already used in the HAL module

fix compiler warning

make sure we never access an invalid pa_bluetooth_device object

make sure we always read in all properties

get rid of old 'Connected' property parsing and make sure we don't execute two case branches

introduce typedef for pa_bt_audio_state and use it everywhere

Merge commit 'elmarco/bluetooth-fixes'

Merge commit 'elmarco/dbus'

Use pa_source_set_max_rewind_within_thread() for updating the monitor source's max_rewind

gtk-test: updated to use PA_CONTEXT_NOAUTOSPAWN|PA_CONTEXT_NOFAIL

pulse: client connect to dbus

daemon: take org.pulseaudio

dbus: split dbus-util into dbus-shared

modules: fix dbus-util include
pulse: get dbus at context connection

Merge commit 'coling/master'

Merge commit 'origin/master'

rework tunneling code

notify clients about tlength changes

typo fix

add pa_memblockq_apply_attr()/pa_memblockq_get_attr()

document more often the context certain functions are called in

simplify latency config functions a bit and make them callable in more contexts

Trigger move callback a little bit earlier so that no IO thread is running

don't enabled tsched on software ALSA devices

bluetooth: use new audio State properties

bluetooth: GetProperties after profile UUID show up

bluetooth: remove racy GetProperties to check profile

bluetooth: don't access outside array range

bluetooth: fail when switching on non-connected profile

bluetooth: connected can be -1, check > 0

Small fix for MacOSX compat

Issue originally pointed out by Igor Zubkov.

fix typo

Do not reconfigure capabilities.

Query and make use of the current configuration.

Update ipc to match new message headers introduced on BlueZ 4.34.

Fix misuse of 'frame.joint' when estimating the frame length.

'frame.joint' is not the flag for joint stereo mode, it is a set of bits which
show for which subbands channels joining was actually used.

Maintain the original code style for sbc.

raop: Add call to pa_sink_set_max_request()

This is just a copy from the ESD version which is a little crude, but will do for now.

add missing initialization

make sure the discovery module is only loaded once

on monitor source be fine with any latency range set by the sink

add missing whitespace

copy latency flags from sink to monitor source

use u->use_tsched everywhere

fix value of DYNAMIC_LATENCY

fix misplace _ref() calls that should have been _assert_ref()

optionally skip initial frames in backtrace

don't show full so path in backtrace

add simple ref counting debugging framework

trivial simplification

beef up esd sink properties a bit

initialize max_request to SO_SNDBUF

don't fiddle with latency range in sources with static latency

initialize max_request

again, don't fiddle with latency range in sinks with static latency

Don't initialize userdata twice

don't fiddle with latency range because we cannot adjust it dynamically

set latency range only in tsched mode

link jack modules's max_request to the jack buffer size

simplify things and make sure timing setters can be called in most contexts

introduce new flag that marks sinks/sources which can adjust the latency dynamically

make pa_source_set_max_rewind() work similar to pa_sink_set_max_rewind()

get rid of 'default' min/max latencies, simplify things by just having absolute boundaries

bluetooth: fix #NOKIA, correctly unlink sink/src

fix bad memory access when destroying m-b-d

Allow calling pa_{sink|source}_update_proplist() without an actual proprlist

minimal reordering

set request/rewind sizes only via accessor functions

simplify latency range by not allowing stored 'wildcard' ranges anymore

only decrease timer slack, never increase

don't fail when no session bus is available

add pa_assert_cc() for compile time assertions

pulse: check context (do not user pstream when NULL)

only store card profile if flagged for that

enable bluetooth default by support

Merge commit 'origin/master-tx'

Sending translation for po/sr@latin.po

Sending translation for po/sr.po

readd volume control logic