prakashsurya commented on this pull request.


-       ASSERT(zilog->zl_root_zio == NULL);
+       ASSERT(MUTEX_HELD(&zilog->zl_writer_lock));

Yes. I squash them to a single commit before pushing to illumos and openzfs. 
Leaving them split into multiple commits prior to merging can make it easier 
for folks to see the individual revisions for a change more easily (mostly to 
workaround a deficiency in GitHub's UX when PRs are rebased).

-- 
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/447#discussion_r136607929
------------------------------------------
openzfs-developer
Archives: 
https://openzfs.topicbox.com/groups/developer/discussions/Tb8f4a0d0bed9b5eb-Mb94c0787fada9924b63128a2
Powered by Topicbox: https://topicbox.com

Reply via email to