From: kbuild test robot <lkp@intel.com>
To: David Lamparter <equinox@diac24.net>
Cc: kbuild-all@01.org, netdev@vger.kernel.org,
amine.kherbouche@6wind.com, roopa@cumulusnetworks.com,
David Lamparter <equinox@diac24.net>
Subject: Re: [PATCH 2/6] mpls: split forwarding path on rx/tx boundary
Date: Sun, 20 Aug 2017 01:10:08 +0800 [thread overview]
Message-ID: <201708200035.x3RDmbbq%fengguang.wu@intel.com> (raw)
In-Reply-To: <20170816170202.456851-3-equinox@diac24.net>
[-- Attachment #1: Type: text/plain, Size: 3916 bytes --]
Hi David,
[auto build test WARNING on net-next/master]
[also build test WARNING on next-20170817]
[cannot apply to v4.13-rc5]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/David-Lamparter/bridge-learn-dst-metadata-in-FDB/20170820-001849
config: s390-allmodconfig (attached as .config)
compiler: s390x-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
reproduce:
wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=s390
Note: it may well be a FALSE warning. FWIW you are at least aware of it now.
http://gcc.gnu.org/wiki/Better_Uninitialized_Warnings
All warnings (new ones prefixed by >>):
In file included from include/uapi/linux/stddef.h:1:0,
from include/linux/stddef.h:4,
from include/uapi/linux/posix_types.h:4,
from include/uapi/linux/types.h:13,
from include/linux/types.h:5,
from net/mpls/af_mpls.c:1:
net/mpls/af_mpls.c: In function 'mpls_rt_xmit':
>> include/linux/compiler.h:239:26: warning: 'out_dev' may be used uninitialized in this function [-Wmaybe-uninitialized]
case 8: *(__u64 *)res = *(volatile __u64 *)p; break; \
^
net/mpls/af_mpls.c:384:21: note: 'out_dev' was declared here
struct net_device *out_dev;
^~~~~~~
--
In file included from include/uapi/linux/stddef.h:1:0,
from include/linux/stddef.h:4,
from include/uapi/linux/posix_types.h:4,
from include/uapi/linux/types.h:13,
from include/linux/types.h:5,
from net//mpls/af_mpls.c:1:
net//mpls/af_mpls.c: In function 'mpls_rt_xmit':
>> include/linux/compiler.h:239:26: warning: 'out_dev' may be used uninitialized in this function [-Wmaybe-uninitialized]
case 8: *(__u64 *)res = *(volatile __u64 *)p; break; \
^
net//mpls/af_mpls.c:384:21: note: 'out_dev' was declared here
struct net_device *out_dev;
^~~~~~~
vim +/out_dev +239 include/linux/compiler.h
230fa253 Christian Borntraeger 2014-11-25 232
d976441f Andrey Ryabinin 2015-10-19 233 #define __READ_ONCE_SIZE \
d976441f Andrey Ryabinin 2015-10-19 234 ({ \
d976441f Andrey Ryabinin 2015-10-19 235 switch (size) { \
d976441f Andrey Ryabinin 2015-10-19 236 case 1: *(__u8 *)res = *(volatile __u8 *)p; break; \
d976441f Andrey Ryabinin 2015-10-19 237 case 2: *(__u16 *)res = *(volatile __u16 *)p; break; \
d976441f Andrey Ryabinin 2015-10-19 238 case 4: *(__u32 *)res = *(volatile __u32 *)p; break; \
d976441f Andrey Ryabinin 2015-10-19 @239 case 8: *(__u64 *)res = *(volatile __u64 *)p; break; \
d976441f Andrey Ryabinin 2015-10-19 240 default: \
d976441f Andrey Ryabinin 2015-10-19 241 barrier(); \
d976441f Andrey Ryabinin 2015-10-19 242 __builtin_memcpy((void *)res, (const void *)p, size); \
d976441f Andrey Ryabinin 2015-10-19 243 barrier(); \
d976441f Andrey Ryabinin 2015-10-19 244 } \
d976441f Andrey Ryabinin 2015-10-19 245 })
d976441f Andrey Ryabinin 2015-10-19 246
:::::: The code at line 239 was first introduced by commit
:::::: d976441f44bc5d48635d081d277aa76556ffbf8b compiler, atomics, kasan: Provide READ_ONCE_NOCHECK()
:::::: TO: Andrey Ryabinin <aryabinin@virtuozzo.com>
:::::: CC: Ingo Molnar <mingo@kernel.org>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 47252 bytes --]
next prev parent reply other threads:[~2017-08-19 17:10 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-16 17:01 [RFC net-next] VPLS support David Lamparter
2017-08-16 17:01 ` [PATCH 1/6] bridge: learn dst metadata in FDB David Lamparter
2017-08-16 20:38 ` Nikolay Aleksandrov
2017-08-16 20:38 ` [Bridge] " Nikolay Aleksandrov
2017-08-17 11:03 ` David Lamparter
2017-08-17 11:03 ` [Bridge] " David Lamparter
2017-08-17 11:39 ` Nikolay Aleksandrov
2017-08-17 11:39 ` [Bridge] " Nikolay Aleksandrov
2017-08-17 11:51 ` Nikolay Aleksandrov
2017-08-17 11:51 ` [Bridge] " Nikolay Aleksandrov
2017-08-17 12:10 ` David Lamparter
2017-08-17 12:10 ` [Bridge] " David Lamparter
2017-08-17 12:19 ` Nikolay Aleksandrov
2017-08-17 12:19 ` [Bridge] " Nikolay Aleksandrov
2017-08-17 12:20 ` David Lamparter
2017-08-17 12:20 ` [Bridge] " David Lamparter
2017-08-17 12:45 ` David Lamparter
2017-08-17 12:45 ` [Bridge] " David Lamparter
2017-08-17 13:04 ` Nikolay Aleksandrov
2017-08-17 13:04 ` [Bridge] " Nikolay Aleksandrov
2017-08-17 16:16 ` David Lamparter
2017-08-17 16:16 ` [Bridge] " David Lamparter
2017-08-16 17:01 ` [PATCH 2/6] mpls: split forwarding path on rx/tx boundary David Lamparter
2017-08-19 17:10 ` kbuild test robot [this message]
2017-08-19 17:42 ` kbuild test robot
2017-08-16 17:01 ` [PATCH 3/6] mpls: add VPLS entry points David Lamparter
2017-08-19 18:27 ` kbuild test robot
2017-08-21 14:01 ` Amine Kherbouche
2017-08-21 15:55 ` David Lamparter
2017-08-21 16:13 ` Amine Kherbouche
2017-08-16 17:02 ` [PATCH 4/6] mpls: VPLS support David Lamparter
2017-08-21 15:14 ` Amine Kherbouche
2017-08-21 16:18 ` David Lamparter
2017-08-21 16:11 ` Amine Kherbouche
2017-08-16 17:02 ` [PATCH 5/6] bridge: add VPLS pseudowire info in fdb dump David Lamparter
2017-08-16 17:02 ` [PATCH 6/6] mpls: pseudowire control word support David Lamparter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201708200035.x3RDmbbq%fengguang.wu@intel.com \
--to=lkp@intel.com \
--cc=amine.kherbouche@6wind.com \
--cc=equinox@diac24.net \
--cc=kbuild-all@01.org \
--cc=netdev@vger.kernel.org \
--cc=roopa@cumulusnetworks.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.