Author: mav
Date: Mon Oct  5 09:20:09 2015
New Revision: 288757
URL: https://svnweb.freebsd.org/changeset/base/288757

Log:
  MFC r287784: Implement data/status aggregation for secondary HA node.
  
  For short read requests this reduces latency by 30%, reporting command
  completion after two interlink roundtrips instead of full three.

Modified:
  stable/10/sys/cam/ctl/ctl.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/cam/ctl/ctl.c
==============================================================================
--- stable/10/sys/cam/ctl/ctl.c Mon Oct  5 09:19:14 2015        (r288756)
+++ stable/10/sys/cam/ctl/ctl.c Mon Oct  5 09:20:09 2015        (r288757)
@@ -980,6 +980,8 @@ ctl_isc_event_handler(ctl_ha_channel cha
                         * when the datamove is complete.
                         */
                        io->io_hdr.serializing_sc = msg->hdr.serializing_sc;
+                       if (msg->hdr.status == CTL_SUCCESS)
+                               io->io_hdr.status = msg->hdr.status;
 
                        if (msg->dt.sg_sequence == 0) {
                                i = msg->dt.kern_sg_entries +
@@ -1062,6 +1064,8 @@ ctl_isc_event_handler(ctl_ha_channel cha
                                memcpy(&io->scsiio.sense_data,
                                    &msg->scsi.sense_data,
                                    msg->scsi.sense_len);
+                               if (msg->hdr.status == CTL_SUCCESS)
+                                       io->io_hdr.flags |= 
CTL_FLAG_STATUS_SENT;
                        }
                        ctl_enqueue_isc(io);
                        break;
@@ -12209,6 +12213,7 @@ ctl_datamove(union ctl_io *io)
                msg.hdr.original_sc = io->io_hdr.original_sc;
                msg.hdr.serializing_sc = io;
                msg.hdr.nexus = io->io_hdr.nexus;
+               msg.hdr.status = io->io_hdr.status;
                msg.dt.flags = io->io_hdr.flags;
                /*
                 * We convert everything into a S/G list here.  We can't
@@ -12592,10 +12597,12 @@ ctl_datamove_remote_xfer(union ctl_io *i
         * failure.
         */
        if ((rq == NULL)
-        && ((io->io_hdr.status & CTL_STATUS_MASK) != CTL_STATUS_NONE))
+        && ((io->io_hdr.status & CTL_STATUS_MASK) != CTL_STATUS_NONE &&
+            (io->io_hdr.status & CTL_STATUS_MASK) != CTL_SUCCESS))
                ctl_set_busy(&io->scsiio);
 
-       if ((io->io_hdr.status & CTL_STATUS_MASK) != CTL_STATUS_NONE) {
+       if ((io->io_hdr.status & CTL_STATUS_MASK) != CTL_STATUS_NONE &&
+           (io->io_hdr.status & CTL_STATUS_MASK) != CTL_SUCCESS) {
 
                if (rq != NULL)
                        ctl_dt_req_free(rq);
_______________________________________________
svn-src-stable-10@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-stable-10
To unsubscribe, send any mail to "svn-src-stable-10-unsubscr...@freebsd.org"

Reply via email to