Re: [PATCH v3] bitfield.h: add FIELD_MAX() and field_max()

From: Alex Elder
Date: Wed Apr 01 2020 - 14:24:32 EST


On 4/1/20 12:35 PM, Nick Desaulniers wrote:
>> Define FIELD_MAX(), which supplies the maximum value that can be
>> represented by a field value. Define field_max() as well, to go
>> along with the lower-case forms of the field mask functions.
>>
>> Signed-off-by: Alex Elder <elder@xxxxxxxxxx>
>> Acked-by: Jakub Kicinski <kuba@xxxxxxxxxx>
>> ---
>> v3: Rebased on latest netdev-next/master.
>>
>> David, please take this into net-next as soon as possible. When the
>> IPA code was merged the other day this prerequisite patch was not
>> included, and as a result the IPA driver fails to build. Thank you.
>>
>> See: https://lkml.org/lkml/2020/3/10/1839
>>
>> -Alex
>
> In particular, this seems to now have regressed into mainline for the 5.7
> merge window as reported by Linaro's ToolChain Working Group's CI.
> Link: https://github.com/ClangBuiltLinux/linux/issues/963

Is the problem you're referring to the result of a build done
in the midst of a bisect?

The fix for this build error is currently present in the
torvalds/linux.git master branch:
6fcd42242ebc soc: qcom: ipa: kill IPA_RX_BUFFER_ORDER

I may be mistaken, but I believe this is the same problem I discussed
with Maxim Kuvyrkov this morning. A different build problem led to
an automated bisect, which conluded this was the cause because it
landed somewhere between the initial pull of the IPA code and the fix
I reference above.

-Alex