Re: [issue727] unloading sound driver panics box

["Simon 'corecode' Schubert" <corecode@xxxxxxxxxxxx>]

Matthew Dillon wrote:
> :actually it assumes so only in one place, the uaudio driver, and this is =
> :even a bug which has been fixed in freebsd-7.  so i'd say let's sacrifice=
> : uaudio (seems our only consumer didn't come back) and make the rest work=
> : properly.
>     The design of the audio stuff does not fill me with confidence.
>     I really think we need to stick with high level locks just from
>     a safety and kernel stability point of view, because snafus 
>     will be a lot easier to debug.  Our spinlocks are simply not
>     designed (on purpose) to handle large swaths of code.

I agree, however we definitely want to avoid the interrupt thread blocking when grabbing the lock, because this will stall all interrupt processing on this IRQ.  Spinlocks worked very well since the first import of the updated sound code, so we should keep them for this release.  We can still find something better after the release.

cheers
 simon

--
Serve - BSD     +++  RENT this banner advert  +++    ASCII Ribbon   /"\
Work - Mac      +++  space for low €€€ NOW!1  +++      Campaign     \ /
Party Enjoy Relax   |   http://dragonflybsd.org      Against  HTML   \
Dude 2c 2 the max   !   http://golden-apple.biz       Mail + News   / \

Attachment: signature.asc
Description: OpenPGP digital signature