Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 14 Jun 2014 00:54:58 +0000 (UTC)
From:      Xin LI <delphij@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-9@freebsd.org
Subject:   svn commit: r267458 - in stable/9/sys/dev: hpt27xx hptmv hptrr
Message-ID:  <201406140054.s5E0sw3A029625@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: delphij
Date: Sat Jun 14 00:54:57 2014
New Revision: 267458
URL: http://svnweb.freebsd.org/changeset/base/267458

Log:
  MFC r267368:
  
  Apply vendor fixes to the High Point drivers:
  
   - Don't call xpt_free_path() in os_query_remove_device() and
     always return TRUE.
   - Update os_buildsgl() to support build logical SG table which
     will be used by lower RAID module.
   - Return CAM_SEL_TIMEOUTstatus for SCSIcommand failed as target
     missing.
  
  Many thanks to HighPoint for providing this driver update.
  
  Submitted by:	Steve Chang
  Reviewed by:	mav
  Approved by:	re (gjb)

Modified:
  stable/9/sys/dev/hpt27xx/hpt27xx_os_bsd.c
  stable/9/sys/dev/hpt27xx/hpt27xx_osm_bsd.c
  stable/9/sys/dev/hptmv/entry.c
  stable/9/sys/dev/hptrr/hptrr_os_bsd.c
  stable/9/sys/dev/hptrr/hptrr_osm_bsd.c
Directory Properties:
  stable/9/sys/   (props changed)
  stable/9/sys/dev/   (props changed)

