Re: [GIT PULL] xfs: fixes for 5.18-rc5

[Dave Chinner]

On Thu, Apr 28, 2022 at 09:46:14AM -0700, Linus Torvalds wrote:
> On Wed, Apr 27, 2022 at 11:19 PM Dave Chinner <david@xxxxxxxxxxxxx> wrote:
> >
> > It's all minor stuff;  a 5.18 build regession fix, a deadlock fix
> > and an update to remove redundant fields from the XFS entry in the
> > MAINTAINERS file.
> 
> Done.

Thanks!

> I had to look twice at that patch going "why does it complain about
> the xfs flags thing" until I realized it was due to XBF_UNMAPPED being
> (1 <<31) and the compiler then seeing explicit negative values being
> assigned.
> 
> We have a lot of "int flags" in various places, very much not just
> xfs, and yeah, we should probably try to clean them up.

Yup, I've already got a bunch of similar cleanup patches for
XFS ready for the next merge window.

> But 99% of the time it's just not worth the noise.

Mostly I agree, but in coverting another dozen+ sets of flags the
biggest benefit I've seen comes from the bit flag definitions all
having a consistent format across the code base.

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx