Re: [RFC] pgflags_t

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Oct 06, 2021 at 03:22:29PM +0000, Al Viro wrote:
> On Wed, Oct 06, 2021 at 03:58:14PM +0100, Matthew Wilcox wrote:
> > David expressed some unease about the lack of typesafety in patches
> > 1 & 2 of the page->slab conversion [1], and I'll admit to not being
> > particularly a fan of passing around an unsigned long.  That crystallised
> > in a discussion with Kent [2] about how to lock a page when you don't know
> > its type (solution: every memory descriptor type starts with a
> > pgflags_t)
> 
> Why bother making it a struct?  What's wrong with __bitwise and letting
> sparse catch conversions?

People don't run sparse.  I happen to have a built allmodconfig tree
here and running make C=2 fs/ gives 1147 lines of warnings.  Why would
adding more warnings help?





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux