Re: [PATCH v2 04/16] fs: Increase fmode_t size

From: Christoph Hellwig
Date: Wed Dec 13 2023 - 11:04:05 EST


On Wed, Dec 13, 2023 at 02:02:31PM +0100, Christian Brauner wrote:
> > typedef unsigned int __bitwise gfp_t;
> > typedef unsigned int __bitwise slab_flags_t;
> > -typedef unsigned int __bitwise fmode_t;
> > +typedef unsigned long __bitwise fmode_t;
>
> As Jan said, that's likely a bad idea. There's a bunch of places that
> assume fmode_t is 32bit. So not really a change we want to make if we
> can avoid it.

Oh well, let me dust of my series to move the fairly static flags out
of it. But even without that do we even need to increase it? There's
still quite a lot of space after FMODE_EXEC for example.