From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 760B83D47B7 for ; Thu, 11 Jun 2026 10:38:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.129.124 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781174295; cv=none; b=s5EvYEKcNa7G2ZAfhwaoh0to76gaCRrjzIj3AQ/P0Kl5q9A+bm+emVpQhRQwLFHPXYZn/lIV53o2Suek/pxI2toe8gJM/YrB0EMAEYnkZX5cuErfBII65f1psAXKdZFHUqQiKKDkG0WgedWhVbNoNDe0ncK5dH9+Fp4JQqYQseU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781174295; c=relaxed/simple; bh=XoTYR/TeqMK2ycCgG5+xbyxi/eud39OEFhZfHmqkKeQ=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=VS+iV5lAtP7XdyBzZQCggOUJm6lnmkWkAFNVkMDAtj55dDp6nXhoMC7czawX/xnDAeiLsFvmjomJ5hVDCOzfLpJ6FV4F45Q/eqAyxXpjUzXRJ5q69WcVySougZEwCjIXkgYstS3GYJQLi75kGPyucC/9h2G40F820FL6QvdLkwo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=KOr4eiL9; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b=ZSbRJ7kj; arc=none smtp.client-ip=170.10.129.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="KOr4eiL9"; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b="ZSbRJ7kj" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1781174293; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=X/vtY/v3M5U6llxqJZ4Yn6/DNDaMuf4s018w2I93KIY=; b=KOr4eiL9y3zw82rQbC2xif0vWjnLMDnkkJWniygba8JDegEofT2Iw9Cquc02ZY9/5QcvjU kFoHsAoRD1OQr4DM/oHJwjRQXirm6zzTWxY3wBdnPfbSYxOG+m0y7XIL1SlZ2mT36xx73k CD7gPYk29Awk53vsw1AYVc0A14WNnqY= Received: from mail-wr1-f70.google.com (mail-wr1-f70.google.com [209.85.221.70]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-449-fZO19F9uMTSMSW45Ff-pSA-1; Thu, 11 Jun 2026 06:38:11 -0400 X-MC-Unique: fZO19F9uMTSMSW45Ff-pSA-1 X-Mimecast-MFC-AGG-ID: fZO19F9uMTSMSW45Ff-pSA_1781174291 Received: by mail-wr1-f70.google.com with SMTP id ffacd0b85a97d-45eecec7667so5446150f8f.3 for ; Thu, 11 Jun 2026 03:38:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=google; t=1781174290; x=1781779090; darn=vger.kernel.org; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=X/vtY/v3M5U6llxqJZ4Yn6/DNDaMuf4s018w2I93KIY=; b=ZSbRJ7kj7BxGFtclYdpc8Wv2tz5F7c5fyTZtYv1Dhg69LSx8ujzTYhQgCpz/Q+wNhM VOamNl9qq8lQJsqApPjKykLJ7iheRgeJEHdjYC3rX7f/53bHYXHF0ujErA1913nNP/Uw qAzy51s8X6KXTqb0EGkZmbflZPaRSFyQc4m84VKiQCpEiTnSve5/Suf1TirLkTLL9duI PGxZ4PSyzYNUWZJguA239+vEwOrkeHjwqITcV5Z9LEKn7Lr8oDL9QD8zFmfTlQ7K9+jO iv++ptQFtzssf9NUYnK9EZgm3IFCFz9s276nJ9gzdcaXc2EhE+b6Ca8vnQfhKErVnInY RcBQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1781174290; x=1781779090; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=X/vtY/v3M5U6llxqJZ4Yn6/DNDaMuf4s018w2I93KIY=; b=O5SPB+z90CSK08+5gk6DARpB9O25sx/DiPMBl/b7mDDuVIxrPV2OSK6JGThN4kSUG8 oBjJNKkaYJgWJNvRVgXIKAmbVxzN3cQt8vDYu1iCjM139AT2WKRgqgJCzauKaiBqtA+W 9rQoGHcEtGGajIlJ0VWZC7LJiYp9T0MGHFDBMSEOzruPCKhFThlaIuo42i8QEDtX4SLw ubNhp05iOm849ljkLKe/KUDzl21AIFTntiJFm0CrblWNjfCVJKpVkN7DkkHwmTBe8MKP 8TKLhKsPlcoIRW1uSUKn8saq+MUNNIpQGDLmQ/rhK7ZOlWtPQ1/IdI8Gh1xQ2q7TWB6X v+eg== X-Forwarded-Encrypted: i=1; AFNElJ/n7Z8DDKgwz3xTCspypjohho02Py8BAojWWBZIt95FOpI6eCve01oe9SBuT1rPUh380raVQUNpbf39TEY=@vger.kernel.org X-Gm-Message-State: AOJu0YwN8GmLTenJQc0NnmPZwqARgUmhnsVDTts6Gu1XRvbqAc+TYXYj 0vzrG8xFyjOskMZ1NkRvjNivmTXitN2beQsqWwvwuSFeelOLc6yQjZek0nhEmIZtq2CGEa1FokB H6EEeEeAESqnFaiaZ5SZ/ShJxGqqv9fgsyIUzM2wJ5D0SlL9Vn+2+iO5pBaw8OM8uvg== X-Gm-Gg: Acq92OFDRyRTWI5+6sAwnyHeWZywHUCVWi8axj6v1ZFY/qxRxRR4H3IfrhjChiDyAMd 2VPqRXtGTcEfR/4SVCMK/KLWTDP7ha42d/59U1vZhNO1kUqvHSt6dtFeCmuPbsl4iQlTEjX6rl0 OzfxB8Glcc6g2oNGWRz8HgmETVeH6FTYZevYdLR6i2ouWZZAGPk1byyzXN3Xe7DDFgFl8DJUzDW 1YpCGf9/08XOb4rjWyaD+X7nETkR4O7c5fABe1ingP7VXGjc/VWXQtCnk6DgzlZtx1JJEkDk1BA cRkB6ftTvvR5ul6XPtx5xW+jy4oJH4ip1pWRFWfsJIFYaT4YXYhud1fV+uYwzJp7HGWC8S3fQEy /6MGTbaQEh9ow4Xl/JUIrr6VComZ1p/7stF4NTVadL0syvCWT6LQ2SQ== X-Received: by 2002:a05:6000:41e3:b0:460:3234:293f with SMTP id ffacd0b85a97d-460677e92a0mr3648442f8f.42.1781174290511; Thu, 11 Jun 2026 03:38:10 -0700 (PDT) X-Received: by 2002:a05:6000:41e3:b0:460:3234:293f with SMTP id ffacd0b85a97d-460677e92a0mr3648382f8f.42.1781174289911; Thu, 11 Jun 2026 03:38:09 -0700 (PDT) Received: from redhat.com (IGLD-80-230-85-71.inter.net.il. [80.230.85.71]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-4601f2f67c6sm76938127f8f.16.2026.06.11.03.38.07 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 11 Jun 2026 03:38:09 -0700 (PDT) Date: Thu, 11 Jun 2026 06:38:06 -0400 From: "Michael S. Tsirkin" To: Eugenio Perez Martin Cc: Dan Carpenter , oe-kbuild@lists.linux.dev, lkp@intel.com, oe-kbuild-all@lists.linux.dev, virtualization@lists.linux.dev, Jason Wang , Cindy Lu , Xuan Zhuo , Stefano Garzarella , linux-kernel@vger.kernel.org, Laurent Vivier , Yongji Xie , Maxime Coquelin Subject: Re: [PATCH v3] vduse: Add suspend Message-ID: <20260611063730-mutt-send-email-mst@kernel.org> References: <20260610083452.477759-1-eperezma@redhat.com> <202606111115.tKKe1qCE-lkp@intel.com> <20260611050140-mutt-send-email-mst@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: On Thu, Jun 11, 2026 at 11:30:23AM +0200, Eugenio Perez Martin wrote: > On Thu, Jun 11, 2026 at 11:20 AM Dan Carpenter wrote: > > > > On Thu, Jun 11, 2026 at 05:03:24AM -0400, Michael S. Tsirkin wrote: > > > On Thu, Jun 11, 2026 at 10:18:51AM +0300, Dan Carpenter wrote: > > > > Hi Eugenio, > > > > > > > > kernel test robot noticed the following build warnings: > > > > > > > > https://git-scm.com/docs/git-format-patch#_base_tree_information] > > > > > > > > url: https://github.com/intel-lab-lkp/linux/commits/Eugenio-P-rez/vduse-Add-suspend/20260610-164534 > > > > base: next-20260609 > > > > patch link: https://lore.kernel.org/r/20260610083452.477759-1-eperezma%40redhat.com > > > > patch subject: [PATCH v3] vduse: Add suspend > > > > config: arm64-randconfig-r072-20260610 (https://download.01.org/0day-ci/archive/20260611/202606111115.tKKe1qCE-lkp@intel.com/config) > > > > compiler: aarch64-linux-gcc (GCC) 8.5.0 > > > > smatch: v0.5.0-9185-gbcc58b9c > > > > > > > > If you fix the issue in a separate patch/commit (i.e. not just a new version of > > > > the same patch/commit), kindly add following tags > > > > | Reported-by: kernel test robot > > > > | Reported-by: Dan Carpenter > > > > | Closes: https://lore.kernel.org/r/202606111115.tKKe1qCE-lkp@intel.com/ > > > > > > > > smatch warnings: > > > > drivers/vdpa/vdpa_user/vduse_dev.c:577 vduse_vq_kick() warn: inconsistent returns '&vq->kick_lock'. > > > > drivers/vdpa/vdpa_user/vduse_dev.c:1302 vduse_dev_queue_irq_work() warn: inconsistent returns '&dev->rwsem'. > > > > > > > > vim +577 drivers/vdpa/vdpa_user/vduse_dev.c > > > > > > > > c8a6153b6c59d9 Xie Yongji 2021-08-31 562 static void vduse_vq_kick(struct vduse_virtqueue *vq) > > > > c8a6153b6c59d9 Xie Yongji 2021-08-31 563 { > > > > c8a6153b6c59d9 Xie Yongji 2021-08-31 564 spin_lock(&vq->kick_lock); > > > > ^^^^^^^^^^^^^^^^^^^^^^^^^^ > > > > c8a6153b6c59d9 Xie Yongji 2021-08-31 565 if (!vq->ready) > > > > c8a6153b6c59d9 Xie Yongji 2021-08-31 566 goto unlock; > > > > c8a6153b6c59d9 Xie Yongji 2021-08-31 567 > > > > 9c4307e82fa1dc Eugenio Pérez 2026-06-10 568 guard(rwsem_read)(&vq->dev->rwsem); > > > > 9c4307e82fa1dc Eugenio Pérez 2026-06-10 569 if (vq->dev->suspended) > > > > 9c4307e82fa1dc Eugenio Pérez 2026-06-10 570 return; > > > > > > > > unlock before returning? > > > > > > > > 9c4307e82fa1dc Eugenio Pérez 2026-06-10 571 > > > > c8a6153b6c59d9 Xie Yongji 2021-08-31 572 if (vq->kickfd) > > > > 3652117f854819 Christian Brauner 2023-11-22 573 eventfd_signal(vq->kickfd); > > > > c8a6153b6c59d9 Xie Yongji 2021-08-31 574 else > > > > c8a6153b6c59d9 Xie Yongji 2021-08-31 575 vq->kicked = true; > > > > c8a6153b6c59d9 Xie Yongji 2021-08-31 576 unlock: > > > > c8a6153b6c59d9 Xie Yongji 2021-08-31 @577 spin_unlock(&vq->kick_lock); > > > > c8a6153b6c59d9 Xie Yongji 2021-08-31 578 } > > > > > > > > > I think this is fixed by: > > > > > > commit e4a249d15eb2d4b28213bebb1eefaf2e6d99de0b (HEAD -> vhost, linux-next-vhost/linux-next, kernel.org/vhost, kernel.org/test) > > > Author: Nathan Chancellor > > > Date: Wed Jun 10 12:16:49 2026 -0700 > > > > > > vduse: Fix error around jumping over a __cleanup() variable > > > > > > right? > > > > These things haven't hit linux-next yet. I found the email. > > https://lore.kernel.org/all/20260610-vduse_vq_kick-fix-guard-usage-v1-1-0ce02c08006e@kernel.org/ > > > > That only fixes the bug in vduse_vq_kick(), not the bug in > > vduse_dev_queue_irq_work(). I don't see a fix for that > > yet on lore but I may have missed it. > > > > No, I can test & send a fast patch for that. > > Thanks! Alright. -- MST