Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 28 Feb 2009 10:40:53 +0000 (UTC)
From:      Edward Tomasz Napierala <trasz@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-7@freebsd.org
Subject:   svn commit: r189151 - in stable/7/sys: . cam/scsi contrib/pf dev/ath/ath_hal dev/cxgb
Message-ID:  <200902281040.n1SAer4K006139@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: trasz
Date: Sat Feb 28 10:40:52 2009
New Revision: 189151
URL: http://svn.freebsd.org/changeset/base/189151

Log:
  MFC r187028:
  
  Don't call destroy_dev(9) with a mutex held.  While here, shuffle
  things around so the periph destructors look alike.  Based on a patch
  by Jaakko Heinonen.
  
  Submitted by:	Jaakko Heinonen
  Reviewed by:	scottl
  Approved by:	rwatson (mentor)
  Sponsored by:	FreeBSD Foundation

Modified:
  stable/7/sys/   (props changed)
  stable/7/sys/cam/scsi/scsi_ch.c
  stable/7/sys/cam/scsi/scsi_pass.c
  stable/7/sys/cam/scsi/scsi_pt.c
  stable/7/sys/cam/scsi/scsi_sa.c
  stable/7/sys/cam/scsi/scsi_ses.c
  stable/7/sys/cam/scsi/scsi_sg.c
  stable/7/sys/contrib/pf/   (props changed)
  stable/7/sys/dev/ath/ath_hal/   (props changed)
  stable/7/sys/dev/cxgb/   (props changed)

Modified: stable/7/sys/cam/scsi/scsi_ch.c
==============================================================================
--- stable/7/sys/cam/scsi/scsi_ch.c	Sat Feb 28 10:40:37 2009	(r189150)
+++ stable/7/sys/cam/scsi/scsi_ch.c	Sat Feb 28 10:40:52 2009	(r189151)
@@ -262,9 +262,11 @@ chcleanup(struct cam_periph *periph)
 
 	softc = (struct ch_softc *)periph->softc;
 
+	xpt_print(periph->path, "removing device entry\n");
 	devstat_remove_entry(softc->device_stats);
+	cam_periph_unlock(periph);
 	destroy_dev(softc->dev);
-	xpt_print(periph->path, "removing device entry\n");
+	cam_periph_lock(periph);
 	free(softc, M_DEVBUF);
 }
 

Modified: stable/7/sys/cam/scsi/scsi_pass.c
==============================================================================
--- stable/7/sys/cam/scsi/scsi_pass.c	Sat Feb 28 10:40:37 2009	(r189150)
+++ stable/7/sys/cam/scsi/scsi_pass.c	Sat Feb 28 10:40:52 2009	(r189151)
@@ -165,13 +165,12 @@ passcleanup(struct cam_periph *periph)
 
 	softc = (struct pass_softc *)periph->softc;
 
+	if (bootverbose)
+		xpt_print(periph->path, "removing device entry\n");
 	devstat_remove_entry(softc->device_stats);
-
+	cam_periph_unlock(periph);
 	destroy_dev(softc->dev);
-
-	if (bootverbose) {
-		xpt_print(periph->path, "removing device entry\n");
-	}
+	cam_periph_lock(periph);
 	free(softc, M_DEVBUF);
 }
 

Modified: stable/7/sys/cam/scsi/scsi_pt.c
==============================================================================
--- stable/7/sys/cam/scsi/scsi_pt.c	Sat Feb 28 10:40:37 2009	(r189150)
+++ stable/7/sys/cam/scsi/scsi_pt.c	Sat Feb 28 10:40:52 2009	(r189151)
@@ -342,11 +342,11 @@ ptdtor(struct cam_periph *periph)
 
 	softc = (struct pt_softc *)periph->softc;
 
+	xpt_print(periph->path, "removing device entry\n");
 	devstat_remove_entry(softc->device_stats);
-
+	cam_periph_unlock(periph);
 	destroy_dev(softc->dev);
-
-	xpt_print(periph->path, "removing device entry\n");
+	cam_periph_lock(periph);
 	free(softc, M_DEVBUF);
 }
 

Modified: stable/7/sys/cam/scsi/scsi_sa.c
==============================================================================
--- stable/7/sys/cam/scsi/scsi_sa.c	Sat Feb 28 10:40:37 2009	(r189150)
+++ stable/7/sys/cam/scsi/scsi_sa.c	Sat Feb 28 10:40:52 2009	(r189151)
@@ -1377,17 +1377,16 @@ sacleanup(struct cam_periph *periph)
 
 	softc = (struct sa_softc *)periph->softc;
 
+	xpt_print(periph->path, "removing device entry\n");
 	devstat_remove_entry(softc->device_stats);
-
+	cam_periph_unlock(periph);
 	destroy_dev(softc->devs.ctl_dev);
-
 	for (i = 0; i < SA_NUM_MODES; i++) {
 		destroy_dev(softc->devs.mode_devs[i].r_dev);
 		destroy_dev(softc->devs.mode_devs[i].nr_dev);
 		destroy_dev(softc->devs.mode_devs[i].er_dev);
 	}
-
-	xpt_print(periph->path, "removing device entry\n");
+	cam_periph_lock(periph);
 	free(softc, M_SCSISA);
 }
 

Modified: stable/7/sys/cam/scsi/scsi_ses.c
==============================================================================
--- stable/7/sys/cam/scsi/scsi_ses.c	Sat Feb 28 10:40:37 2009	(r189150)
+++ stable/7/sys/cam/scsi/scsi_ses.c	Sat Feb 28 10:40:52 2009	(r189151)
@@ -227,9 +227,10 @@ sescleanup(struct cam_periph *periph)
 
 	softc = (struct ses_softc *)periph->softc;
 
-	destroy_dev(softc->ses_dev);
-
 	xpt_print(periph->path, "removing device entry\n");
+	cam_periph_unlock(periph);
+	destroy_dev(softc->ses_dev);
+	cam_periph_lock(periph);
 	free(softc, M_SCSISES);
 }
 

Modified: stable/7/sys/cam/scsi/scsi_sg.c
==============================================================================
--- stable/7/sys/cam/scsi/scsi_sg.c	Sat Feb 28 10:40:37 2009	(r189150)
+++ stable/7/sys/cam/scsi/scsi_sg.c	Sat Feb 28 10:40:52 2009	(r189151)
@@ -200,11 +200,12 @@ sgcleanup(struct cam_periph *periph)
 	struct sg_softc *softc;
 
 	softc = (struct sg_softc *)periph->softc;
+	if (bootverbose)
+		xpt_print(periph->path, "removing device entry\n");
 	devstat_remove_entry(softc->device_stats);
+	cam_periph_unlock(periph);
 	destroy_dev(softc->dev);
-	if (bootverbose) {
-		xpt_print(periph->path, "removing device entry\n");
-	}
+	cam_periph_lock(periph);
 	free(softc, M_DEVBUF);
 }
 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200902281040.n1SAer4K006139>