cvs commit: src/sys/dev/fdc fdc.c

John Baldwin jhb at freebsd.org
Fri Jan 11 09:13:09 PST 2008


On Friday 11 January 2008 11:50:53 am Konstantin Belousov wrote:
> kib         2008-01-11 16:50:53 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/dev/fdc          fdc.c 
>   Log:
>   The wakeup() line from the rev. 1.319 is wrong and reintroduces
>   a panic race on module unload. The wakeup() is internal to
>   kproc_exit/kthread_exit. The correct fix is to fix the msleep() in
>   detach to sleep on fdc->fdc_thread instead of &fdc->fdc_thread.
>   
>   Noted and reviewed by:  jhb
>   Pointy hat to:  kib
>   MFC after:      1 week

Original pointy hat to:	jhb

As I missed updating the msleep() when I fixed the unload race the first
time around which lead to the hang seen in the original PR.

-- 
John Baldwin


More information about the cvs-src mailing list