On Mon, 08 Apr 2019 10:55:54 +0200, <twischer@xxxxxxxxxxxxxx> wrote: > > From: Timo Wischer <twischer@xxxxxxxxxxxxxx> > > Some applications do not expect that get_period_size_min() could > return 0. Therefore these applications cannot use the null plugin without > this patch. > Due to there is no use case for having a period size of 0 this patch > disallows a period size of 0 when using the null plugin. > > Signed-off-by: Timo Wischer <twischer@xxxxxxxxxxxxxx> > --- > > Hi Takashi, > > > Why 32? > I thought there is also no use case for a period of 1 frame. Therefore I > increased it to a 2^n value to also lower the CPU usage in case of null > plugin. > > > Or does it cause a problem? > I do not see any issues here. But for example portaudio will automatically > choose a period size of 1 frame which will result in a higher CPU load. > > But anyway I would also be fine with this higher CPU load if you prefer > this patch. Well, this is rather a bug in portaudio, then. Clearly it's nonsense to return zero for the minimal size, but any positive number can be seen as a theoretically valid number. This doesn't mean that any value is usable for the practical use case, because such parameters strongly depend on the other components like the CPU speed, memory speed, whatever. From the same reason, this can't be fixed from the driver or the plugin itself. So, I took now your v2 patch as is. The rest should be a tuning issue. thanks, Takashi _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx https://mailman.alsa-project.org/mailman/listinfo/alsa-devel
