Thread (15 messages) 15 messages, 3 authors, 2025-08-28

Re: [PATCH] Revert "virtio_pci: Support surprise removal of virtio pci device"

From: "Michael S. Tsirkin" <mst@redhat.com>
Date: 2025-08-27 10:49:24
Also in: stable

On Wed, Aug 27, 2025 at 06:21:28AM -0400, Michael S. Tsirkin wrote:
On Tue, Aug 26, 2025 at 06:52:11PM +0000, Parav Pandit wrote:
quoted
quoted
quoted
quoted
If it does not, and a user pull out the working device, how does
your patch help?
A driver must tell that it will not follow broken ancient behaviour and at that
point device would stop its ancient backward compatibility mode.



I don't know what is "ancient backward compatibility mode".
Let me explain.
Sadly, CSPs virtio pci device implementation is done such a way that, it works with ancient Linux kernel which does not have commit 43bb40c5b9265.

OK we are getting new information here.

So let me summarize. There's a virtual system that pretends, to the
guest, that device was removed by surprise removal, but actually
device is there and is still doing DMA.
Is that a fair summary?
If that is the case, the thing to do would be to try and detect the fake
removal and then work with device as usual - device not doing DMA
after removal is pretty fundamental, after all.

For example, how about reading device control+status?

If we get all ones device has been removed
If we get 0 in bus master: device has been removed but re-inserted
Anything else is a fake removal

Hmm?


-- 
MST
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help