cvs commit: src/sys/dev/iwi if_iwi.c if_iwivar.h
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 :).
More information about the cvs-src