Hi Michael, Stefan,
> From: Parav Pandit <pa...@nvidia.com>
> Sent: 02 June 2025 08:15 AM
> To: m...@redhat.com; stefa...@redhat.com; ax...@kernel.dk;
> virtualizat...@lists.linux.dev; linux-block@vger.kernel.org
> Cc: sta...@vger.kernel.org; NBU-Contact-Li Rongqing (EXTERNAL)
> <lirongq...@baidu.com>; Chaitanya Kulkarni <k...@nvidia.com>;
> xuanz...@linux.alibaba.com; pbonz...@redhat.com;
> jasow...@redhat.com; alok.a.tiw...@oracle.com; Parav Pandit
> <pa...@nvidia.com>; Max Gurtovoy <mgurto...@nvidia.com>; Israel
> Rukshin <isra...@nvidia.com>
> Subject: [PATCH v5] virtio_blk: Fix disk deletion hang on device surprise
> removal
>
> When the PCI device is surprise removed, requests may not complete the
> device as the VQ is marked as broken. Due to this, the disk deletion hangs.
>
> Fix it by aborting the requests when the VQ is broken.
>
> With this fix now fio completes swiftly.
> An alternative of IO timeout has been considered, however when the driver
> knows about unresponsive block device, swiftly clearing them enables users
> and upper layers to react quickly.
>
> Verified with multiple device unplug iterations with pending requests in
> virtio
> used ring and some pending with the device.
>
> Fixes: 43bb40c5b926 ("virtio_pci: Support surprise removal of virtio pci
> device")
> Cc: sta...@vger.kernel.org
> Reported-by: Li RongQing <lirongq...@baidu.com>
> Closes:
> https://lore.kernel.org/virtualization/c45dd68698cd47238c55fb73ca9b4741
> @baidu.com/
> Reviewed-by: Max Gurtovoy <mgurto...@nvidia.com>
> Reviewed-by: Israel Rukshin <isra...@nvidia.com>
> Signed-off-by: Parav Pandit <pa...@nvidia.com>
>
> ---
> v4->v5:
> - fixed comment style where comment to start with one empty line at start
> - Addressed comments from Alok
> - fixed typo in broken vq check
Did you get a chance to review this version where I fixed all the comments you
proposed?
[..]
