svn commit: r249417 - head/sys/dev/nvme

[Jim Harris]

Author: jimharris
Date: Fri Apr 12 17:36:48 2013
New Revision: 249417
URL: http://svnweb.freebsd.org/changeset/base/249417

Log:
  Rename the controller's fail_req_lock, so that it can be used for other
  locking operations on the controller.
  
  Sponsored by:	Intel

Modified:
  head/sys/dev/nvme/nvme_ctrlr.c
  head/sys/dev/nvme/nvme_private.h

Modified: head/sys/dev/nvme/nvme_ctrlr.c
==============================================================================
--- head/sys/dev/nvme/nvme_ctrlr.c	Fri Apr 12 17:34:49 2013	(r249416)
+++ head/sys/dev/nvme/nvme_ctrlr.c	Fri Apr 12 17:36:48 2013	(r249417)
@@ -307,9 +307,9 @@ nvme_ctrlr_post_failed_request(struct nv
     struct nvme_request *req)
 {
 
-	mtx_lock(&ctrlr->fail_req_lock);
+	mtx_lock(&ctrlr->lock);
 	STAILQ_INSERT_TAIL(&ctrlr->fail_req, req, stailq);
-	mtx_unlock(&ctrlr->fail_req_lock);
+	mtx_unlock(&ctrlr->lock);
 	taskqueue_enqueue(ctrlr->taskqueue, &ctrlr->fail_req_task);
 }
 
@@ -319,14 +319,14 @@ nvme_ctrlr_fail_req_task(void *arg, int 
 	struct nvme_controller	*ctrlr = arg;
 	struct nvme_request	*req;
 
-	mtx_lock(&ctrlr->fail_req_lock);
+	mtx_lock(&ctrlr->lock);
 	while (!STAILQ_EMPTY(&ctrlr->fail_req)) {
 		req = STAILQ_FIRST(&ctrlr->fail_req);
 		STAILQ_REMOVE_HEAD(&ctrlr->fail_req, stailq);
 		nvme_qpair_manual_complete_request(req->qpair, req,
 		    NVME_SCT_GENERIC, NVME_SC_ABORTED_BY_REQUEST, TRUE);
 	}
-	mtx_unlock(&ctrlr->fail_req_lock);
+	mtx_unlock(&ctrlr->lock);
 }
 
 static int
@@ -935,6 +935,8 @@ nvme_ctrlr_construct(struct nvme_control
 
 	ctrlr->dev = dev;
 
+	mtx_init(&ctrlr->lock, "nvme ctrlr lock", NULL, MTX_DEF);
+
 	status = nvme_ctrlr_allocate_bar(ctrlr);
 
 	if (status != 0)
@@ -1033,8 +1035,6 @@ intx:
 	TASK_INIT(&ctrlr->reset_task, 0, nvme_ctrlr_reset_task, ctrlr);
 
 	TASK_INIT(&ctrlr->fail_req_task, 0, nvme_ctrlr_fail_req_task, ctrlr);
-	mtx_init(&ctrlr->fail_req_lock, "nvme ctrlr fail req lock", NULL,
-	    MTX_DEF);
 	STAILQ_INIT(&ctrlr->fail_req);
 	ctrlr->is_failed = FALSE;
 

Modified: head/sys/dev/nvme/nvme_private.h
==============================================================================
--- head/sys/dev/nvme/nvme_private.h	Fri Apr 12 17:34:49 2013	(r249416)
+++ head/sys/dev/nvme/nvme_private.h	Fri Apr 12 17:36:48 2013	(r249417)
@@ -247,6 +247,8 @@ struct nvme_controller {
 
 	device_t		dev;
 
+	struct mtx		lock;
+
 	uint32_t		ready_timeout_in_ms;
 
 	bus_space_tag_t		bus_tag;
@@ -325,7 +327,6 @@ struct nvme_controller {
 
 	uint32_t		is_resetting;
 
-	struct mtx			fail_req_lock;
 	boolean_t			is_failed;
 	STAILQ_HEAD(, nvme_request)	fail_req;