virtio-blk: set req->state to MQ_RQ_COMPLETE after polling I/O is finished
authorSuwan Kim <suwan.kim027@gmail.com>
Wed, 21 Dec 2022 14:54:55 +0000 (23:54 +0900)
committerMichael S. Tsirkin <mst@redhat.com>
Tue, 21 Feb 2023 00:26:57 +0000 (19:26 -0500)
commit489e18f3d73282f6bf6203324b3b17d459e2e750
treefc216609728a45236f7c2d918c125c5a5ec15326
parent2b034e82ffc5f24ec54e47785f22f2bc33fda383
virtio-blk: set req->state to MQ_RQ_COMPLETE after polling I/O is finished

Driver should set req->state to MQ_RQ_COMPLETE after it finishes to process
req. But virtio-blk doesn't set MQ_RQ_COMPLETE after virtblk_poll() handles
req and req->state still remains MQ_RQ_IN_FLIGHT. Fortunately so far there
is no issue about it because blk_mq_end_request_batch() sets req->state to
MQ_RQ_IDLE.

In this patch, virblk_poll() calls blk_mq_complete_request_remote() to set
req->state to MQ_RQ_COMPLETE before it adds req to a batch completion list.
So it properly sets req->state after polling I/O is finished.

Fixes: 4e0400525691 ("virtio-blk: support polling I/O")
Signed-off-by: Suwan Kim <suwan.kim027@gmail.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Message-Id: <20221221145456.281218-2-suwan.kim027@gmail.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
drivers/block/virtio_blk.c