avg-I commented on this pull request.


>        */
-       error = dmu_tx_assign(tx, TXG_WAITED);
-       if (error != 0) {
-               ASSERT3S(error, ==, EIO);
-               dmu_tx_abort(tx);
-               return (NULL);
+       while ((error = dmu_tx_assign(tx, TXG_WAITED)) != 0) {
+               VERIFY3S(error, ==, ERESTART);
+               dmu_tx_unassign(tx);

And it seems that if `dmu_tx_assign` returns an error, then `dmu_tx_unassign` 
has already been called by it.

-- 
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/507#discussion_r158336442
------------------------------------------
openzfs-developer
Archives: 
https://openzfs.topicbox.com/groups/developer/discussions/T05cff66aac5a33a7-Md4e6d5daebf0745ce1238407
Powered by Topicbox: https://topicbox.com

Reply via email to