* mainline build: 137 builds: 0 failed, 137 passed, 296 warnings (v4.4-10454-g3e1e21c7bfcf) [not found] <56a1c8c2.6953c20a.25484.ffff85a9@mx.google.com> @ 2016-01-22 13:19 ` Arnd Bergmann 2016-01-22 17:23 ` Tyler Baker 0 siblings, 1 reply; 5+ messages in thread From: Arnd Bergmann @ 2016-01-22 13:19 UTC (permalink / raw) To: linux-arm-kernel On Thursday 21 January 2016 22:14:26 kernelci. org bot wrote: > Warnings summary: > > 224 <stdin>:1307:2: warning: #warning syscall copy_file_range not implemented [-Wcpp] I don't think anyone has sent this so far, sending a patch now. > 29 drivers/tty/serial/amba-pl011.c:190:27: warning: 'vendor_zte' defined but not used [-Wunused-variable] Patch sent a couple of days ago, got an Ack and waiting to be applied. > 12 net/bluetooth/mgmt.c:5493:8: warning: 'r192' may be used uninitialized in this function [-Wmaybe-uninitialized] > 12 net/bluetooth/mgmt.c:5493:8: warning: 'h192' may be used uninitialized in this function [-Wmaybe-uninitialized] This happens only in gcc-4.7 and gcc-4.8 > 5 crypto/wp512.c:987:1: warning: the frame size of 1168 bytes is larger than 1024 bytes [-Wframe-larger-than=] > 1 crypto/wp512.c:987:1: warning: the frame size of 1112 bytes is larger than 1024 bytes [-Wframe-larger-than=] and this one only in gcc-4.7 Any chance we can update the toolchain on kernelci.org to 4.9 or higher? I'm normally testing with gcc-5.2, which has a little better warnings. > 2 drivers/gpu/drm/gma500/cdv_intel_dp.c:868:2: warning: 'i2c_dp_aux_add_bus' is deprecated (declared at ../drivers/gpu/drm/gma500/cdv_intel_dp.c:224) [-Wdeprecated-declarations] x86-only, I'm not touching this. > 2 /usr/include/x86_64-linux-gnu/bits/stdio2.h:64:3: warning: call to __builtin___snprintf_chk will always overflow destination buffer [enabled by default] also x86-only, I don't even know what this comes from. > 1 mm/page_alloc.c:5707:34: warning: array subscript is below array bounds [-Warray-bounds] > 1 drivers/soc/qcom/spm.c:305:5: warning: 'found' may be used uninitialized in this function [-Wuninitialized] Added a patch to arm-soc/fixes. > 1 drivers/net/ethernet/marvell/mvneta.c:2910:30: warning: unused variable 'other_port' [-Wunused-variable] Sent a patch to netdev today. > 1 .config:930:warning: override: NOHIGHMEM changes choice state > 1 .config:929:warning: override: SLOB changes choice state > 1 .config:927:warning: override: KERNEL_XZ changes choice state > 1 .config:884:warning: override: SLOB changes choice state > 1 .config:839:warning: override: SLOB changes choice state > 1 .config:837:warning: override: KERNEL_XZ changes choice state Interesting: "make tinyconfig" first calls "make allnoconfig" and then calls "make tiny.config", which in turn calls scripts/kconfig/merge_config.sh -m .config kernel/configs/tiny.config and this overrides some choice statements, causing the warning. A better command would be scripts/kconfig/merge_config.sh -n kernel/configs/tiny.config which has the same effect but avoids the warning. Need to think about this some more. Arnd ^ permalink raw reply [flat|nested] 5+ messages in thread
* mainline build: 137 builds: 0 failed, 137 passed, 296 warnings (v4.4-10454-g3e1e21c7bfcf) 2016-01-22 13:19 ` mainline build: 137 builds: 0 failed, 137 passed, 296 warnings (v4.4-10454-g3e1e21c7bfcf) Arnd Bergmann @ 2016-01-22 17:23 ` Tyler Baker 2016-01-25 18:18 ` Kevin Hilman 0 siblings, 1 reply; 5+ messages in thread From: Tyler Baker @ 2016-01-22 17:23 UTC (permalink / raw) To: linux-arm-kernel On 22 January 2016 at 05:19, Arnd Bergmann <arnd@arndb.de> wrote: > On Thursday 21 January 2016 22:14:26 kernelci. org bot wrote: >> Warnings summary: >> >> 224 <stdin>:1307:2: warning: #warning syscall copy_file_range not implemented [-Wcpp] > > I don't think anyone has sent this so far, sending a patch now. > >> 29 drivers/tty/serial/amba-pl011.c:190:27: warning: 'vendor_zte' defined but not used [-Wunused-variable] > > Patch sent a couple of days ago, got an Ack and waiting to be applied. > >> 12 net/bluetooth/mgmt.c:5493:8: warning: 'r192' may be used uninitialized in this function [-Wmaybe-uninitialized] >> 12 net/bluetooth/mgmt.c:5493:8: warning: 'h192' may be used uninitialized in this function [-Wmaybe-uninitialized] > > This happens only in gcc-4.7 and gcc-4.8 > >> 5 crypto/wp512.c:987:1: warning: the frame size of 1168 bytes is larger than 1024 bytes [-Wframe-larger-than=] >> 1 crypto/wp512.c:987:1: warning: the frame size of 1112 bytes is larger than 1024 bytes [-Wframe-larger-than=] > > and this one only in gcc-4.7 > > Any chance we can update the toolchain on kernelci.org to 4.9 or higher? > I'm normally testing with gcc-5.2, which has a little better warnings. I'm ok with upgrading to 4.9 or higher. Kevin any issues with this? ^ permalink raw reply [flat|nested] 5+ messages in thread
* mainline build: 137 builds: 0 failed, 137 passed, 296 warnings (v4.4-10454-g3e1e21c7bfcf) 2016-01-22 17:23 ` Tyler Baker @ 2016-01-25 18:18 ` Kevin Hilman 2016-01-25 20:52 ` Arnd Bergmann 0 siblings, 1 reply; 5+ messages in thread From: Kevin Hilman @ 2016-01-25 18:18 UTC (permalink / raw) To: linux-arm-kernel Tyler Baker <tyler.baker@linaro.org> writes: > On 22 January 2016 at 05:19, Arnd Bergmann <arnd@arndb.de> wrote: >> On Thursday 21 January 2016 22:14:26 kernelci. org bot wrote: >>> Warnings summary: >>> >>> 224 <stdin>:1307:2: warning: #warning syscall copy_file_range not implemented [-Wcpp] >> >> I don't think anyone has sent this so far, sending a patch now. >> >>> 29 drivers/tty/serial/amba-pl011.c:190:27: warning: 'vendor_zte' defined but not used [-Wunused-variable] >> >> Patch sent a couple of days ago, got an Ack and waiting to be applied. >> >>> 12 net/bluetooth/mgmt.c:5493:8: warning: 'r192' may be used uninitialized in this function [-Wmaybe-uninitialized] >>> 12 net/bluetooth/mgmt.c:5493:8: warning: 'h192' may be used uninitialized in this function [-Wmaybe-uninitialized] >> >> This happens only in gcc-4.7 and gcc-4.8 >> >>> 5 crypto/wp512.c:987:1: warning: the frame size of 1168 bytes is larger than 1024 bytes [-Wframe-larger-than=] >>> 1 crypto/wp512.c:987:1: warning: the frame size of 1112 bytes is larger than 1024 bytes [-Wframe-larger-than=] >> >> and this one only in gcc-4.7 >> >> Any chance we can update the toolchain on kernelci.org to 4.9 or higher? >> I'm normally testing with gcc-5.2, which has a little better warnings. > > I'm ok with upgrading to 4.9 or higher. Kevin any issues with this? No objection in principle, but we probably can't do this across the board without fallout. I suspect we'll have issues with older stable kernels that have never been compiled with > 4.7, but I haven't actually tried. Kevin ^ permalink raw reply [flat|nested] 5+ messages in thread
* mainline build: 137 builds: 0 failed, 137 passed, 296 warnings (v4.4-10454-g3e1e21c7bfcf) 2016-01-25 18:18 ` Kevin Hilman @ 2016-01-25 20:52 ` Arnd Bergmann 2016-01-26 19:22 ` Kevin Hilman 0 siblings, 1 reply; 5+ messages in thread From: Arnd Bergmann @ 2016-01-25 20:52 UTC (permalink / raw) To: linux-arm-kernel On Monday 25 January 2016 10:18:56 Kevin Hilman wrote: > Tyler Baker <tyler.baker@linaro.org> writes: > > > On 22 January 2016 at 05:19, Arnd Bergmann <arnd@arndb.de> wrote: > >> On Thursday 21 January 2016 22:14:26 kernelci. org bot wrote: > >>> Warnings summary: > >>> > >>> 224 <stdin>:1307:2: warning: #warning syscall copy_file_range not implemented [-Wcpp] > >> > >> I don't think anyone has sent this so far, sending a patch now. > >> > >>> 29 drivers/tty/serial/amba-pl011.c:190:27: warning: 'vendor_zte' defined but not used [-Wunused-variable] > >> > >> Patch sent a couple of days ago, got an Ack and waiting to be applied. > >> > >>> 12 net/bluetooth/mgmt.c:5493:8: warning: 'r192' may be used uninitialized in this function [-Wmaybe-uninitialized] > >>> 12 net/bluetooth/mgmt.c:5493:8: warning: 'h192' may be used uninitialized in this function [-Wmaybe-uninitialized] > >> > >> This happens only in gcc-4.7 and gcc-4.8 > >> > >>> 5 crypto/wp512.c:987:1: warning: the frame size of 1168 bytes is larger than 1024 bytes [-Wframe-larger-than=] > >>> 1 crypto/wp512.c:987:1: warning: the frame size of 1112 bytes is larger than 1024 bytes [-Wframe-larger-than=] > >> > >> and this one only in gcc-4.7 > >> > >> Any chance we can update the toolchain on kernelci.org to 4.9 or higher? > >> I'm normally testing with gcc-5.2, which has a little better warnings. > > > > I'm ok with upgrading to 4.9 or higher. Kevin any issues with this? > > No objection in principle, but we probably can't do this across the > board without fallout. > > I suspect we'll have issues with older stable kernels that have never > been compiled with > 4.7, but I haven't actually tried. > My guess is that there is very little of that. We had one issues with gcc-4.8 through 4.8.2 generating bad code, but aside of that I don't remember a recent case where a newer compiler broke stuff. There might be a couple of extra warnings in older kernels, but I've tried to ensure that all new warnings that got introduced from more recent gcc versions are fixed mainline kernels. Arnd ^ permalink raw reply [flat|nested] 5+ messages in thread
* mainline build: 137 builds: 0 failed, 137 passed, 296 warnings (v4.4-10454-g3e1e21c7bfcf) 2016-01-25 20:52 ` Arnd Bergmann @ 2016-01-26 19:22 ` Kevin Hilman 0 siblings, 0 replies; 5+ messages in thread From: Kevin Hilman @ 2016-01-26 19:22 UTC (permalink / raw) To: linux-arm-kernel Arnd Bergmann <arnd@arndb.de> writes: > On Monday 25 January 2016 10:18:56 Kevin Hilman wrote: >> Tyler Baker <tyler.baker@linaro.org> writes: >> >> > On 22 January 2016 at 05:19, Arnd Bergmann <arnd@arndb.de> wrote: >> >> On Thursday 21 January 2016 22:14:26 kernelci. org bot wrote: >> >>> Warnings summary: >> >>> >> >>> 224 <stdin>:1307:2: warning: #warning syscall copy_file_range not implemented [-Wcpp] >> >> >> >> I don't think anyone has sent this so far, sending a patch now. >> >> >> >>> 29 drivers/tty/serial/amba-pl011.c:190:27: warning: 'vendor_zte' defined but not used [-Wunused-variable] >> >> >> >> Patch sent a couple of days ago, got an Ack and waiting to be applied. >> >> >> >>> 12 net/bluetooth/mgmt.c:5493:8: warning: 'r192' may be used uninitialized in this function [-Wmaybe-uninitialized] >> >>> 12 net/bluetooth/mgmt.c:5493:8: warning: 'h192' may be used uninitialized in this function [-Wmaybe-uninitialized] >> >> >> >> This happens only in gcc-4.7 and gcc-4.8 >> >> >> >>> 5 crypto/wp512.c:987:1: warning: the frame size of 1168 bytes is larger than 1024 bytes [-Wframe-larger-than=] >> >>> 1 crypto/wp512.c:987:1: warning: the frame size of 1112 bytes is larger than 1024 bytes [-Wframe-larger-than=] >> >> >> >> and this one only in gcc-4.7 >> >> >> >> Any chance we can update the toolchain on kernelci.org to 4.9 or higher? >> >> I'm normally testing with gcc-5.2, which has a little better warnings. >> > >> > I'm ok with upgrading to 4.9 or higher. Kevin any issues with this? >> >> No objection in principle, but we probably can't do this across the >> board without fallout. >> >> I suspect we'll have issues with older stable kernels that have never >> been compiled with > 4.7, but I haven't actually tried. >> > > My guess is that there is very little of that. We had one issues with > gcc-4.8 through 4.8.2 generating bad code, but aside of that I don't > remember a recent case where a newer compiler broke stuff. > > There might be a couple of extra warnings in older kernels, but I've > tried to ensure that all new warnings that got introduced from more > recent gcc versions are fixed mainline kernels. OK, and I guess if we find any breakage, fixes should be submitted to stable anyways, so I'm OK with the change. Kevin ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-01-26 19:22 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <56a1c8c2.6953c20a.25484.ffff85a9@mx.google.com>
2016-01-22 13:19 ` mainline build: 137 builds: 0 failed, 137 passed, 296 warnings (v4.4-10454-g3e1e21c7bfcf) Arnd Bergmann
2016-01-22 17:23 ` Tyler Baker
2016-01-25 18:18 ` Kevin Hilman
2016-01-25 20:52 ` Arnd Bergmann
2016-01-26 19:22 ` Kevin Hilman
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).