pcd1193182 commented on this pull request.


> +                     dbn->dbn_dirty = B_FALSE;
+               }
+       }
+#ifdef ZFS_DEBUG
+       for (dsl_bookmark_node_t *dbn = avl_first(&ds->ds_bookmarks);
+           dbn != NULL; dbn = AVL_NEXT(&ds->ds_bookmarks, dbn)) {
+               ASSERT(!dbn->dbn_dirty);
+       }
+#endif
+}
+
+/*
+ * Return the TXG of the most recent bookmark (or 0 if there are no bookmarks).
+ */
+uint64_t
+dsl_bookmark_latest_txg(dsl_dataset_t *ds)

I'm also a bit confused about this comment; is it a question about how 
ds_bookmarks is protected? Or is it referencing the wrong section of code as 
well?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/484#discussion_r152379555
------------------------------------------
openzfs-developer
Archives: 
https://openzfs.topicbox.com/groups/developer/discussions/T1fcfd4fc81bcdfff-M1120a80607dd9286f52f2609
Powered by Topicbox: https://topicbox.com

Reply via email to