ahrens commented on this pull request.
> @@ -392,6 +392,22 @@ static int dump_freeobjects(dmu_sendarg_t *dsp, uint64_t firstobj, uint64_t numobjs) { struct drr_freeobjects *drrfo = &(dsp->dsa_drr->drr_u.drr_freeobjects); + uint64_t maxobj = DNODES_PER_BLOCK * + (DMU_META_DNODE(dsp->dsa_os)->dn_maxblkid + 1); + + /* Yes, we should update the comment. The simplest change would be to say `Older versions of ZFS did not handle ...`. If we want to be more specific, we could say `Prior to the fix for bug 1234, ZFS did not handle...` -- 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/685#discussion_r217824770 ------------------------------------------ openzfs: openzfs-developer Permalink: https://openzfs.topicbox.com/groups/developer/Tcf25f566d2c2fe2d-Mca410141593f8e40024a5864 Delivery options: https://openzfs.topicbox.com/groups/developer/subscription