[vlc-devel] [PATCH] ALSA: set periods before setting buffer time

[Edward Sheldrake]

2012/3/11 Rémi Denis-Courmont <remi at remlab.net>:
> Le dimanche 11 mars 2012 17:16:04 Edward Sheldrake, vous avez écrit :
>> Fixes "cannot set buffer duration near 3000000 us: Invalid argument"
>> error, resulting in no sound, with HDA audio hardware and 44100Hz sources.
>
> Err? Is this really correct and not a driver bug? This reorder seems to put an
> extra constraint before selecting the buffer size. This may cause a smaller
> buffer size to be selected than would be available with more periods, if I
> read the documentation correctly.
>
Perhaps it should be an ALSA bug - without the patch, ALSA chooses a
buffer time of (3000090 3000091) which it then finds unacceptable
(alsa fails to refine the remaining parameters), with the patch it
chooses a buffer time of (2999727 2999728).