Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 6 Sep 2009 16:31:37 GMT
From:      Alexander Motin <mav@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 168245 for review
Message-ID:  <200909061631.n86GVbwZ089622@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=168245

Change 168245 by mav@mav_mavbook on 2009/09/06 16:30:52

	Reorder some struct cam_sim elements to improve read caching.

Affected files ...

.. //depot/projects/scottl-camlock/src/sys/cam/cam_sim.h#12 edit

Differences ...

==== //depot/projects/scottl-camlock/src/sys/cam/cam_sim.h#12 (text+ko) ====

@@ -94,23 +94,13 @@
 	const char		*sim_name;
 	void			*softc;
 	struct mtx		*mtx;
-	TAILQ_HEAD(, ccb_hdr)	sim_doneq;
+	struct cam_devq 	*devq;	/* Device Queue to use for this SIM */
 	TAILQ_ENTRY(cam_sim)	links;
 	u_int32_t		path_id;/* The Boot device may set this to 0? */
 	u_int32_t		unit_number;
 	u_int32_t		bus_id;
 	int			max_tagged_dev_openings;
 	int			max_dev_openings;
-	u_int32_t		flags;
-#define	CAM_SIM_REL_TIMEOUT_PENDING	0x01
-#define	CAM_SIM_MPSAFE			0x02
-#define CAM_SIM_ON_DONEQ		0x04
-	struct callout		callout;
-	struct cam_devq 	*devq;	/* Device Queue to use for this SIM */
-	int			refcount; /* References to the SIM. */
-
-	/* "Pool" of inactive ccbs managed by xpt_get_ccb and xpt_release_ccb */
-	SLIST_HEAD(,ccb_hdr)	ccb_freeq;
 	/*
 	 * Maximum size of ccb pool.  Modified as devices are added/removed
 	 * or have their * opening counts changed.
@@ -118,7 +108,15 @@
 	u_int			max_ccbs;
 	/* Current count of allocated ccbs */
 	u_int			ccb_count;
-
+	struct callout		callout;
+	u_int32_t		flags;
+#define	CAM_SIM_REL_TIMEOUT_PENDING	0x01
+#define	CAM_SIM_MPSAFE			0x02
+#define CAM_SIM_ON_DONEQ		0x04
+	int			refcount; /* References to the SIM. */
+	TAILQ_HEAD(, ccb_hdr)	sim_doneq; /* Completed requests queue. */
+	/* "Pool" of inactive ccbs managed by xpt_get_ccb and xpt_release_ccb */
+	SLIST_HEAD(, ccb_hdr)	ccb_freeq;
 };
 
 #define CAM_SIM_LOCK(sim)	mtx_lock((sim)->mtx);



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