Re: [PATCH v2] virtgpu: don't reset on shutdown

From: Gerd Hoffmann
Date: Wed Apr 16 2025 - 10:13:13 EST


On Tue, Apr 15, 2025 at 10:00:48AM -0400, Michael S. Tsirkin wrote:
> On Tue, Apr 15, 2025 at 01:16:32PM +0200, Gerd Hoffmann wrote:
> > Hi,
> >
> > > +static void virtio_gpu_shutdown(struct virtio_device *vdev)
> > > +{
> > > + /*
> > > + * drm does its own synchronization on shutdown.
> > > + * Do nothing here, opt out of device reset.
> > > + */
> >
> > I think a call to 'drm_dev_unplug()' is what you need here.
> >
> > take care,
> > Gerd
>
> My patch reverts the behaviour back to what it was, so pls go
> ahead and send a patch on top? I won't be able to explain
> what it does and why it's needed.

See below. Untested.

Eric, can you give this a spin?

thanks,
Gerd

----------------------- cut here -------------------------------