lkml.org 
[lkml]   [2021]   [Sep]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v5 2/8] configfs: use BIT() for internal flags
On Mon, Sep 20, 2021 at 4:50 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
>
> Hi Bartosz,
>
> On Mon, Sep 20, 2021 at 4:47 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
> > On Mon, Sep 20, 2021 at 4:30 PM Christoph Hellwig <hch@lst.de> wrote:
> > > On Mon, Sep 20, 2021 at 04:29:30PM +0200, Geert Uytterhoeven wrote:
> > > > On Mon, Sep 20, 2021 at 4:09 PM Christoph Hellwig <hch@lst.de> wrote:
> > > > > On Mon, Sep 20, 2021 at 04:05:03PM +0200, Bartosz Golaszewski wrote:
> > > > > > For better readability and maintenance: use the BIT() macro for flag
> > > > > > definitions.
> > > > >
> > > > > NAK. BIT() is the stupidest macro in the kernel and shall not be used
> > > > > ever. And I'm pretty sure we had this discussion a few times.
> > > >
> > > > Care to explain why it is a stupid macro?
> > >
> > > Please look at the previous thread. I'm tired of this discussion.
> >
> > The only previous answer from Christoph is this:
> > https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg2399968.html
>
> Yep, found that one, too.
>
> Now, as these definitions do not correspond to hardware register bits,
> perhaps use an enum?
>

These are flags, we can set more than one to any given dirent.

Anyway, I'm more interested in Al's comments on the implementation of
committable items than bikeshedding about whether to use or not to use
BIT() so I'll wait for those before revisiting this patch.

Bart

\
 
 \ /
  Last update: 2021-09-20 17:02    [W:0.882 / U:0.008 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site