Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 30 Aug 2012 00:43:14 +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: r239889 - head/sys/dev/isci/scil
Message-ID:  <201208300043.q7U0hEQk042566@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: jimharris
Date: Thu Aug 30 00:43:14 2012
New Revision: 239889
URL: http://svn.freebsd.org/changeset/base/239889

Log:
  Do not call sati_check_data_io() for SATI_UNMAP sequences.
  
  This routine is intended only for commands such as INQUIRY where
  the controller may fill out a smaller amount of data than allocated
  by the host.
  
  The end result of this bug was that isci(4) would report non-zero
  resid for successful SCSI_UNMAP commands.
  
  Sponsored by:	Intel
  MFC after:	3 days

Modified:
  head/sys/dev/isci/scil/sati.c

Modified: head/sys/dev/isci/scil/sati.c
==============================================================================
--- head/sys/dev/isci/scil/sati.c	Wed Aug 29 22:58:52 2012	(r239888)
+++ head/sys/dev/isci/scil/sati.c	Thu Aug 30 00:43:14 2012	(r239889)
@@ -1095,10 +1095,6 @@ SATI_STATUS sati_translate_command_respo
          status = sati_unmap_translate_response(
                      sequence, scsi_io, ata_io
                   );
-         if(status == SATI_COMPLETE)
-         {
-            status = sati_check_data_io(sequence);
-         }
       break;
 #endif // !defined(DISABLE_SATI_UNMAP)
 



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