Re: BUG: lock held when returning to user space

[Daniel Walker <dwalker@mvista.com>]

On Wed, 2008-03-12 at 22:40 +0100, Jiri Kosina wrote:

> No, as it is a real bug if you use mutexes in this way. What happens if 
> process that has called open() on your device (and has not closed it yet) 
> calls fork()? 
> Another breakage scenario -- what if the filedescriptor is sent through 
> unix socket to another process? etc.

There's a number of places where a semaphore is used across system
calls. 

for instance the usb skeleton, 
	drivers/usb/usb-skeleton.c

Several of the watchdog drivers,
	drivers/watchdog/s3c2410_wdt.c

These need to be removed, but the usage is clearly not compatible with
the mutex API ..

If you convert them to atomic counts then you loose the sleeping aspect
of the semaphore, which you'd have to add back somehow.

The only API that seems straight forward is using complete's .. Then you
get an atomic count and all the sleeping function calls you might want..
(include/linux/completion.h) The problem with complete's is that you
can't start them out at "1" or "completed" unless you actually run
complete() once during initialization (that's kind of ugly) ..

Daniel