From: kernel test robot <lkp@intel.com>
To: ycaibb <ycaibb@gmail.com>,
davem@davemloft.net, yoshfuji@linux-ipv6.org, dsahern@kernel.org,
kuba@kernel.org, ast@kernel.org, daniel@iogearbox.net,
andrii@kernel.org, kafai@fb.com, songliubraving@fb.com,
yhs@fb.com
Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org
Subject: Re: [PATCH] inet: missing lock releases in udp.c
Date: Fri, 21 Jan 2022 14:26:32 +0800 [thread overview]
Message-ID: <202201211410.CFe494On-lkp@intel.com> (raw)
In-Reply-To: <20220121031553.5342-1-ycaibb@gmail.com>
Hi ycaibb,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on net-next/master]
[also build test WARNING on net/master horms-ipvs/master linus/master v5.16 next-20220121]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/ycaibb/inet-missing-lock-releases-in-udp-c/20220121-111922
base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git 8aaaf2f3af2ae212428f4db1af34214225f5cec3
config: mips-cu1000-neo_defconfig (https://download.01.org/0day-ci/archive/20220121/202201211410.CFe494On-lkp@intel.com/config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project d4baf3b1322b84816aa623d8e8cb45a49cb68b84)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install mips cross compiling tool for clang build
# apt-get install binutils-mips-linux-gnu
# https://github.com/0day-ci/linux/commit/1b84613d303e1496ea759aa171b7b36dfd8f26c3
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review ycaibb/inet-missing-lock-releases-in-udp-c/20220121-111922
git checkout 1b84613d303e1496ea759aa171b7b36dfd8f26c3
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=mips SHELL=/bin/bash net/ipv4/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> net/ipv4/udp.c:2979:5: warning: misleading indentation; statement is not part of the previous 'if' [-Wmisleading-indentation]
goto found;
^
net/ipv4/udp.c:2976:4: note: previous statement is here
if (afinfo->family == AF_UNSPEC ||
^
1 warning generated.
vim +/if +2979 net/ipv4/udp.c
^1da177e4c3f415 Linus Torvalds 2005-04-16 2952
645ca708f936b2f Eric Dumazet 2008-10-29 2953 static struct sock *udp_get_first(struct seq_file *seq, int start)
^1da177e4c3f415 Linus Torvalds 2005-04-16 2954 {
^1da177e4c3f415 Linus Torvalds 2005-04-16 2955 struct sock *sk;
9e8ca27afab6c92 Yonghong Song 2020-06-23 2956 struct udp_seq_afinfo *afinfo;
^1da177e4c3f415 Linus Torvalds 2005-04-16 2957 struct udp_iter_state *state = seq->private;
6f191efe48af62d Denis V. Lunev 2008-03-28 2958 struct net *net = seq_file_net(seq);
^1da177e4c3f415 Linus Torvalds 2005-04-16 2959
9e8ca27afab6c92 Yonghong Song 2020-06-23 2960 if (state->bpf_seq_afinfo)
9e8ca27afab6c92 Yonghong Song 2020-06-23 2961 afinfo = state->bpf_seq_afinfo;
9e8ca27afab6c92 Yonghong Song 2020-06-23 2962 else
9e8ca27afab6c92 Yonghong Song 2020-06-23 2963 afinfo = PDE_DATA(file_inode(seq->file));
9e8ca27afab6c92 Yonghong Song 2020-06-23 2964
a3d2599b24462c7 Christoph Hellwig 2018-04-10 2965 for (state->bucket = start; state->bucket <= afinfo->udp_table->mask;
f86dcc5aa8c7908 Eric Dumazet 2009-10-07 2966 ++state->bucket) {
a3d2599b24462c7 Christoph Hellwig 2018-04-10 2967 struct udp_hslot *hslot = &afinfo->udp_table->hash[state->bucket];
f86dcc5aa8c7908 Eric Dumazet 2009-10-07 2968
ca065d0cf80fa54 Eric Dumazet 2016-04-01 2969 if (hlist_empty(&hslot->head))
f86dcc5aa8c7908 Eric Dumazet 2009-10-07 2970 continue;
f86dcc5aa8c7908 Eric Dumazet 2009-10-07 2971
645ca708f936b2f Eric Dumazet 2008-10-29 2972 spin_lock_bh(&hslot->lock);
ca065d0cf80fa54 Eric Dumazet 2016-04-01 2973 sk_for_each(sk, &hslot->head) {
878628fbf2589eb YOSHIFUJI Hideaki 2008-03-26 2974 if (!net_eq(sock_net(sk), net))
a91275eff43a527 Daniel Lezcano 2008-03-21 2975 continue;
9e8ca27afab6c92 Yonghong Song 2020-06-23 2976 if (afinfo->family == AF_UNSPEC ||
9e8ca27afab6c92 Yonghong Song 2020-06-23 2977 sk->sk_family == afinfo->family)
1b84613d303e149 Ryan Cai 2022-01-21 2978 spin_unlock_bh(&hslot->lock);
^1da177e4c3f415 Linus Torvalds 2005-04-16 @2979 goto found;
^1da177e4c3f415 Linus Torvalds 2005-04-16 2980 }
645ca708f936b2f Eric Dumazet 2008-10-29 2981 spin_unlock_bh(&hslot->lock);
^1da177e4c3f415 Linus Torvalds 2005-04-16 2982 }
^1da177e4c3f415 Linus Torvalds 2005-04-16 2983 sk = NULL;
^1da177e4c3f415 Linus Torvalds 2005-04-16 2984 found:
^1da177e4c3f415 Linus Torvalds 2005-04-16 2985 return sk;
^1da177e4c3f415 Linus Torvalds 2005-04-16 2986 }
^1da177e4c3f415 Linus Torvalds 2005-04-16 2987
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
parent reply other threads:[~2022-01-21 6:27 UTC|newest]
Thread overview: expand[flat|nested] mbox.gz Atom feed
[parent not found: <20220121031553.5342-1-ycaibb@gmail.com>]
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=202201211410.CFe494On-lkp@intel.com \
--to=lkp@intel.com \
--cc=andrii@kernel.org \
--cc=ast@kernel.org \
--cc=daniel@iogearbox.net \
--cc=davem@davemloft.net \
--cc=dsahern@kernel.org \
--cc=kafai@fb.com \
--cc=kbuild-all@lists.01.org \
--cc=kuba@kernel.org \
--cc=llvm@lists.linux.dev \
--cc=songliubraving@fb.com \
--cc=ycaibb@gmail.com \
--cc=yhs@fb.com \
--cc=yoshfuji@linux-ipv6.org \
/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 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).