[MPlayer-dev-eng] ALSA issue, r29549

[Clemens Ladisch]

Reimar Döffinger wrote:
> that commit caused quite a few issues, see e.g.
> http://bugs.gentoo.org/show_bug.cgi?id=286020

The mplayer output in the referenced mail shows that the new code
chooses a buffer size of 16384 frames, which is perfectly sane and
happens to be exactly the same size that the old code would have
chosen.

The only difference is the period size (new: 2048, old: 1024 frames).

> If nobody comes up with an explanation or a fix I threaten
> to revert this.

Apparently, ao_data.outburst being 12288 instead of 6144 bytes trips up
the audio playback code.  (Unfortunately, the assumption in
http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/2009-August/062150.html
seems not to be entirely true.)

It would be possible to reduce the period size, but without knowing what
the actual constraints on outburst are, this would be just guessing and
can break in other circumstances, so it might be better to revert it.


Best regards,
Clemens