Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: PWmGuT7SdGIU/yrpQIeRPic3962FNFL0Bs0nfERKTWBCj0JqDEBq4uLOMiwaoWIiq4x681Pp5w
 dMYRYisYvcxQ==
IronPort-SDR: m3mkSkCpbjzAY8hxYgOPiYcHhw2hEvofD643W9CtUicrjLvBK7c+V8G1xETpEXp8aQwp9cVrbV
 JDRcmuZNwpkg==
From:   "Xia, Hui" <hui.xia@intel.com>
To:     "Michael S. Tsirkin" <mst@redhat.com>, lkp <lkp@intel.com>
CC:     Jason Wang <jasowang@redhat.com>,
        "kbuild-all@lists.01.org" <kbuild-all@lists.01.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [kbuild-all] Re: drivers/vhost/net.c:1010
 vhost_net_rx_peek_head_len() error: double unlocked
 'sk->sk_receive_queue.lock' (orig line 1002)
Thread-Topic: [kbuild-all] Re: drivers/vhost/net.c:1010
 vhost_net_rx_peek_head_len() error: double unlocked
 'sk->sk_receive_queue.lock' (orig line 1002)
Thread-Index: AQHWbxKepQkVASIWYUe4twSsrm/XSak9fIFw
Date:   Tue, 18 Aug 2020 07:01:00 +0000
Message-ID: <MWHPR11MB161364B47AF529EB61BDA348E55C0@MWHPR11MB1613.namprd11.prod.outlook.com>
References: <202008081502.F9nEsAf6%lkp@intel.com>
 <20200810083313-mutt-send-email-mst@kernel.org>
In-Reply-To: <20200810083313-mutt-send-email-mst@kernel.org>
Accept-Language: en-US
Content-Language: en-US
dlp-product: dlpe-windows
dlp-reaction: no-action
dlp-version: 11.5.1.3
Content-Type: text/plain; charset="iso-2022-jp"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 9c5584a1-928b-487a-31e2-08d843447716
X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Aug 2020 07:01:00.7265
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: AgzKB9QTp83mPdAlefceI8CG04CEkTHiT53b8/bDNI7cKmyNl646nytvBrCNtzNMQar3e5JZZh9T6VPSG+qTLw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR11MB0078
X-OriginatorOrg: intel.com
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


>-----Original Message-----
>From: Michael S. Tsirkin <mst@redhat.com>
>Sent: 2020=1B$BG/=1B(B8=1B$B7n=1B(B10=1B$BF|=1B(B 20:34
>To: kernel test robot <lkp@intel.com>
>Cc: Jason Wang <jasowang@redhat.com>; kbuild-all@lists.01.org; linux-
>kernel@vger.kernel.org
>Subject: [kbuild-all] Re: drivers/vhost/net.c:1010 vhost_net_rx_peek_head_=
len()
>error: double unlocked 'sk->sk_receive_queue.lock' (orig line 1002)
>
>On Sat, Aug 08, 2020 at 03:37:07PM +0800, kernel test robot wrote:
>> Hi Jason,
>>
>> First bad commit (maybe !=3D root cause):
>>
>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.g=
it master
>> head:   449dc8c97089a6e09fb2dac4d92b1b7ac0eb7c1e
>> commit: 20c384f1ea1a0bc7320bc445c72dd02d2970d594 vhost: refine vhost
>and vringh kconfig
>> date:   4 months ago
>> config: arm-randconfig-m031-20200808 (attached as .config)
>> compiler: arm-linux-gnueabi-gcc (GCC) 9.3.0
>>
>> If you fix the issue, kindly add following tag as appropriate
>> Reported-by: kernel test robot <lkp@intel.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)
>
>Weird. Can't figure out where the double unlock is ...
Please ignore this report. It is a false positive.=20
We have changed the rule to avoid send out this warning type directly. Sorr=
y for inconvenient.

>
>
>> Old smatch warnings:
>> drivers/vhost/net.c:1528 vhost_net_set_backend() warn: passing a valid p=
ointer
>to 'PTR_ERR'
>
>this stumped me too...
Ignore this one too. It is old warnings.

Thanks,
Hui
>
>> vim +1010 drivers/vhost/net.c
>>
>> 8dd014adfea6f17 David Stevens   2010-07-27   994
>> be294a51adfc1e1 Toshiaki Makita 2018-07-03   995  static int
>vhost_net_rx_peek_head_len(struct vhost_net *net, struct sock *sk,
>> be294a51adfc1e1 Toshiaki Makita 2018-07-03   996
>	      bool *busyloop_intr)
>> 030881372460654 Jason Wang      2016-03-04   997  {
>> 28b9b33b983f4de Toshiaki Makita 2018-07-03   998  	struct
>vhost_net_virtqueue *rnvq =3D &net->vqs[VHOST_NET_VQ_RX];
>> 28b9b33b983f4de Toshiaki Makita 2018-07-03   999  	struct
>vhost_net_virtqueue *tnvq =3D &net->vqs[VHOST_NET_VQ_TX];
>> 6369fec5be0aad4 Toshiaki Makita 2018-07-03  1000  	struct vhost_virtqueu=
e
>*rvq =3D &rnvq->vq;
>> 28b9b33b983f4de Toshiaki Makita 2018-07-03  1001  	struct vhost_virtqueu=
e
>*tvq =3D &tnvq->vq;
>> 28b9b33b983f4de Toshiaki Makita 2018-07-03 @1002  	int len =3D
>peek_head_len(rnvq, sk);
>> 030881372460654 Jason Wang      2016-03-04  1003
>> dc151282bbdaed0 Tonghao Zhang   2018-09-25  1004  	if (!len && rvq-
>>busyloop_timeout) {
>> f5a4941aa6d190e Jason Wang      2018-05-29  1005  		/* Flush
>batched heads first */
>> 09c3248938c3e3b Jason Wang      2018-07-20  1006
>	vhost_net_signal_used(rnvq);
>> 030881372460654 Jason Wang      2016-03-04  1007  		/* Both tx vq
>and rx socket were polled here */
>> dc151282bbdaed0 Tonghao Zhang   2018-09-25  1008
>	vhost_net_busy_poll(net, rvq, tvq, busyloop_intr, true);
>> 030881372460654 Jason Wang      2016-03-04  1009
>> 28b9b33b983f4de Toshiaki Makita 2018-07-03 @1010  		len =3D
>peek_head_len(rnvq, sk);
>> 030881372460654 Jason Wang      2016-03-04  1011  	}
>> 030881372460654 Jason Wang      2016-03-04  1012
>> 030881372460654 Jason Wang      2016-03-04  1013  	return len;
>> 030881372460654 Jason Wang      2016-03-04  1014  }
>> 030881372460654 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@lists.01.org
>
>_______________________________________________
>kbuild-all mailing list -- kbuild-all@lists.01.org To unsubscribe send an =
email to
>kbuild-all-leave@lists.01.org