Re: [bisected] 3c8ba0d61d04ced9f8d9ff93977995a9e4e96e91 oopses on s390
From: Sebastian Ott
Date: Mon Apr 09 2018 - 13:27:57 EST
On Mon, 9 Apr 2018, Linus Torvalds wrote:
> On Mon, Apr 9, 2018 at 10:03 AM, Linus Torvalds
> <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote:
> >
> > Our old "min()" had the internal variables called "min1" and "min2",
> > which is crazy too.
>
> Actually, no, it used the really cumbersome "__UNIQUE_ID" and then
> passed that odd as the name 'min1/2',
>
> Ugh, I find that really nasty to read, but it was obviously done
> because we hit this before.
>
> And our __UNIQUE_ID() macro is garbage anyway, since it falls back on
> the line number, which doesn't really work for macros anyway. But we
> have proper macros for both clang and gcc, so maybe we should ignore
> the broken fallback.
>
> A patch like the attached, perhaps?
I applied this on top of 38c23685b273cfb4ccf31a199feccce3bdcb5d83
and everything works as expected.
Thanks!