Date: Thu, 17 Jul 2008 15:46:35 +0000 (UTC) From: Andrew Gallatin <gallatin@FreeBSD.org> To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/dev/mxge if_mxge.c Message-ID: <200807171546.m6HFkrK9081309@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
gallatin 2008-07-17 15:46:35 UTC
FreeBSD src repository
Modified files:
sys/dev/mxge if_mxge.c
Log:
SVN rev 180567 on 2008-07-17 15:46:35Z by gallatin
Clean up mxge's use of callouts as pointed out by jhb,
and handle NIC hardware watchdog resets.
- remove buggy code at the top of mxge_tick() which tried
to detect a race which is already detected in the kernel's
callout code.
- move callout_stop() and callout_reset() into mxge_close()
mxge_open() rather than doing the callout manipulation
all over the place.
- use callout_drain(), rather than callout_stop() to prevent
a potential race between mxge_tick() and mxge_detach()
which could lead to softclock using a destroyed mutex
- restructure the mxge_tick() and mxge_watchdog_reset()
routines to avoid resetting a callout, and then
immediately stopping it if the watchdog reset routine
is called, and fails.
- enable the driver to handle NIC hardware watchdog
resets by restoring the NIC's PCI config space, which is
lost when the NIC hardware watchdog triggers.
Reviewed by: jhb (previus version)
Revision Changes Path
1.50 +19 -36 src/sys/dev/mxge/if_mxge.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200807171546.m6HFkrK9081309>