cvs commit: src/sys/dev/iwi if_iwi.c if_iwivar.h

Luigi Rizzo rizzo at icir.org
Tue Feb 20 18:00:04 UTC 2007


On Tue, Feb 20, 2007 at 09:06:08AM -0800, Sam Leffler wrote:
> Luigi Rizzo wrote:
> 
> >   + add debugging code IWI_LOCK_ASSERT() to detect missing locks.
> >     These only do a printf, and should go away once we figure out why
> >     the driver sometimes freezes the system due to a (yet unidentified)
> >     race condition.
> 
> LOCK_ASSERT should do a mtx_assert and not printf.  If you want to
> diverge from existing convention in all other drivers please use a
> different name; e.g. IWI_LOCK_CHECK.

done, thanks for pointing it out.

> >   + add a device_printf() in iwi_ioctl() in certain conditions
> >     (see comment in the code).  This helps preventing the race condition
> >     mentioned above, and makes the system survive. This printf will
> >     also go away once fixing this bug is completed.
> 
> Please don't commit debug stuff like this; we've discussed this

Also replaced with better code to address the problem.

> privately.  This driver has been in use for many months w/o complaints
> of the sort you are seeing.

The fact that I am the only one seeing (or reporting) the problem
does not mean that there is no bug (or i wouldn't spend so much
time trying to fix it :).

	cheers
	luigi


More information about the cvs-src mailing list