On 09/18/2007 03:57 PM, Rene Herman wrote: Subject was supposed to be as above. > Well, if you insist, but thought I'd submit is seperately once again. > The schedule_timeout() calls in there are so-so, will simply not > schedule, but that > > while (instr->use) > schedule_timeout(1); > > loop is dangerous. There's nothing that I can see that's stopping the > compiler from turning this into an infinite loop: > > if (instr->use) > while (1) > schedule_timeout(1); > > I'll admit I have no idea where this code ends up, but if it's _ever_ > used this seems to not be good. > > Ofcourse, feel free to drop on the floor if you really do insist. > > Signed-off-by: Rene Herman <rene.herman@xxxxxxxxx> Rene. _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
