On Wed, Sep 16, 2020 at 05:10:42PM -0700, Darrick J. Wong wrote: > Please don't go adding more flags for a debugging check. > > You /could/ just detect intent-done items by the fact that their item > ops don't have unpin or push methods, kind of like what we do for > detecting intent items in log recovery. > > Oh wait, you mowed /that/ down too. Yes, I'd much rather change based on method presence instead of creating redundant flags.