Hi @Jason Both hypervisor and backend device may need to support virtio-mmio device reset Adding CONFIG_VIRTIO_MMIO_POLL_RESET is for the system that has such support.
A config option (feature bit) is a better approach. Thanks for the review. BR Jianguo -----Original Message----- From: Jason Wang <[email protected]> Sent: Tuesday, September 24, 2024 3:39 PM To: Jianguo Sun (QUIC) <[email protected]> Cc: [email protected]; [email protected]; [email protected]; [email protected]; Anant Goel (QUIC) <[email protected]> Subject: Re: [PATCH 1/2] Virtio: Add CONFIG_VIRTIO_MMIO_POLL_RESET On Sun, Sep 22, 2024 at 8:35 PM Jianguo Sun <[email protected]> wrote: > > Add CONFIG_VIRTIO_MMIO_POLL_RESET to as an option to enable > synchronous reset for the MMIO based transport for virtio. > > Change-Id: Ifa7e29b4c0cfa26922535f921c6ab69eacdc4cfc > Signed-off-by: Jianguo Sun <[email protected]> > Signed-off-by: Anant Goel <[email protected]> Any reason this is not a feature but a config option? > --- > drivers/virtio/Kconfig | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/drivers/virtio/Kconfig b/drivers/virtio/Kconfig index > 42a48ac763ee..fb17b1078d53 100644 > --- a/drivers/virtio/Kconfig > +++ b/drivers/virtio/Kconfig > @@ -188,4 +188,12 @@ config VIRTIO_DEBUG > > If unsure, say N. > > +config VIRTIO_MMIO_POLL_RESET > + bool "Virti-mmio device synchronous reset support" > + depends on VIRTIO_MMIO > + help > + Say y here to enable synchronous reset for the MMIO transport based > + virtio device. After writing 0 to device_status, the driver must > + wait for a read of device_status to return 0 before reinitializing > + the device. > endif # VIRTIO_MENU > -- > 2.17.1 > Thanks
