Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 18 Aug 2009 09:27:17 +0000 (UTC)
From:      Alexander Motin <mav@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r196353 - head/sys/cam/ata
Message-ID:  <200908180927.n7I9RHOn010243@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: mav
Date: Tue Aug 18 09:27:17 2009
New Revision: 196353
URL: http://svn.freebsd.org/changeset/base/196353

Log:
  Fix copy/paste bug, that requests data read during ATA device probe sequence
  for ATA_SETFEATURES/ATA_SF_SETXFER command which by definition transfers no
  data. Most of controllers are irrelevant to this bug, but some nVidia's
  doesn't.
  
  Tested on:	current@
  Approved by:	re (kib)

Modified:
  head/sys/cam/ata/ata_xpt.c

Modified: head/sys/cam/ata/ata_xpt.c
==============================================================================
--- head/sys/cam/ata/ata_xpt.c	Tue Aug 18 08:46:54 2009	(r196352)
+++ head/sys/cam/ata/ata_xpt.c	Tue Aug 18 09:27:17 2009	(r196353)
@@ -370,10 +370,10 @@ probestart(struct cam_periph *periph, un
 		cam_fill_ataio(ataio,
 		      1,
 		      probedone,
-		      /*flags*/CAM_DIR_IN,
-		      MSG_SIMPLE_Q_TAG,
-		      /*data_ptr*/(u_int8_t *)ident_buf,
-		      /*dxfer_len*/sizeof(struct ata_params),
+		      /*flags*/CAM_DIR_NONE,
+		      0,
+		      /*data_ptr*/NULL,
+		      /*dxfer_len*/0,
 		      30 * 1000);
 		ata_36bit_cmd(ataio, ATA_SETFEATURES, ATA_SF_SETXFER, 0,
 		    ata_max_mode(ident_buf, ATA_UDMA6, ATA_UDMA6));



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