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

Log:
  MFC r287868: Make COMPARE AND WRITE report offset of difference.

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

Modified: stable/10/sys/cam/ctl/ctl_backend_block.c
==============================================================================
--- stable/10/sys/cam/ctl/ctl_backend_block.c   Mon Oct  5 09:24:08 2015        
(r288762)
+++ stable/10/sys/cam/ctl/ctl_backend_block.c   Mon Oct  5 09:25:04 2015        
(r288763)
@@ -354,6 +354,48 @@ ctl_complete_beio(struct ctl_be_block_io
        }
 }
 
+static size_t
+cmp(uint8_t *a, uint8_t *b, size_t size)
+{
+       size_t i;
+
+       for (i = 0; i < size; i++) {
+               if (a[i] != b[i])
+                       break;
+       }
+       return (i);
+}
+
+static void
+ctl_be_block_compare(union ctl_io *io)
+{
+       struct ctl_be_block_io *beio;
+       uint64_t off, res;
+       int i;
+       uint8_t info[8];
+
+       beio = (struct ctl_be_block_io *)PRIV(io)->ptr;
+       off = 0;
+       for (i = 0; i < beio->num_segs; i++) {
+               res = cmp(beio->sg_segs[i].addr,
+                   beio->sg_segs[i + CTLBLK_HALF_SEGS].addr,
+                   beio->sg_segs[i].len);
+               off += res;
+               if (res < beio->sg_segs[i].len)
+                       break;
+       }
+       if (i < beio->num_segs) {
+               scsi_u64to8b(off, info);
+               ctl_set_sense(&io->scsiio, /*current_error*/ 1,
+                   /*sense_key*/ SSD_KEY_MISCOMPARE,
+                   /*asc*/ 0x1D, /*ascq*/ 0x00,
+                   /*type*/ SSD_ELEM_INFO,
+                   /*size*/ sizeof(info), /*data*/ &info,
+                   /*type*/ SSD_ELEM_NONE);
+       } else
+               ctl_set_success(&io->scsiio);
+}
+
 static int
 ctl_be_block_move_done(union ctl_io *io)
 {
@@ -363,7 +405,6 @@ ctl_be_block_move_done(union ctl_io *io)
 #ifdef CTL_TIME_IO
        struct bintime cur_bt;
 #endif
-       int i;
 
        beio = (struct ctl_be_block_io *)PRIV(io)->ptr;
        be_lun = beio->lun;
@@ -391,21 +432,7 @@ ctl_be_block_move_done(union ctl_io *io)
                        ctl_set_success(&io->scsiio);
                } else if (lbalen->flags & CTL_LLF_COMPARE) {
                        /* We have two data blocks ready for comparison. */
-                       for (i = 0; i < beio->num_segs; i++) {
-                               if (memcmp(beio->sg_segs[i].addr,
-                                   beio->sg_segs[i + CTLBLK_HALF_SEGS].addr,
-                                   beio->sg_segs[i].len) != 0)
-                                       break;
-                       }
-                       if (i < beio->num_segs)
-                               ctl_set_sense(&io->scsiio,
-                                   /*current_error*/ 1,
-                                   /*sense_key*/ SSD_KEY_MISCOMPARE,
-                                   /*asc*/ 0x1D,
-                                   /*ascq*/ 0x00,
-                                   SSD_ELEM_NONE);
-                       else
-                               ctl_set_success(&io->scsiio);
+                       ctl_be_block_compare(io);
                }
        } else if ((io->io_hdr.port_status != 0) &&
            ((io->io_hdr.status & CTL_STATUS_MASK) == CTL_STATUS_NONE ||
_______________________________________________
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