summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authormav <mav@FreeBSD.org>2017-07-26 17:36:30 +0000
committermav <mav@FreeBSD.org>2017-07-26 17:36:30 +0000
commit178d676339ea9d88cf42000c2704089cdab356b6 (patch)
treeba999d8256a697867bf754ed11a9a41a6cb0b464
parent266c279b28f9c6c89b28790cacca8cc3b643825e (diff)
downloadFreeBSD-src-178d676339ea9d88cf42000c2704089cdab356b6.zip
FreeBSD-src-178d676339ea9d88cf42000c2704089cdab356b6.tar.gz
MFC r318935: 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.pankov@gmail.com> Reviewed by: Matthew Ahrens <mahrens@delphix.com> Approved by: Robert Mustacchi <rm@joyent.com> Author: Alan Somers <asomers@gmail.com>
-rw-r--r--sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dbuf.c5
-rw-r--r--sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dsl_scan.c4
2 files changed, 8 insertions, 1 deletions
diff --git a/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dbuf.c b/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dbuf.c
index ac70c1f..89b7873 100644
--- a/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dbuf.c
+++ b/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dbuf.c
@@ -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);
diff --git a/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dsl_scan.c b/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dsl_scan.c
index 4ebbc1a..1061cf6 100644
--- a/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dsl_scan.c
+++ b/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dsl_scan.c
@@ -807,7 +807,7 @@ dsl_scan_visitbp(blkptr_t *bp, const zbookmark_phys_t *zb,
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)
{
OpenPOWER on IntegriCloud