Message ID | 20200313211709.148967-1-josef@toxicpanda.com (mailing list archive) |
---|---|
Headers | show |
Series | Relocation and backref resolution fixes | expand |
On Fri, Mar 13, 2020 at 05:17:05PM -0400, Josef Bacik wrote: > These are standalone fixes that came out of my debugging Zygo's problems. The > first two address a problem with how we handle restarting relocation. > Previously this rarely happened, because if it had people would have complained. > The restart logic was broken in a few subtle ways, and these two patches address > those issues. > > The third patch just boggles my mind. We were recording reloc roots based on > their current bytenr. This worked fine if we never restarted, but broke if we > had to lookup a ref to a reloc root that we found on the tree. This is because > that would point at the commit root of the reloc root, but if we had modified > the reloc root we'd no longer be able to find it. > > And finally the last one was a weird deadlock that Zygo's insane test rig found, > as he runs the dedup thing while balancing and deleting snapshots, which made > this thing fall out. Thanks, This series is now in misc-next, thanks.