On Thu, Jun 8, 2017 at 9:48 PM, Build bot for Mark Brown
[off-list ref] wrote:
Tree/Branch: v3.18.55-rt61
Git describe: v3.18.55-rt61
Commit: a2896fc906 Linux 3.18.55-rt61
Build Time: 0 min 7 sec
Passed: 6 / 6 (100.00 %)
Failed: 0 / 6 ( 0.00 %)
Errors: 0
Warnings: 5
Section Mismatches: 0
-------------------------------------------------------------------------------
defconfigs with issues (other than build errors):
9 warnings 0 mismatches : arm-allmodconfig
3 warnings 0 mismatches : arm-multi_v7_defconfig
2 warnings 0 mismatches : arm-multi_v5_defconfig
3 warnings 0 mismatches : x86_64-defconfig
-------------------------------------------------------------------------------
stable/3.18.y builds cleanly these days, so most warnings are all
regressions in -rt, possibly not the KM_TYPE_NR warning, which
seems to be an actual bug in stable kernels.
-------------------------------------------------------------------------------
arm-allmodconfig : PASS, 0 errors, 9 warnings, 0 section mismatches
Warnings:
../block/blk-core.c:103:5: warning: "CONFIG_PREEMPT_RT_FULL" is not defined [-Wundef]
../block/blk-core.c:103:5: warning: "CONFIG_PREEMPT_RT_FULL" is not defined [-Wundef]
harmless typo in commit 091971373443 ("block/mq: don't complete requests
via IPI"), #if needs to be #ifdef
../net/core/dev.c:2899:1: warning: no return statement in function returning non-void [-Wreturn-type]
Caused by fd7ce3b6ea7e ("net: move xmit_recursion to per-task variable on -RT"):
xmit_rec_dec() should return 'void', not 'int'. Harmless.
../arch/arm/include/asm/kmap_types.h:7:0: warning: "KM_TYPE_NR" redefined
../arch/arm/include/asm/kmap_types.h:7:0: warning: "KM_TYPE_NR" redefined
We should backport
adec640e0366 ("mlx5: stop including <asm-generic/kmap_types.h>")
In stable/linux-3.18.y, this is also wrong, but no warning shows up, since
we don't include the right header there, only the wrong one. It's probably
harmless, but I'm not completely sure about htis one.
Greg, maybe you can just add this to 3.18.y?
-------------------------------------------------------------------------------
arm-multi_v7_defconfig : PASS, 0 errors, 3 warnings, 0 section mismatches
Warnings:
../include/linux/spinlock.h:253:3: warning: 'flags' may be used uninitialized in this function [-Wmaybe-uninitialized]
Caused by e09f810843a2 ("drivers-tty-pl011-irq-disable-madness.patch"),
which removes the workaround for the warning.
-------------------------------------------------------------------------------
x86_64-defconfig : PASS, 0 errors, 3 warnings, 0 section mismatches
Warnings:
../drivers/thermal/x86_pkg_temp_thermal.c:414:1: warning: no return statement in function returning non-void [-Wreturn-type]
Caused by 87df656d74fe ("thermal: Defer thermal wakups to threads"),
thermal_notify_work_cleanup() needs to return 'void', not 'int'.
I'm guessing that the same problems exist in the other stable-rt kernels,
so we probably need to fix them on all branches.
Arnd
On Mon, Jun 12, 2017 at 04:01:21PM +0200, Arnd Bergmann wrote:
On Thu, Jun 8, 2017 at 9:48 PM, Build bot for Mark Brown
quoted
../arch/arm/include/asm/kmap_types.h:7:0: warning: "KM_TYPE_NR" redefined
../arch/arm/include/asm/kmap_types.h:7:0: warning: "KM_TYPE_NR" redefined
We should backport
adec640e0366 ("mlx5: stop including <asm-generic/kmap_types.h>")
In stable/linux-3.18.y, this is also wrong, but no warning shows up, since
we don't include the right header there, only the wrong one. It's probably
harmless, but I'm not completely sure about htis one.
Greg, maybe you can just add this to 3.18.y?
Ok, now queued up, thanks.
greg k-h
Arnd,
Thanks for Ccing us. I'll have a look. Some of these small issues
probably should be fixed in upstream RT as well. I'll see if the #if
not #ifdef is in the RT development branch, as I usually just do
straight pulls from that.
-- Steve
On Mon, 12 Jun 2017 16:01:21 +0200
Arnd Bergmann [off-list ref] wrote:
On Thu, Jun 8, 2017 at 9:48 PM, Build bot for Mark Brown
[off-list ref] wrote:
quoted
Tree/Branch: v3.18.55-rt61
Git describe: v3.18.55-rt61
Commit: a2896fc906 Linux 3.18.55-rt61
Build Time: 0 min 7 sec
Passed: 6 / 6 (100.00 %)
Failed: 0 / 6 ( 0.00 %)
Errors: 0
Warnings: 5
Section Mismatches: 0
-------------------------------------------------------------------------------
defconfigs with issues (other than build errors):
9 warnings 0 mismatches : arm-allmodconfig
3 warnings 0 mismatches : arm-multi_v7_defconfig
2 warnings 0 mismatches : arm-multi_v5_defconfig
3 warnings 0 mismatches : x86_64-defconfig
-------------------------------------------------------------------------------
stable/3.18.y builds cleanly these days, so most warnings are all
regressions in -rt, possibly not the KM_TYPE_NR warning, which
seems to be an actual bug in stable kernels.
quoted
-------------------------------------------------------------------------------
arm-allmodconfig : PASS, 0 errors, 9 warnings, 0 section mismatches
Warnings:
../block/blk-core.c:103:5: warning: "CONFIG_PREEMPT_RT_FULL" is not defined [-Wundef]
../block/blk-core.c:103:5: warning: "CONFIG_PREEMPT_RT_FULL" is not defined [-Wundef]
harmless typo in commit 091971373443 ("block/mq: don't complete requests
via IPI"), #if needs to be #ifdef
quoted
../net/core/dev.c:2899:1: warning: no return statement in function returning non-void [-Wreturn-type]
Caused by fd7ce3b6ea7e ("net: move xmit_recursion to per-task variable on -RT"):
xmit_rec_dec() should return 'void', not 'int'. Harmless.
quoted
../arch/arm/include/asm/kmap_types.h:7:0: warning: "KM_TYPE_NR" redefined
../arch/arm/include/asm/kmap_types.h:7:0: warning: "KM_TYPE_NR" redefined
We should backport
adec640e0366 ("mlx5: stop including <asm-generic/kmap_types.h>")
In stable/linux-3.18.y, this is also wrong, but no warning shows up, since
we don't include the right header there, only the wrong one. It's probably
harmless, but I'm not completely sure about htis one.
Greg, maybe you can just add this to 3.18.y?
quoted
-------------------------------------------------------------------------------
arm-multi_v7_defconfig : PASS, 0 errors, 3 warnings, 0 section mismatches
Warnings:
../include/linux/spinlock.h:253:3: warning: 'flags' may be used uninitialized in this function [-Wmaybe-uninitialized]
Caused by e09f810843a2 ("drivers-tty-pl011-irq-disable-madness.patch"),
which removes the workaround for the warning.
quoted
-------------------------------------------------------------------------------
x86_64-defconfig : PASS, 0 errors, 3 warnings, 0 section mismatches
Warnings:
../drivers/thermal/x86_pkg_temp_thermal.c:414:1: warning: no return statement in function returning non-void [-Wreturn-type]
Caused by 87df656d74fe ("thermal: Defer thermal wakups to threads"),
thermal_notify_work_cleanup() needs to return 'void', not 'int'.
I'm guessing that the same problems exist in the other stable-rt kernels,
so we probably need to fix them on all branches.
Arnd