Index: sys/dev/sfxge/sfxge_port.c
===================================================================
--- sys/dev/sfxge/sfxge_port.c
+++ sys/dev/sfxge/sfxge_port.c
@@ -48,7 +48,7 @@
 	unsigned int count;
 	int rc;
 
-	mtx_lock(&port->lock);
+	mtx_assert(&port->lock, MA_OWNED);
 
 	if (port->init_state != SFXGE_PORT_STARTED) {
 		rc = 0;
@@ -82,7 +82,6 @@
 
 	rc = ETIMEDOUT;
 out:
-	mtx_unlock(&port->lock);
 	return (rc);
 }
 
@@ -93,12 +92,16 @@
 	unsigned int id = arg2;
 	int rc;
 
+	mtx_lock(&sc->port.lock);
 	if ((rc = sfxge_mac_stat_update(sc)) != 0)
-		return (rc);
+		goto out;
 
-	return (SYSCTL_OUT(req,
-			  (uint64_t *)sc->port.mac_stats.decode_buf + id,
-			  sizeof(uint64_t)));
+	rc = SYSCTL_OUT(req,
+			(uint64_t *)sc->port.mac_stats.decode_buf + id,
+			sizeof(uint64_t));
+out:
+	mtx_unlock(&sc->port.lock);
+	return (rc);
 }
 
 static void
@@ -453,7 +456,7 @@
 	unsigned int count;
 	int rc;
 
-	mtx_lock(&port->lock);
+	mtx_assert(&port->lock, MA_OWNED);
 
 	if (port->init_state != SFXGE_PORT_STARTED) {
 		rc = 0;
@@ -487,7 +490,6 @@
 
 	rc = ETIMEDOUT;
 out:
-	mtx_unlock(&port->lock);
 	return (rc);
 }
 
@@ -498,12 +500,16 @@
 	unsigned int id = arg2;
 	int rc;
 
+	mtx_lock(&sc->port.lock);
 	if ((rc = sfxge_phy_stat_update(sc)) != 0)
-		return (rc);
+		goto out;
 
-	return (SYSCTL_OUT(req,
-			  (uint32_t *)sc->port.phy_stats.decode_buf + id,
-			  sizeof(uint32_t)));
+	rc = SYSCTL_OUT(req,
+			(uint32_t *)sc->port.phy_stats.decode_buf + id,
+			sizeof(uint32_t));
+out:
+	mtx_unlock(&sc->port.lock);
+	return (rc);
 }
 
 static void