Author: jh
Date: Sat Apr 16 06:54:41 2011
New Revision: 220686
URL: http://svn.freebsd.org/changeset/base/220686
Log:
Move TUNABLE_INT_FETCH() out of the SIM lock to avoid sleeping while
holding the lock. The fix is analogous to r220618 for ada(4).
Reviewed by: mav
Modified:
head/sys/cam/scsi/scsi_cd.c
head/sys/cam/scsi/scsi_da.c
Modified: head/sys/cam/scsi/scsi_cd.c
==============================================================================
--- head/sys/cam/scsi/scsi_cd.c Sat Apr 16 06:20:49 2011 (r220685)
+++ head/sys/cam/scsi/scsi_cd.c Sat Apr 16 06:54:41 2011 (r220686)
@@ -687,6 +687,8 @@ cdregister(struct cam_periph *periph, vo
else
softc->minimum_command_size = 6;
+ (void)cam_periph_hold(periph, PRIBIO);
+ cam_periph_unlock(periph);
/*
* Load the user's default, if any.
*/
@@ -712,7 +714,6 @@ cdregister(struct cam_periph *periph, vo
* WORM peripheral driver. WORM drives will also have the WORM
* driver attached to them.
*/
- cam_periph_unlock(periph);
softc->disk = disk_alloc();
softc->disk->d_devstat = devstat_new_entry("cd",
periph->unit_number, 0,
@@ -746,6 +747,7 @@ cdregister(struct cam_periph *periph, vo
softc->disk->d_hba_subdevice = cpi.hba_subdevice;
disk_create(softc->disk, DISK_VERSION);
cam_periph_lock(periph);
+ cam_periph_unhold(periph);
/*
* Add an async callback so that we get
Modified: head/sys/cam/scsi/scsi_da.c
==============================================================================
--- head/sys/cam/scsi/scsi_da.c Sat Apr 16 06:20:49 2011 (r220685)
+++ head/sys/cam/scsi/scsi_da.c Sat Apr 16 06:54:41 2011 (r220686)
@@ -1231,35 +1231,6 @@ daregister(struct cam_periph *periph, vo
TASK_INIT(&softc->sysctl_task, 0, dasysctlinit, periph);
/*
- * RBC devices don't have to support READ(6), only READ(10).
- */
- if (softc->quirks & DA_Q_NO_6_BYTE || SID_TYPE(&cgd->inq_data) == T_RBC)
- softc->minimum_cmd_size = 10;
- else
- softc->minimum_cmd_size = 6;
-
- /*
- * Load the user's default, if any.
- */
- snprintf(tmpstr, sizeof(tmpstr), "kern.cam.da.%d.minimum_cmd_size",
- periph->unit_number);
- TUNABLE_INT_FETCH(tmpstr, &softc->minimum_cmd_size);
-
- /*
- * 6, 10, 12 and 16 are the currently permissible values.
- */
- if (softc->minimum_cmd_size < 6)
- softc->minimum_cmd_size = 6;
- else if ((softc->minimum_cmd_size > 6)
- && (softc->minimum_cmd_size <= 10))
- softc->minimum_cmd_size = 10;
- else if ((softc->minimum_cmd_size > 10)
- && (softc->minimum_cmd_size <= 12))
- softc->minimum_cmd_size = 12;
- else if (softc->minimum_cmd_size > 12)
- softc->minimum_cmd_size = 16;
-
- /*
* Register this media as a disk
*/
@@ -1291,6 +1262,35 @@ daregister(struct cam_periph *periph, vo
dasendorderedtag, softc);
mtx_unlock(periph->sim->mtx);
+ /*
+ * RBC devices don't have to support READ(6), only READ(10).
+ */
+ if (softc->quirks & DA_Q_NO_6_BYTE || SID_TYPE(&cgd->inq_data) == T_RBC)
+ softc->minimum_cmd_size = 10;
+ else
+ softc->minimum_cmd_size = 6;
+
+ /*
+ * Load the user's default, if any.
+ */
+ snprintf(tmpstr, sizeof(tmpstr), "kern.cam.da.%d.minimum_cmd_size",
+ periph->unit_number);
+ TUNABLE_INT_FETCH(tmpstr, &softc->minimum_cmd_size);
+
+ /*
+ * 6, 10, 12 and 16 are the currently permissible values.
+ */
+ if (softc->minimum_cmd_size < 6)
+ softc->minimum_cmd_size = 6;
+ else if ((softc->minimum_cmd_size > 6)
+ && (softc->minimum_cmd_size <= 10))
+ softc->minimum_cmd_size = 10;
+ else if ((softc->minimum_cmd_size > 10)
+ && (softc->minimum_cmd_size <= 12))
+ softc->minimum_cmd_size = 12;
+ else if (softc->minimum_cmd_size > 12)
+ softc->minimum_cmd_size = 16;
+
softc->disk = disk_alloc();
softc->disk->d_devstat = devstat_new_entry(periph->periph_name,
periph->unit_number, 0,
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
