On Tue, Nov 10, 2020 at 04:44:56PM -0800, Darrick J. Wong wrote: > - e = attr_remove_path(&f, aname, ATTR_DONTFOLLOW) < 0 ? errno : 0; > + e = attr_remove_path(&f, aname) < 0 ? errno : 0; > check_cwd(); > if (v) > printf("%d/%d: attr_remove %s %s %d\n", Same comment as for the last one, otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>