Re: mainline build: 15 warnings 0 failures (mainline/v4.16-rc2-64-gaf3e79d)

Re: mainline build: 15 warnings 0 failures (mainline/v4.16-rc2-64-gaf3e79d)

[Arnd Bergmann]

On Tue, Feb 20, 2018 at 7:34 PM, Olof's autobuilder <build@lixom.net> wrote:
> Here are the build results from automated periodic testing.
>
> The tree being built was mainline, found at:
>
> URL: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
>
>
> Warnings:
>
>         arm64.allmodconfig:
> include/uapi/linux/byteorder/big_endian.h:31:52: warning: passing argument 1 of '__fswab64' makes integer from pointer without a cast [-Wint-conversion]

I sent two patches to the bnxt_re maintainers yesterday. I expect them
to get applied
soon, but haven't heard back yet.

> WARNING: modpost: missing MODULE_LICENSE() in drivers/phy/qualcomm/phy-qcom-ufs.o

Still waiting for Kishon to apply an older patch I sent for this.

>         arm.allmodconfig:
> /tmp/ccQBDjc9.s:18123: Warning: using r15 results in unpredictable behaviour
> /tmp/ccQBDjc9.s:18195: Warning: using r15 results in unpredictable behaviour

I have done multiple versions of a patch for this, none was
universally liked, but
we need to decide on one of them.

> include/linux/bitops.h:7:24: warning: left shift count >= width of type [-Wshift-count-overflow]
> include/linux/bitops.h:7:24: warning: left shift count >= width of type [-Wshift-count-overflow]
> include/linux/bitops.h:7:24: warning: left shift count >= width of type [-Wshift-count-overflow]
> drivers/infiniband/hw/bnxt_re/qplib_fp.c:333:23: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
> drivers/infiniband/hw/bnxt_re/qplib_fp.c:336:12: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
> include/uapi/linux/swab.h:127:46: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]

Also bnxt_re, same as above

> drivers/infiniband/core/cq.c:78:1: warning: the frame size of 1032 bytes is larger than 1024 bytes [-Wframe-larger-than=]

Sent a patch yesterday, my workaround was suboptimal, discussion about
a better fix
is ongoing.

>         arm.spear3xx_defconfig:
> net/core/filter.c:3382:6: warning: unused variable 'val' [-Wunused-variable]
>
>         arm.spear6xx_defconfig:
> net/core/filter.c:3382:6: warning: unused variable 'val' [-Wunused-variable]
>
>         arm.tct_hammer_defconfig:
> net/core/filter.c:3382:6: warning: unused variable 'val' [-Wunused-variable]

Sent two versions of a patch yesterday, the second version was applied and
should make it into mainline next week.

       Arnd

Re: mainline build: 15 warnings 0 failures (mainline/v4.16-rc2-64-gaf3e79d)

[Kishon Vijay Abraham I]

On Wednesday 21 February 2018 06:12 PM, Arnd Bergmann wrote:
> On Tue, Feb 20, 2018 at 7:34 PM, Olof's autobuilder <build@lixom.net> wrote:
>> Here are the build results from automated periodic testing.
>>
>> The tree being built was mainline, found at:
>>
>> URL: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
>>
>>
>> Warnings:
>>
>>         arm64.allmodconfig:
>> include/uapi/linux/byteorder/big_endian.h:31:52: warning: passing argument 1 of '__fswab64' makes integer from pointer without a cast [-Wint-conversion]
> 
> I sent two patches to the bnxt_re maintainers yesterday. I expect them
> to get applied
> soon, but haven't heard back yet.
> 
>> WARNING: modpost: missing MODULE_LICENSE() in drivers/phy/qualcomm/phy-qcom-ufs.o
> 
> Still waiting for Kishon to apply an older patch I sent for this.

I had merged it locally. Now pushed everything.

Thanks
Kishon

> 
>>         arm.allmodconfig:
>> /tmp/ccQBDjc9.s:18123: Warning: using r15 results in unpredictable behaviour
>> /tmp/ccQBDjc9.s:18195: Warning: using r15 results in unpredictable behaviour
> 
> I have done multiple versions of a patch for this, none was
> universally liked, but
> we need to decide on one of them.
> 
>> include/linux/bitops.h:7:24: warning: left shift count >= width of type [-Wshift-count-overflow]
>> include/linux/bitops.h:7:24: warning: left shift count >= width of type [-Wshift-count-overflow]
>> include/linux/bitops.h:7:24: warning: left shift count >= width of type [-Wshift-count-overflow]
>> drivers/infiniband/hw/bnxt_re/qplib_fp.c:333:23: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
>> drivers/infiniband/hw/bnxt_re/qplib_fp.c:336:12: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
>> include/uapi/linux/swab.h:127:46: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
> 
> Also bnxt_re, same as above
> 
>> drivers/infiniband/core/cq.c:78:1: warning: the frame size of 1032 bytes is larger than 1024 bytes [-Wframe-larger-than=]
> 
> Sent a patch yesterday, my workaround was suboptimal, discussion about
> a better fix
> is ongoing.
> 
>>         arm.spear3xx_defconfig:
>> net/core/filter.c:3382:6: warning: unused variable 'val' [-Wunused-variable]
>>
>>         arm.spear6xx_defconfig:
>> net/core/filter.c:3382:6: warning: unused variable 'val' [-Wunused-variable]
>>
>>         arm.tct_hammer_defconfig:
>> net/core/filter.c:3382:6: warning: unused variable 'val' [-Wunused-variable]
> 
> Sent two versions of a patch yesterday, the second version was applied and
> should make it into mainline next week.
> 
>        Arnd
> 

