Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 5 Dec 2012 20:21:34 +0000 (UTC)
From:      Jim Harris <jimharris@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r243904 - head/sys/dev/isci
Message-ID:  <201212052021.qB5KLYKh029517@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: jimharris
Date: Wed Dec  5 20:21:33 2012
New Revision: 243904
URL: http://svnweb.freebsd.org/changeset/base/243904

Log:
  Don't call bus_dmamap_load in CAM_DIR_NONE case, since there is nothing
  to map, and technically this isn't allowed.
  
  Functionally, it works OK (at least on x86) to call bus_dmamap_load with
  a NULL data pointer and zero length, so this is primarily for correctness
  and consistency with other drivers.
  
  While here, remove check in isci_io_request_construct for nseg==0.
  Previously, bus_dmamap_load would pass nseg==1, even for case where
  buffer is NULL and length = 0, which allowed CAM_DIR_NONE CCBs
  to get processed.  This check is not correct though, and needed to be
  removed both for the changes elsewhere in this patch, as well as jeff's
  preliminary bus_dmamap_load_ccb patch (which uncovered all of this in
  the first place).
  
  MFC after:	3 days

Modified:
  head/sys/dev/isci/isci_io_request.c

Modified: head/sys/dev/isci/isci_io_request.c
==============================================================================
--- head/sys/dev/isci/isci_io_request.c	Wed Dec  5 19:45:24 2012	(r243903)
+++ head/sys/dev/isci/isci_io_request.c	Wed Dec  5 20:21:33 2012	(r243904)
@@ -670,8 +670,7 @@ isci_io_request_construct(void *arg, bus
 	io_request->sge = seg;
 	ccb = io_request->ccb;
 
-	/* XXX More cleanup is needed here */
-	if ((nseg == 0) || (error != 0)) {
+	if (error != 0) {
 		ccb->ccb_h.status = CAM_REQ_INVALID;
 		xpt_done(ccb);
 		return;
@@ -757,18 +756,21 @@ isci_io_request_execute_scsi_io(union cc
 		panic("Unexpected CAM_DATA_PHYS flag!  flags = 0x%x\n",
 		    ccb->ccb_h.flags);
 
-	error = bus_dmamap_load(io_request->parent.dma_tag,
-	    io_request->parent.dma_map, csio->data_ptr, csio->dxfer_len,
-	    isci_io_request_construct, io_request, 0x0);
-
-	/* A resource shortage from BUSDMA will be automatically
-	 * continued at a later point, pushing the CCB processing
-	 * forward, which will in turn unfreeze the simq.
-	 */
-	if (error == EINPROGRESS) {
-		xpt_freeze_simq(controller->sim, 1);
-		ccb->ccb_h.flags |= CAM_RELEASE_SIMQ;
-	}
+	if ((ccb->ccb_h.flags & CAM_DIR_MASK) != CAM_DIR_NONE) {
+		error = bus_dmamap_load(io_request->parent.dma_tag,
+		    io_request->parent.dma_map, csio->data_ptr, csio->dxfer_len,
+		    isci_io_request_construct, io_request, 0x0);
+
+		/* A resource shortage from BUSDMA will be automatically
+		 * continued at a later point, pushing the CCB processing
+		 * forward, which will in turn unfreeze the simq.
+		 */
+		if (error == EINPROGRESS) {
+			xpt_freeze_simq(controller->sim, 1);
+			ccb->ccb_h.flags |= CAM_RELEASE_SIMQ;
+		}
+	} else
+		isci_io_request_construct(io_request, NULL, 0, 0);
 }
 
 void



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