Author: avg
Date: Fri May 26 11:49:42 2017
New Revision: 318935
URL: https://svnweb.freebsd.org/changeset/base/318935

Log:
  MFV r318934: 8070 Add some ZFS comments
  
  illumos/illumos-gate@40713f2b249d289022c715107b3951055a63aef0
  
https://github.com/illumos/illumos-gate/commit/40713f2b249d289022c715107b3951055a63aef0
  
  https://www.illumos.org/issues/8070
    Add some ZFS comments left by various developers at different times
  
  Reviewed by: Yuri Pankov <yuri.pan...@gmail.com>
  Reviewed by: Matthew Ahrens <mahr...@delphix.com>
  Approved by: Robert Mustacchi <r...@joyent.com>
  Author: Alan Somers <asom...@gmail.com>
  
  MFC after:    1 week

Modified:
  head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dbuf.c
  head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dsl_scan.c
Directory Properties:
  head/sys/cddl/contrib/opensolaris/   (props changed)

Modified: head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dbuf.c
==============================================================================
--- head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dbuf.c  Fri May 26 
11:48:29 2017        (r318934)
+++ head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dbuf.c  Fri May 26 
11:49:42 2017        (r318935)
@@ -1219,6 +1219,11 @@ dbuf_unoverride(dbuf_dirty_record_t *dr)
        uint64_t txg = dr->dr_txg;
 
        ASSERT(MUTEX_HELD(&db->db_mtx));
+       /*
+        * This assert is valid because dmu_sync() expects to be called by
+        * a zilog's get_data while holding a range lock.  This call only
+        * comes from dbuf_dirty() callers who must also hold a range lock.
+        */
        ASSERT(dr->dt.dl.dr_override_state != DR_IN_DMU_SYNC);
        ASSERT(db->db_level == 0);
 

Modified: head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dsl_scan.c
==============================================================================
--- head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dsl_scan.c      Fri May 
26 11:48:29 2017        (r318934)
+++ head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dsl_scan.c      Fri May 
26 11:49:42 2017        (r318935)
@@ -807,7 +807,7 @@ dsl_scan_visitbp(blkptr_t *bp, const zbo
                return;
 
        /*
-        * If dsl_scan_ddt() has aready visited this block, it will have
+        * If dsl_scan_ddt() has already visited this block, it will have
         * already done any translations or scrubbing, so don't call the
         * callback again.
         */
@@ -1474,6 +1474,7 @@ dsl_scan_active(dsl_scan_t *scn)
        return (used != 0);
 }
 
+/* Called whenever a txg syncs. */
 void
 dsl_scan_sync(dsl_pool_t *dp, dmu_tx_t *tx)
 {
@@ -1892,6 +1893,7 @@ dsl_scan_scrub_cb(dsl_pool_t *dp,
        return (0);
 }
 
+/* Called by the ZFS_IOC_POOL_SCAN ioctl to start a scrub or resilver */
 int
 dsl_scan(dsl_pool_t *dp, pool_scan_func_t func)
 {
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to