Patch to fix locking nits in hme

Marius Strobl marius at alchemy.franken.de
Thu Aug 4 18:05:13 GMT 2005


On Thu, Aug 04, 2005 at 12:53:17PM -0400, John Baldwin wrote:
> I have a patch to fix several nits in the hme(4) driver's locking including:
> 
> - Using the driver lock instead of Giant in a busdma callback.
> - Using callout_drain() in detach.
> - Locking access to the mii registers with the driver lock.
> - Clearing the RUNNING and OACTIVE flags in stop.
> - Re-enabling updating the media from hme_init_locked().
> - Making the mutex non-recursive.
> - Fixing locking in ioctl.
> 
> Patch is at http://www.freebsd.org/~jhb/patches/hme_locking.patch
> 

@@ -871,11 +859,7 @@
 #endif
 
 	/* Set the current media. */
-	/*
-	 * HME_UNLOCK(sc);	
-	 * mii_mediachg(sc->sc_mii);
-	 * HME_LOCK(sc);	
-	 */
+	mii_mediachg(sc->sc_mii);

Calling mii_mediachg() will trigger some unsolved problems with
the MIF configuration (most likely) which used to cause dhclient(8)
not working on Ultra 2 and result in no link at all on Netra T1.



More information about the freebsd-sparc64 mailing list