Modified: stable/9/sys/dev/hpt27xx/hpt27xx_os_bsd.c
==============================================================================
--- stable/9/sys/dev/hpt27xx/hpt27xx_os_bsd.c	Sat Jun 14 00:44:57 2014	(r267457)
+++ stable/9/sys/dev/hpt27xx/hpt27xx_os_bsd.c	Sat Jun 14 00:54:57 2014	(r267458)
@@ -324,21 +324,7 @@ int os_revalidate_device(void *osext, in
 
 int os_query_remove_device(void *osext, int id)
 {
-	PVBUS_EXT				vbus_ext = (PVBUS_EXT)osext;
-	struct cam_periph		*periph = NULL;
-    struct cam_path			*path;
-    int						status,retval = 0;
-
-    status = xpt_create_path(&path, NULL, vbus_ext->sim->path_id, id, 0);
-    if (status == CAM_REQ_CMP) {
-		if((periph = cam_periph_find(path, "da")) != NULL){
-			if(periph->refcount >= 1)	
-				retval = -1;
-		}
-		xpt_free_path(path);
-    }
-
-    return retval;
+	return 0;
 }
 
 HPT_U8 os_get_vbus_seq(void *osext)

Modified: stable/9/sys/dev/hpt27xx/hpt27xx_osm_bsd.c
==============================================================================
--- stable/9/sys/dev/hpt27xx/hpt27xx_osm_bsd.c	Sat Jun 14 00:44:57 2014	(r267457)
+++ stable/9/sys/dev/hpt27xx/hpt27xx_osm_bsd.c	Sat Jun 14 00:54:57 2014	(r267458)
@@ -474,6 +474,16 @@ static void os_cmddone(PCOMMAND pCmd)
 
 static int os_buildsgl(PCOMMAND pCmd, PSG pSg, int logical)
 {
+	POS_CMDEXT ext = (POS_CMDEXT)pCmd->priv;
+	union ccb *ccb = ext->ccb;
+
+	if (logical) {
+		os_set_sgptr(pSg, (HPT_U8 *)ccb->csio.data_ptr);
+		pSg->size = ccb->csio.dxfer_len;
+		pSg->eot = 1;
+		return TRUE;
+	}
+
 	/* since we have provided physical sg, nobody will ask us to build physical sg */
 	HPT_ASSERT(0);
 	return FALSE;
@@ -547,7 +557,7 @@ static void hpt_scsi_io(PVBUS_EXT vbus_e
 	vd = ldm_find_target(vbus, ccb->ccb_h.target_id);
 
 	if (!vd) {
-		ccb->ccb_h.status = CAM_TID_INVALID;
+		ccb->ccb_h.status = CAM_SEL_TIMEOUT;
 		xpt_done(ccb);
 		return;
 	}

Modified: stable/9/sys/dev/hptmv/entry.c
==============================================================================
--- stable/9/sys/dev/hptmv/entry.c	Sat Jun 14 00:44:57 2014	(r267457)
+++ stable/9/sys/dev/hptmv/entry.c	Sat Jun 14 00:54:57 2014	(r267458)
@@ -2015,7 +2015,7 @@ hpt_probe(device_t dev)
 	{
 		KdPrintI((CONTROLLER_NAME " found\n"));
 		device_set_desc(dev, CONTROLLER_NAME);
-		return 0;
+		return (BUS_PROBE_DEFAULT);
 	}
 	else
 		return(ENXIO);
@@ -2622,7 +2622,14 @@ launch_worker_thread(void)
 
 int HPTLIBAPI fOsBuildSgl(_VBUS_ARG PCommand pCmd, FPSCAT_GATH pSg, int logical)
 {
-
+	union ccb *ccb = (union ccb *)pCmd->pOrgCommand;
+ 
+	if (logical) {
+		pSg->dSgAddress = (ULONG_PTR)(UCHAR *)ccb->csio.data_ptr;
+		pSg->wSgSize = ccb->csio.dxfer_len;
+		pSg->wSgFlag = SG_FLAG_EOT;
+		return TRUE;
+	}
 	/* since we have provided physical sg, nobody will ask us to build physical sg */
 	HPT_ASSERT(0);
 	return FALSE;

Modified: stable/9/sys/dev/hptrr/hptrr_os_bsd.c
==============================================================================
--- stable/9/sys/dev/hptrr/hptrr_os_bsd.c	Sat Jun 14 00:44:57 2014	(r267457)
+++ stable/9/sys/dev/hptrr/hptrr_os_bsd.c	Sat Jun 14 00:54:57 2014	(r267458)
@@ -256,21 +256,7 @@ int os_revalidate_device(void *osext, in
 
 int os_query_remove_device(void *osext, int id)
 {
-	PVBUS_EXT				vbus_ext = (PVBUS_EXT)osext;
-	struct cam_periph		*periph = NULL;
-    struct cam_path			*path;
-    int						status,retval = 0;
-
-    status = xpt_create_path(&path, NULL, vbus_ext->sim->path_id, id, 0);
-    if (status == CAM_REQ_CMP) {
-		if((periph = cam_periph_find(path, "da")) != NULL){
-			if(periph->refcount >= 1)	
-				retval = -1;
-		}
-		xpt_free_path(path);
-    }
-
-    return retval;
+	return 0;
 }
 
 HPT_U8 os_get_vbus_seq(void *osext)

Modified: stable/9/sys/dev/hptrr/hptrr_osm_bsd.c
==============================================================================
--- stable/9/sys/dev/hptrr/hptrr_osm_bsd.c	Sat Jun 14 00:44:57 2014	(r267457)
+++ stable/9/sys/dev/hptrr/hptrr_osm_bsd.c	Sat Jun 14 00:54:57 2014	(r267458)
@@ -61,7 +61,7 @@ static int hpt_probe(device_t dev)
 				memset(hba, 0, sizeof(HBA));
 				hba->ext_type = EXT_TYPE_HBA;
 				hba->ldm_adapter.him = him;
-				return 0;
+				return (BUS_PROBE_DEFAULT);
 			}
 		}
 	}
@@ -481,6 +481,15 @@ static void os_cmddone(PCOMMAND pCmd)
 
 static int os_buildsgl(PCOMMAND pCmd, PSG pSg, int logical)
 {
+	POS_CMDEXT ext = (POS_CMDEXT)pCmd->priv;
+	union ccb *ccb = ext->ccb;
+
+	if (logical) {
+		os_set_sgptr(pSg, (HPT_U8 *)ccb->csio.data_ptr);
+		pSg->size = ccb->csio.dxfer_len;
+		pSg->eot = 1;
+		return TRUE;
+	}
 
 	/* since we have provided physical sg, nobody will ask us to build physical sg */
 	HPT_ASSERT(0);
@@ -554,7 +563,7 @@ static void hpt_scsi_io(PVBUS_EXT vbus_e
 	vd = ldm_find_target(vbus, ccb->ccb_h.target_id);
 
 	if (!vd) {
-		ccb->ccb_h.status = CAM_TID_INVALID;
+		ccb->ccb_h.status = CAM_SEL_TIMEOUT;
 		xpt_done(ccb);
 		return;
 	}



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