Re: [PATCH] rme96 add stream synchronization and PM support

[Knut Petersen <Knut_Petersen@t-online.de>]

On 15.08.2013 08:22, Takashi Iwai wrote:
> At Thu, 15 Aug 2013 08:01:05 +0200,
> Knut Petersen wrote:
>> On 14.08.2013 17:06, Takashi Iwai wrote:
>>> Please resubmit only the second patch with the fixes.
>> Ah, yes. That should have been obvious.
>>
>> After a 2nd thought I believe that  it is an even better idea
>> not to abort but to handle that error condition graciously.
> Why?  It's a serious error.

The probability that two 64k vmallocs will fail is pretty low,
and the probability that after that failure there is enough memory
for an audio application to be executed successfully is even lower.

Nevertheless, in that unlikely case I would prefer max 0.37 seconds
of silence (=skipped audio) during resume to a failed start of a
recording caused by an unloaded module. Limited functionality
is better than complete failure.


cu,
  Knut