andy-js commented on this pull request.
> @@ -1091,7 +1091,9 @@ dmu_object_remap_indirects(objset_t *os, uint64_t
> object,
return (err);
}
- if (dn->dn_nlevels <= 1) {
+ rw_enter(&dn->dn_struct_rwlock, RW_WRITER);
+ if (dn->dn_phys->dn_nlevels <= 1) {
+ rw_exit(&dn->dn_struct_rwlock);
Shouldn't there be another rw_exit at the end of this if?
--
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/603#pullrequestreview-108085342
------------------------------------------
openzfs: openzfs-developer
Permalink:
https://openzfs.topicbox.com/groups/developer/discussions/T79296838de064147-M39c4274b35ad1c988e09f66d
Delivery options: https://openzfs.topicbox.com/groups
