Re: Current ad1848/cs4231 mce_down code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 18 Sep 2007 02:53:28 +0200
Rene Herman <rene.herman@xxxxxxxxx> wrote:


> At (---) we are no longer atomic, so imagine being pre-empted there. When we 
> get back, we may/will well be past end_time and although by now calibration 
> has long finished, we bail out believing we've timed out.
> 
Good catch.

> 
> Moreover, we aren't under lock at all in the cs4231 version (which also 
> means that msleep(1) thing is in fact fine there). Can't we do that for 
> ad1848 as well?
> 

The lock is right thing to do as Trent found out. The in() call writes
to register. On the SMP setup it may backfire (cs4231 does not have
locks but sparc cs4231 has them).

Regards,
Krzysztof
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux