From: kernel test robot <lkp@intel.com>
To: kbuild@lists.01.org
Subject: drivers/vhost/net.c:1010 vhost_net_rx_peek_head_len() error: double unlocked 'sk->sk_receive_queue.lock' (orig line 1002)
Date: Sat, 11 Jul 2020 08:36:21 +0800 [thread overview]
Message-ID: <202007110813.eQcrZ2ui%lkp@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 3869 bytes --]
CC: kbuild-all(a)lists.01.org
CC: linux-kernel(a)vger.kernel.org
TO: Jason Wang <jasowang@redhat.com>
CC: "Michael S. Tsirkin" <mst@redhat.com>
Hi Jason,
First bad commit (maybe != root cause):
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: aa0c9086b40c17a7ad94425b3b70dd1fdd7497bf
commit: 20c384f1ea1a0bc7320bc445c72dd02d2970d594 vhost: refine vhost and vringh kconfig
date: 3 months ago
:::::: branch date: 4 hours ago
:::::: commit date: 3 months ago
config: microblaze-randconfig-m031-20200711 (attached as .config)
compiler: microblaze-linux-gcc (GCC) 9.3.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
New smatch warnings:
drivers/vhost/net.c:1010 vhost_net_rx_peek_head_len() error: double unlocked 'sk->sk_receive_queue.lock' (orig line 1002)
drivers/vhost/net.c:1010 vhost_net_rx_peek_head_len() error: double unlocked 'sk->sk_receive_queue.lock' (orig line 1002)
drivers/vhost/net.c:1010 vhost_net_rx_peek_head_len() error: double unlocked 'sk->sk_receive_queue.lock' (orig line 1002)
Old smatch warnings:
arch/microblaze/include/asm/thread_info.h:94 current_thread_info() error: uninitialized symbol 'sp'.
drivers/vhost/net.c:1528 vhost_net_set_backend() warn: passing a valid pointer to 'PTR_ERR'
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=20c384f1ea1a0bc7320bc445c72dd02d2970d594
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git remote update linus
git checkout 20c384f1ea1a0bc7320bc445c72dd02d2970d594
vim +1010 drivers/vhost/net.c
8dd014adfea6f1 David Stevens 2010-07-27 994
be294a51adfc1e Toshiaki Makita 2018-07-03 995 static int vhost_net_rx_peek_head_len(struct vhost_net *net, struct sock *sk,
be294a51adfc1e Toshiaki Makita 2018-07-03 996 bool *busyloop_intr)
03088137246065 Jason Wang 2016-03-04 997 {
28b9b33b983f4d Toshiaki Makita 2018-07-03 998 struct vhost_net_virtqueue *rnvq = &net->vqs[VHOST_NET_VQ_RX];
28b9b33b983f4d Toshiaki Makita 2018-07-03 999 struct vhost_net_virtqueue *tnvq = &net->vqs[VHOST_NET_VQ_TX];
6369fec5be0aad Toshiaki Makita 2018-07-03 1000 struct vhost_virtqueue *rvq = &rnvq->vq;
28b9b33b983f4d Toshiaki Makita 2018-07-03 1001 struct vhost_virtqueue *tvq = &tnvq->vq;
28b9b33b983f4d Toshiaki Makita 2018-07-03 @1002 int len = peek_head_len(rnvq, sk);
03088137246065 Jason Wang 2016-03-04 1003
dc151282bbdaed Tonghao Zhang 2018-09-25 1004 if (!len && rvq->busyloop_timeout) {
f5a4941aa6d190 Jason Wang 2018-05-29 1005 /* Flush batched heads first */
09c3248938c3e3 Jason Wang 2018-07-20 1006 vhost_net_signal_used(rnvq);
03088137246065 Jason Wang 2016-03-04 1007 /* Both tx vq and rx socket were polled here */
dc151282bbdaed Tonghao Zhang 2018-09-25 1008 vhost_net_busy_poll(net, rvq, tvq, busyloop_intr, true);
03088137246065 Jason Wang 2016-03-04 1009
28b9b33b983f4d Toshiaki Makita 2018-07-03 @1010 len = peek_head_len(rnvq, sk);
03088137246065 Jason Wang 2016-03-04 1011 }
03088137246065 Jason Wang 2016-03-04 1012
03088137246065 Jason Wang 2016-03-04 1013 return len;
03088137246065 Jason Wang 2016-03-04 1014 }
03088137246065 Jason Wang 2016-03-04 1015
:::::: The code at line 1010 was first introduced by commit
:::::: 28b9b33b983f4de3ce9e660e3efe1e08adabf779 vhost_net: Rename local variables in vhost_net_rx_peek_head_len
:::::: TO: Toshiaki Makita <makita.toshiaki@lab.ntt.co.jp>
:::::: CC: David S. Miller <davem@davemloft.net>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 27240 bytes --]
next reply other threads:[~2020-07-11 0:36 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-11 0:36 kernel test robot [this message]
-- strict thread matches above, loose matches on Subject: below --
2020-08-08 7:37 drivers/vhost/net.c:1010 vhost_net_rx_peek_head_len() error: double unlocked 'sk->sk_receive_queue.lock' (orig line 1002) kernel test robot
2020-08-08 7:37 ` kernel test robot
2020-08-10 12:33 ` Michael S. Tsirkin
2020-08-10 12:33 ` Michael S. Tsirkin
2020-08-11 3:03 ` Jason Wang
2020-08-11 3:03 ` Jason Wang
2020-08-18 7:01 ` Xia, Hui
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=202007110813.eQcrZ2ui%lkp@intel.com \
--to=lkp@intel.com \
--cc=kbuild@lists.01.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 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.