Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20210511203716.117010-1-rikard.falkeborn@gmail.com>
 <CAMj1kXF0rMwjgm27=i3XkrXJ=21C_x4he5Ls+7FSKUhsva970Q@mail.gmail.com>
 <YKaukFKw4KTmnOVd@rikard> <YKa8JjT9AFOspIl5@yury-ThinkPad>
In-Reply-To: <YKa8JjT9AFOspIl5@yury-ThinkPad>
From:   Arnd Bergmann <arnd@arndb.de>
Date:   Thu, 20 May 2021 22:15:02 +0200
Message-ID: <CAK8P3a1i1XE_rUrg_kX87NPA7NBJa5m2gVrHSuBE3XzFLUKPWA@mail.gmail.com>
Subject: Re: [PATCH] linux/bits.h: Fix compilation error with GENMASK
To:     Yury Norov <yury.norov@gmail.com>
Cc:     Rikard Falkeborn <rikard.falkeborn@gmail.com>,
        Ard Biesheuvel <ardb@kernel.org>,
        Andrew Morton <akpm@linux-foundation.org>,
        Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
        Tetsuo Handa <penguin-kernel@i-love.sakura.ne.jp>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Peter Zijlstra <peterz@infradead.org>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Thu, May 20, 2021 at 9:44 PM Yury Norov <yury.norov@gmail.com> wrote:
> On Thu, May 20, 2021 at 08:46:40PM +0200, Rikard Falkeborn wrote:
> > > > +/*
> > > > + * This returns a constant expression while determining if an argument is
> > > > + * a constant expression, most importantly without evaluating the argument.
> > > > + * Glory to Martin Uecker <Martin.Uecker@med.uni-goettingen.de>
> > > > + */
> > > > +#define __is_constexpr(x) \
> > > > +       (sizeof(int) == sizeof(*(8 ? ((void *)((long)(x) * 0l)) : (int *)8)))
> > > > +
> > > >  #endif /* _LINUX_CONST_H */
> > > > --
> > > > 2.31.1
> > > >
> >
> > Friendly ping.
>
> I added this patch in my Bitmap tree, but since it's actually a build
> bug fix, I think it's worth to move it faster with Arnd's or Andrew's
> trees?

I have a few days off next week and won't be able to validate it before then,
so it would be faster to have Andrew pick it up.

Moving the definition to const.h seem reasonable to me here, feel
free to add

Acked-by: Arnd Bergmann <arnd@arndb.de>