On Mon, Sep 23, 2019 at 08:48:36AM -0400, Brian Foster wrote: > Prior to this change (and commit 1baa2800e62d), something like an xattr > remote value block would not be considered user data. As of this change, > that is no longer the case. That seems reasonable on first thought (it > is user data after all), but I'm not so sure it's appropriate once you > look through some of the ways xfs_alloc_is_userdata() is used. True. Let's just revert the original patch for now, and sort out the mess of these flags properly later.