Re: 5.2-BETA dsp.c duplicate lock

On Dec 02, Mathew Kanner wrote:
> On Dec 01, Maxime Henrion wrote:
> [snip]
> 	I believe that your patch should fix the problem.  In general
> I see one of three strategies,
> 
> 1)	Your patch,
> 2)	create a new snd_mtxcreate_chan for channels that sets the
> flags DUP_OK.
> 3)	Fix locking to never hold duplicates.  First glance suggests
> that would be contained in dsp.c, the ioctl handler is the real
> problem and seems inconsistent with itself in regards to locking.
> Ugh.

	Why do the best ideas happen after you send?

	4) Make read and write channel locks of a different class.  

	--Mat

-- 
	In general, a standard is very useful, whether it's de facto
	or du jour.
	- Microsoft's Greg Sullivan
	as misquoted by News.Com