src/sys/dev/ie, branch release/8.1.0FreeBSD source treehttps://cgit-dev.freebsd.org/src/atom?h=release%2F8.1.02009-06-26T11:45:06ZUse if_maddr_rlock()/if_maddr_runlock() rather than IF_ADDR_LOCK()/2009-06-26T11:45:06ZRobert Watsonrwatson@FreeBSD.org2009-06-26T11:45:06Zurn:sha1:eb956cd041f956275522092d6ba66671356ff84f
IF_ADDR_UNLOCK() across network device drivers when accessing the
per-interface multicast address list, if_multiaddrs. This will
allow us to change the locking strategy without affecting our driver
programming interface or binary interface.
For two wireless drivers, remove unnecessary locking, since they
don't actually access the multicast address list.
Approved by: re (kib)
MFC after: 6 weeks
Use the new bus device_shutdown hook for performing the shutdown action2008-08-01T21:33:07ZJohn Baldwinjhb@FreeBSD.org2008-08-01T21:33:07Zurn:sha1:9b1db8012555e6cabcde27d1c9d0e8fd53124432
on the ee16 parts rather than explicitly registering an event handler
that wasn't being torn down on detach.
Rename the ie_EE16 module to just 'ie' so that at least one of the modules2008-06-05T14:45:32ZJohn Baldwinjhb@FreeBSD.org2008-06-05T14:45:32Zurn:sha1:9ceedf231bb21f54c43717b3c157e0b87388e27d
for this driver is called 'ie'. Otherwise, ifconfig(8) doesn't recognize
any of the modules as being the ie(4) driver and will always try to kldload
the driver even when it is already present in the kernel.
Reported by: Thierry Herbelot
- Fix two calls to ieinit() in ieioctl() to call ieinit_locked() instead to2008-06-05T14:43:55ZJohn Baldwinjhb@FreeBSD.org2008-06-05T14:43:55Zurn:sha1:d98e0a28593b8d055649b9238418b536703c307a
avoid recursing on the lock.
- Use IFQ_SET_MAXLEN().
Make ie(4) MPSAFE:2008-06-02T19:17:40ZJohn Baldwinjhb@FreeBSD.org2008-06-02T19:17:40Zurn:sha1:00f1da89ab2c9c421435fa1362860e09eef713ec
- Add a mutex to the softc and use it to protect the softc and device.
- Setup the interrupt handler in the common code instead of in each front
end and do it after ether_ifattach().
- Use ie_stop() and ieinit_locked() in iereset() rather than frobbing IFF_UP
and invoking ieioctl().
- Use DELAY() to implement a spin loop on a register with a timeout rather
than scheduling a timeout and then doing a tight spin on the register.
In the non-MPSAFE case this would never have worked because the spinning
code held Giant and the timeout routine would have been blocked on Giant
forever. The same approach would not worke in the MPSAFE case either for
the same reason, hence use a loop around DELAY().
- Clear IFF_DRV_(RUNNING|OACTIVE) in ie_stop() rather than in callers.
- Call ieinit_locked() directly rather than ieioctl(!) from ie_mc_reset().
- Don't leak the rx frame buffer on detach.
Tested by: Thierry Herbelot thierry of herbelot.com
Fix build with DEBUG defined.2008-05-30T16:22:30ZJohn Baldwinjhb@FreeBSD.org2008-05-30T16:22:30Zurn:sha1:f1150521113d5d4169b56b0813791f06aeebc01c
Reported by: tinderbox
Use if_printf() and if_xname and retire 'unit' from the softc.2008-05-29T17:32:55ZJohn Baldwinjhb@FreeBSD.org2008-05-29T17:32:55Zurn:sha1:3022250a490679e1e476e4c6226cafa68d942970o break newbus api: add a new argument of type driver_filter_t to2007-02-23T12:19:07ZPaolo Pisatipiso@FreeBSD.org2007-02-23T12:19:07Zurn:sha1:ef544f631226436ef590825881e7a28369df82f6
bus_setup_intr()
o add an int return code to all fast handlers
o retire INTR_FAST/IH_FAST
For more info: http://docs.freebsd.org/cgi/getmsg.cgi?fetch=465712+0+current/freebsd-current
Reviewed by: many
Approved by: re@
Since DELAY() was moved, most <machine/clock.h> #includes have been2006-05-16T14:37:58ZPoul-Henning Kampphk@FreeBSD.org2006-05-16T14:37:58Zurn:sha1:c40da00ca320a38816049edd21966bd4045b2303
unnecessary.
Fix -Wundef warnings found when compiling i386 LINT, GENERIC and2005-12-05T11:58:35ZRuslan Ermilovru@FreeBSD.org2005-12-05T11:58:35Zurn:sha1:342ed5d94898694d5ff3b5648cdc247d07dfad16
custom kernels.