From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 217AE23F324; Wed, 11 Dec 2024 18:54:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943256; cv=none; b=ZK8BTNeYfztFxTVZ39h2LJB45bgMLbZsOr8WLuofWD7OHFJi9cRAIjmXbAWrsudROdYgVieg5hSd1x1b71IlH1BYbkUC4C+31K1Jtgr21rkCGG5m2be2HUKr9Wu0rPMgMWZTAhzijJFkXoABex1pKhTZOO/FyPrRbE3psp/XHoQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733943256; c=relaxed/simple; bh=NTv52oivVwj3zN4kZCEZcipef87g6n1wz8iSQxvyuq0=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=YiAkrkqHMfMh4uIVuYqC/avWBg2Kcr2YJF2tCXeXvZhiUSVskJePgQWwxT7kN8y0A23DSLy7kj9TGkKlC53IgD/YcCLgrTLUJas7QAYgJQ+1QtWY2SX8ZNoQrs6dHq7gqtiQu531lJnuAKrHkiFGn06+CX1rf09kVSpgIFHw6gw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=DI/1k1BS; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="DI/1k1BS" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 18E20C4CEDD; Wed, 11 Dec 2024 18:54:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1733943255; bh=NTv52oivVwj3zN4kZCEZcipef87g6n1wz8iSQxvyuq0=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=DI/1k1BS2FK8pQmN/XrShblcCUrclLqYQxg8oa2b+gV0FkQ+Z7jdPnmBtPKPzdFmF 4yB8YyA0GJKioYW+9Somj+X9wS7FjLoyIJfECMEW5rO+9TsDAFLlBv4KtSoQM2oGyK y6eK7Ya9TjjevRjEPLZ1PgfE5dVWjxhfbI+mgoztKIShex+gObBW62vwW142wLNnwV qAQKkvdyY4zqGqVCj2Xv9uwmbv66+BdEPtqm0D6DJ/qlBM/YIcEeQHzBi43Jq+YKE/ PIvF+nAlQ8a5LKUM8JGOpAgltgI4bD8I72sV5SJCJ8Pz4oQ4wUO4VPaR6q1jcLR98a HROAC9ZpPgn3Q== From: Sasha Levin To: linux-kernel@vger.kernel.org, stable@vger.kernel.org Cc: Ming Lei , Yi Sun , "Michael S . Tsirkin" , Jason Wang , Stefan Hajnoczi , virtualization@lists.linux.dev, Marek Szyprowski , Jens Axboe , Sasha Levin , linux-block@vger.kernel.org Subject: [PATCH AUTOSEL 5.15 10/10] virtio-blk: don't keep queue frozen during system suspend Date: Wed, 11 Dec 2024 13:53:51 -0500 Message-ID: <20241211185355.3842902-10-sashal@kernel.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20241211185355.3842902-1-sashal@kernel.org> References: <20241211185355.3842902-1-sashal@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-stable: review X-Patchwork-Hint: Ignore X-stable-base: Linux 5.15.173 Content-Transfer-Encoding: 8bit From: Ming Lei [ Upstream commit 7678abee0867e6b7fb89aa40f6e9f575f755fb37 ] Commit 4ce6e2db00de ("virtio-blk: Ensure no requests in virtqueues before deleting vqs.") replaces queue quiesce with queue freeze in virtio-blk's PM callbacks. And the motivation is to drain inflight IOs before suspending. block layer's queue freeze looks very handy, but it is also easy to cause deadlock, such as, any attempt to call into bio_queue_enter() may run into deadlock if the queue is frozen in current context. There are all kinds of ->suspend() called in suspend context, so keeping queue frozen in the whole suspend context isn't one good idea. And Marek reported lockdep warning[1] caused by virtio-blk's freeze queue in virtblk_freeze(). [1] https://lore.kernel.org/linux-block/ca16370e-d646-4eee-b9cc-87277c89c43c@samsung.com/ Given the motivation is to drain in-flight IOs, it can be done by calling freeze & unfreeze, meantime restore to previous behavior by keeping queue quiesced during suspend. Cc: Yi Sun Cc: Michael S. Tsirkin Cc: Jason Wang Cc: Stefan Hajnoczi Cc: virtualization@lists.linux.dev Reported-by: Marek Szyprowski Signed-off-by: Ming Lei Acked-by: Stefan Hajnoczi Link: https://lore.kernel.org/r/20241112125821.1475793-1-ming.lei@redhat.com Signed-off-by: Jens Axboe Signed-off-by: Sasha Levin --- drivers/block/virtio_blk.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/block/virtio_blk.c b/drivers/block/virtio_blk.c index 7f73e7447ecb5..c1087dfa332e6 100644 --- a/drivers/block/virtio_blk.c +++ b/drivers/block/virtio_blk.c @@ -988,9 +988,12 @@ static void virtblk_remove(struct virtio_device *vdev) static int virtblk_freeze(struct virtio_device *vdev) { struct virtio_blk *vblk = vdev->priv; + struct request_queue *q = vblk->disk->queue; /* Ensure no requests in virtqueues before deleting vqs. */ - blk_mq_freeze_queue(vblk->disk->queue); + blk_mq_freeze_queue(q); + blk_mq_quiesce_queue_nowait(q); + blk_mq_unfreeze_queue(q); /* Ensure we don't receive any more interrupts */ vdev->config->reset(vdev); @@ -1014,8 +1017,8 @@ static int virtblk_restore(struct virtio_device *vdev) return ret; virtio_device_ready(vdev); + blk_mq_unquiesce_queue(vblk->disk->queue); - blk_mq_unfreeze_queue(vblk->disk->queue); return 0; } #endif -- 2.43.0