Re: mainline build: 15 warnings 0 failures (mainline/v4.16-rc2-64-gaf3e79d)

[Arnd Bergmann]

On Wed, Feb 21, 2018 at 3:42 PM, Kishon Vijay Abraham I <kishon@ti.com> wrote:
>
>
> On Wednesday 21 February 2018 06:12 PM, Arnd Bergmann wrote:
>> On Tue, Feb 20, 2018 at 7:34 PM, Olof's autobuilder <build@lixom.net> wrote:
>>> Here are the build results from automated periodic testing.
>>>
>>> The tree being built was mainline, found at:
>>>
>>> URL: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
>>>
>>>
>>> Warnings:
>>>
>>>         arm64.allmodconfig:
>>> include/uapi/linux/byteorder/big_endian.h:31:52: warning: passing argument 1 of '__fswab64' makes integer from pointer without a cast [-Wint-conversion]
>>
>> I sent two patches to the bnxt_re maintainers yesterday. I expect them
>> to get applied
>> soon, but haven't heard back yet.
>>
>>> WARNING: modpost: missing MODULE_LICENSE() in drivers/phy/qualcomm/phy-qcom-ufs.o
>>
>> Still waiting for Kishon to apply an older patch I sent for this.
>
> I had merged it locally. Now pushed everything.

Hi Kishon,

kernelci still reports it as one of the remaining warnings for v4.16-rc,
any chance you can send this as a bugfix for the current release?

       Arnd

Re: mainline build: 15 warnings 0 failures (mainline/v4.16-rc2-64-gaf3e79d)

[Kishon Vijay Abraham I]

Hi Arnd,

On Wednesday 07 March 2018 04:55 AM, Arnd Bergmann wrote:
> On Wed, Feb 21, 2018 at 3:42 PM, Kishon Vijay Abraham I <kishon@ti.com> wrote:
>>
>>
>> On Wednesday 21 February 2018 06:12 PM, Arnd Bergmann wrote:
>>> On Tue, Feb 20, 2018 at 7:34 PM, Olof's autobuilder <build@lixom.net> wrote:
>>>> Here are the build results from automated periodic testing.
>>>>
>>>> The tree being built was mainline, found at:
>>>>
>>>> URL: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
>>>>
>>>>
>>>> Warnings:
>>>>
>>>>         arm64.allmodconfig:
>>>> include/uapi/linux/byteorder/big_endian.h:31:52: warning: passing argument 1 of '__fswab64' makes integer from pointer without a cast [-Wint-conversion]
>>>
>>> I sent two patches to the bnxt_re maintainers yesterday. I expect them
>>> to get applied
>>> soon, but haven't heard back yet.
>>>
>>>> WARNING: modpost: missing MODULE_LICENSE() in drivers/phy/qualcomm/phy-qcom-ufs.o
>>>
>>> Still waiting for Kishon to apply an older patch I sent for this.
>>
>> I had merged it locally. Now pushed everything.
> 
> Hi Kishon,
> 
> kernelci still reports it as one of the remaining warnings for v4.16-rc,
> any chance you can send this as a bugfix for the current release?

I had queued this for next. Sure, I'll try to send this for current -rc.

Thanks
Kishon

Re: mainline build: 15 warnings 0 failures (mainline/v4.16-rc2-64-gaf3e79d)

[Arnd Bergmann]

On Wed, Mar 7, 2018 at 12:02 PM, Kishon Vijay Abraham I <kishon@ti.com> wrote:
> On Wednesday 07 March 2018 04:55 AM, Arnd Bergmann wrote:
>> On Wed, Feb 21, 2018 at 3:42 PM, Kishon Vijay Abraham I <kishon@ti.com> wrote:
>>> On Wednesday 21 February 2018 06:12 PM, Arnd Bergmann wrote:
>>>> On Tue, Feb 20, 2018 at 7:34 PM, Olof's autobuilder <build@lixom.net> wrote:
>>>>> WARNING: modpost: missing MODULE_LICENSE() in drivers/phy/qualcomm/phy-qcom-ufs.o
>>>>
>>>> Still waiting for Kishon to apply an older patch I sent for this.
>>>
>>> I had merged it locally. Now pushed everything.
>>
>> Hi Kishon,
>>
>> kernelci still reports it as one of the remaining warnings for v4.16-rc,
>> any chance you can send this as a bugfix for the current release?
>
> I had queued this for next. Sure, I'll try to send this for current -rc.

Thanks a lot! I got a confirmation on one of the other remaining issues,
so we're down to one or two patches for a clean 4.16 build now,
plus probably another three that we need to get a clean 'randconfig'
build on x86 and arm64.

       Arnd