Author: mav
Date: Fri Oct 14 07:31:47 2016
New Revision: 307284
URL: https://svnweb.freebsd.org/changeset/base/307284

Log:
  MFC r305334: MFV r304157:
  7230 add assertions to dmu_send_impl() to verify that stream includes BEGIN 
and END records
  
  illumos/illumos-gate@12b90ee2d3b10689fc45f4930d2392f5fe1d9cfa
  
https://github.com/illumos/illumos-gate/commit/12b90ee2d3b10689fc45f4930d2392f5f
  e1d9cfa
  
  https://www.illumos.org/issues/7230
    A test failure occurred where a send stream had only a BEGIN record. This
    should not be possible if the send returns without error. Prevented this 
from
    happening in the future by adding an assertion to dmu_send_impl() to verify
    that if the function returns 0 (success) both a BEGIN and END record are
    present. Did this by adding flags to dmu_sendarg_t (indicating whether 
BEGIN o
  r
    END records sent), having dump_record() set flags appropriately, adding 
VERIFY
    statement to dmu_send_impl().
  
  Reviewed by: Matthew Ahrens <mahr...@delphix.com>
  Reviewed by: Paul Dagnelie <p...@delphix.com>
  Reviewed by: Igor Kozhukhov <ikozhuk...@gmail.com>
  Approved by: Robert Mustacchi <r...@joyent.com>
  Author: Matt Krantz <matt.kra...@delphix.com>

Modified:
  stable/11/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dmu_send.c
  stable/11/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/dmu_impl.h
Directory Properties:
  stable/11/   (props changed)

Modified: stable/11/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dmu_send.c
==============================================================================
--- stable/11/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dmu_send.c Fri Oct 
14 07:30:54 2016        (r307283)
+++ stable/11/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dmu_send.c Fri Oct 
14 07:31:47 2016        (r307284)
@@ -160,11 +160,16 @@ dump_record(dmu_sendarg_t *dsp, void *pa
        fletcher_4_incremental_native(dsp->dsa_drr,
            offsetof(dmu_replay_record_t, drr_u.drr_checksum.drr_checksum),
            &dsp->dsa_zc);
-       if (dsp->dsa_drr->drr_type != DRR_BEGIN) {
+       if (dsp->dsa_drr->drr_type == DRR_BEGIN) {
+               dsp->dsa_sent_begin = B_TRUE;
+       } else {
                ASSERT(ZIO_CHECKSUM_IS_ZERO(&dsp->dsa_drr->drr_u.
                    drr_checksum.drr_checksum));
                dsp->dsa_drr->drr_u.drr_checksum.drr_checksum = dsp->dsa_zc;
        }
+       if (dsp->dsa_drr->drr_type == DRR_END) {
+               dsp->dsa_sent_end = B_TRUE;
+       }
        fletcher_4_incremental_native(&dsp->dsa_drr->
            drr_u.drr_checksum.drr_checksum,
            sizeof (zio_cksum_t), &dsp->dsa_zc);
@@ -912,6 +917,8 @@ out:
        list_remove(&to_ds->ds_sendstreams, dsp);
        mutex_exit(&to_ds->ds_sendstream_lock);
 
+       VERIFY(err != 0 || (dsp->dsa_sent_begin && dsp->dsa_sent_end));
+
        kmem_free(drr, sizeof (dmu_replay_record_t));
        kmem_free(dsp, sizeof (dmu_sendarg_t));
 

Modified: 
stable/11/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/dmu_impl.h
==============================================================================
--- stable/11/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/dmu_impl.h     
Fri Oct 14 07:30:54 2016        (r307283)
+++ stable/11/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/dmu_impl.h     
Fri Oct 14 07:31:47 2016        (r307284)
@@ -301,6 +301,8 @@ typedef struct dmu_sendarg {
        uint64_t dsa_last_data_offset;
        uint64_t dsa_resume_object;
        uint64_t dsa_resume_offset;
+       boolean_t dsa_sent_begin;
+       boolean_t dsa_sent_end;
 } dmu_sendarg_t;
 
 void dmu_object_zapify(objset_t *, uint64_t, dmu_object_type_t, dmu_tx_t *);
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to