Re: next build: 208 builds: 3 failed, 205 passed, 5 errors, 23 warnings (next-20170215)

Re: next build: 208 builds: 3 failed, 205 passed, 5 errors, 23 warnings (next-20170215)

[Arnd Bergmann]

On Wed, Feb 15, 2017 at 9:30 AM, kernelci.org bot <bot@kernelci.org> wrote:
> next build: 208 builds: 3 failed, 205 passed, 5 errors, 23 warnings

> Errors and Warnings Detected:
>
> arm64: gcc version 5.3.1 20160412 (Linaro GCC 5.3-2016.05)
> defconfig+CONFIG_KASAN=y 4 warnings
> arm: gcc version 5.3.1 20160412 (Linaro GCC 5.3-2016.05)
> allmodconfig 7 warnings
> imx_v6_v7_defconfig 2 warnings
> mips: gcc version 6.3.0 (GCC)
> bcm47xx_defconfig 1 error
> ip27_defconfig 2 errors
> xway_defconfig 2 errors
> x86: gcc version 4.8.4 (Ubuntu 4.8.4-2ubuntu1~14.04.1)
> allmodconfig 5 warnings
> allmodconfig+CONFIG_OF=n 5 warnings

A couple of extra problems here that Olof's build bot didn't find (I
replied on those
already, won't duplicate those). In total, we seem to be nine patches short of
a clean build now, which is the best we've had since kernelci added MIPS
builds (we had a clean build for one day before that).

> allmodconfig (arm) — PASS, 0 errors, 7 warnings, 0 section mismatches
>
> Warnings:
> drivers/iio/adc/rcar-gyroadc.c:429:27: warning: 'num_channels' may be used
> uninitialized in this function [-Wmaybe-uninitialized]
> drivers/iio/adc/rcar-gyroadc.c:426:22: warning: 'sample_width' may be used
> uninitialized in this function [-Wmaybe-uninitialized]
> drivers/iio/adc/rcar-gyroadc.c:428:23: warning: 'channels' may be used
> uninitialized in this function [-Wmaybe-uninitialized]
> drivers/iio/adc/rcar-gyroadc.c:398:26: warning: 'adcmode' may be used
> uninitialized in this function [-Wmaybe-uninitialized]
> drivers/media/platform/coda/imx-vdoa.c:333:181: warning: passing argument 1
> of '__platform_driver_register' discards 'const' qualifier from pointer
> target type [-Wdiscarded-qualifiers]
> drivers/media/platform/coda/imx-vdoa.c:333:625: warning: passing argument 1
> of 'platform_driver_unregister' discards 'const' qualifier from pointer
> target type [-Wdiscarded-qualifiers]

Covered in my reply to Olof's build bot.

> drivers/usb/gadget/udc/atmel_usba_udc.c:632:550: warning: 'ept_cfg' may be
> used uninitialized in this function [-Wmaybe-uninitialized]

My patch was acked, but not picked up by Felipe or Greg so far:

https://www.spinics.net/lists/linux-usb/msg153162.html

> allmodconfig (x86) — PASS, 0 errors, 5 warnings, 0 section mismatches
>
> Warnings:
> lib/raid6/recov_avx512.c:387:2: warning: #warning "your version of binutils
> lacks AVX512 support" [-Wcpp]

long-standing problem on the kernelci.org x86 toolchain, the only
solution is to update the build bot.

> drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:7263:2: warning: missing braces around
> initializer [-Wmissing-braces]
> drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:7263:2: warning: (near initialization
> for 'ce_payload.regular') [-Wmissing-braces]
> drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:7290:2: warning: missing braces around
> initializer [-Wmissing-braces]
> drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:7290:2: warning: (near initialization
> for 'de_payload.regular') [-Wmissing-braces]

This only happens on old compilers. I sent a patch, no reply so far:

https://patchwork.kernel.org/patch/9554695/

> bcm47xx_defconfig (mips) — FAIL, 1 error, 0 warnings, 0 section mismatches
>
> Errors:
> /home/buildslave/workspace/khilman-kbuilder/next/build-mips/../net/sched/sch_fq_codel.c:468:
> undefined reference to `tcf_destroy_chain'

I have not looked at this one yet, appears to be caused by commit
cf1facda2f61 ("sched: move tcf_proto_destroy and tcf_destroy_chain
helpers into cls_api")

> defconfig+CONFIG_KASAN=y (arm64) — PASS, 0 errors, 4 warnings, 0 section
> mismatches
>
> Warnings:
> net/bridge/br_netlink.c:1339:1: warning: the frame size of 2544 bytes is
> larger than 2048 bytes [-Wframe-larger-than=]
> net/wireless/nl80211.c:1415:1: warning: the frame size of 2208 bytes is
> larger than 2048 bytes [-Wframe-larger-than=]
> net/wireless/nl80211.c:1904:1: warning: the frame size of 3840 bytes is
> larger than 2048 bytes [-Wframe-larger-than=]
> net/wireless/nl80211.c:4443:1: warning: the frame size of 2240 bytes is
> larger than 2048 bytes [-Wframe-larger-than=]

I have an experimental patch, still working on a better solution. This might
take some more time, but it's been like this from the start with KASAN.

> ip27_defconfig (mips) — FAIL, 2 errors, 0 warnings, 0 section mismatches
>
> Errors:
> drivers/net/ethernet/qlogic/qlge/qlge_main.c:4819:1: error: insn does not
> satisfy its constraints:
> drivers/net/ethernet/qlogic/qlge/qlge_main.c:4819:1: internal compiler
> error: in extract_constrain_insn, at recog.c:2190

broken gcc release, apparently fixed in gcc-7 (can't reproduce here at least).
I suggested a workaround, but got no reply so far:

http://www.spinics.net/lists/mips/msg66465.html

> xway_defconfig (mips) — FAIL, 2 errors, 0 warnings, 0 section mismatches
>
> Errors:
> (.text+0x14e10): undefined reference to `physical_memsize'
> (.text+0x14e14): undefined reference to `physical_memsize'

Hauke already did a patch in December, but it has't made it into linux-mips
so far:

https://marc.info/?l=linux-mips&m=148612428414708&w=3

     Arnd

Re: next build: 208 builds: 3 failed, 205 passed, 5 errors, 23 warnings (next-20170215)

[James Hogan]

[-- Attachment #1: Type: text/plain, Size: 2280 bytes --]

On Wed, Feb 15, 2017 at 09:56:48AM +0100, Arnd Bergmann wrote:
> On Wed, Feb 15, 2017 at 9:30 AM, kernelci.org bot <bot@kernelci.org> wrote:
> > bcm47xx_defconfig (mips) — FAIL, 1 error, 0 warnings, 0 section mismatches
> >
> > Errors:
> > /home/buildslave/workspace/khilman-kbuilder/next/build-mips/../net/sched/sch_fq_codel.c:468:
> > undefined reference to `tcf_destroy_chain'
> 
> I have not looked at this one yet, appears to be caused by commit
> cf1facda2f61 ("sched: move tcf_proto_destroy and tcf_destroy_chain
> helpers into cls_api")

So effectively CONFIG_NET_SCH_FQ_CODEL (which bcm47xx_defconfig sets
=y), and just over half of the other packet schedulers now implicitly
depend on CONFIG_NET_CLS (which bcm47xx_defconfig doesn't set =y).

Perhaps revert would be best since the change looks of questionable
value to me, as all the users of it are in sch_*.c anyway.

> > ip27_defconfig (mips) — FAIL, 2 errors, 0 warnings, 0 section mismatches
> >
> > Errors:
> > drivers/net/ethernet/qlogic/qlge/qlge_main.c:4819:1: error: insn does not
> > satisfy its constraints:
> > drivers/net/ethernet/qlogic/qlge/qlge_main.c:4819:1: internal compiler
> > error: in extract_constrain_insn, at recog.c:2190
> 
> broken gcc release, apparently fixed in gcc-7 (can't reproduce here at least).
> I suggested a workaround, but got no reply so far:
> 
> http://www.spinics.net/lists/mips/msg66465.html

I'll look into that.

> 
> > xway_defconfig (mips) — FAIL, 2 errors, 0 warnings, 0 section mismatches
> >
> > Errors:
> > (.text+0x14e10): undefined reference to `physical_memsize'
> > (.text+0x14e14): undefined reference to `physical_memsize'
> 
> Hauke already did a patch in December, but it has't made it into linux-mips
> so far:
> 
> https://marc.info/?l=linux-mips&m=148612428414708&w=3

Hauke: you mentioned sending a new version of this patch. I'll apply the
original patch for now, as looking at the VPE loader, zero doesn't sound
entirely unreasonable:

/*
 * The sde-kit passes 'memsize' to __start in $a3, so set something
 * here...  Or set $a3 to zero and define DFLT_STACK_SIZE and
 * DFLT_HEAP_SIZE when you compile your program
 */
mttgpr(6, v->ntcs);
mttgpr(7, physical_memsize);

Cheers
James

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

Re: next build: 208 builds: 3 failed, 205 passed, 5 errors, 23 warnings (next-20170215)

[Jiri Pirko]

Wed, Feb 15, 2017 at 10:38:07AM CET, james.hogan@imgtec.com wrote:
>On Wed, Feb 15, 2017 at 09:56:48AM +0100, Arnd Bergmann wrote:
>> On Wed, Feb 15, 2017 at 9:30 AM, kernelci.org bot <bot@kernelci.org> wrote:
>> > bcm47xx_defconfig (mips) — FAIL, 1 error, 0 warnings, 0 section mismatches
>> >
>> > Errors:
>> > /home/buildslave/workspace/khilman-kbuilder/next/build-mips/../net/sched/sch_fq_codel.c:468:
>> > undefined reference to `tcf_destroy_chain'
>> 
>> I have not looked at this one yet, appears to be caused by commit
>> cf1facda2f61 ("sched: move tcf_proto_destroy and tcf_destroy_chain
>> helpers into cls_api")
>
>So effectively CONFIG_NET_SCH_FQ_CODEL (which bcm47xx_defconfig sets
>=y), and just over half of the other packet schedulers now implicitly
>depend on CONFIG_NET_CLS (which bcm47xx_defconfig doesn't set =y).
>
>Perhaps revert would be best since the change looks of questionable
>value to me, as all the users of it are in sch_*.c anyway.

Fixing this now.


>
>> > ip27_defconfig (mips) — FAIL, 2 errors, 0 warnings, 0 section mismatches
>> >
>> > Errors:
>> > drivers/net/ethernet/qlogic/qlge/qlge_main.c:4819:1: error: insn does not
>> > satisfy its constraints:
>> > drivers/net/ethernet/qlogic/qlge/qlge_main.c:4819:1: internal compiler
>> > error: in extract_constrain_insn, at recog.c:2190
>> 
>> broken gcc release, apparently fixed in gcc-7 (can't reproduce here at least).
>> I suggested a workaround, but got no reply so far:
>> 
>> http://www.spinics.net/lists/mips/msg66465.html
>
>I'll look into that.
>
>> 
>> > xway_defconfig (mips) — FAIL, 2 errors, 0 warnings, 0 section mismatches
>> >
>> > Errors:
>> > (.text+0x14e10): undefined reference to `physical_memsize'
>> > (.text+0x14e14): undefined reference to `physical_memsize'
>> 
>> Hauke already did a patch in December, but it has't made it into linux-mips
>> so far:
>> 
>> https://marc.info/?l=linux-mips&m=148612428414708&w=3
>
>Hauke: you mentioned sending a new version of this patch. I'll apply the
>original patch for now, as looking at the VPE loader, zero doesn't sound
>entirely unreasonable:
>
>/*
> * The sde-kit passes 'memsize' to __start in $a3, so set something
> * here...  Or set $a3 to zero and define DFLT_STACK_SIZE and
> * DFLT_HEAP_SIZE when you compile your program
> */
>mttgpr(6, v->ntcs);
>mttgpr(7, physical_memsize);
>
>Cheers
>James