OASIS Open Mailing List Archives  ·  All Lists  ·  virtio  ·  2023-03

virtio — archive

[Date Prev]  |  [Thread Prev]  |  [Thread Next]  |  [Date Next]   —  [Date Index]  |  [Thread Index]  |  [Month Index]  |  [List Home]

[PATCH v10 00/10] Introduce device group and device management


? 2023/3/6 08:03, Stefan Hajnoczi ??: > On Sun, Mar 05, 2023 at 04:38:59AM -0500, Michael S. Tsirkin wrote: >> On Fri, Mar 03, 2023 at 03:21:33PM -0500, Stefan Hajnoczi wrote: >>> What happens if a command takes 1 second to complete, is the device >>> allowed to process the next command from the virtqueue during this time, >>> possibly completing it before the first command? >>> >>> This requires additional clarification in the spec because "they are >>> processed by the device in the order in which they are queued" does not >>> explain whether commands block the virtqueue (in order completion) or >>> not (out of order completion). >> Oh I begin to see. Hmm how does e.g. virtio scsi handle this? > virtio-scsi, virtio-blk, and NVMe requests may complete out of order. > Several may be processed by the device at the same time. > > They rely on multi-queue for abort operations: > > In virtio-scsi the abort requests (VIRTIO_SCSI_T_TMF_ABORT_TASK) are > sent on the control virtqueue. The the request identifier namespace is > shared across all virtqueues so it's possible to abort a request that > was submitted to any command virtqueue. > > NVMe also follows the same design where abort commands are sent on the > Admin Submission Queue instead of an I/O Submission Queue. It's possible > to identify NVMe requests by <Submission Queue ID, Command Identifier>. > > virtio-blk doesn't support aborting requests. > > I think the logic behind this design is that if a queue gets stuck > processing long-running requests, then the device should not be forced > to perform lookahead in the queue to find abort commands. A separate > control/admin queue is used for the abort requests. Or device need mandate some kind of QOS here, e.g a request must be complete in some time. Otherwise we don't have sufficient reliability for using it as management task? Thanks > > Stefan

[Date Prev]  |  [Thread Prev]  |  [Thread Next]  |  [Date Next]   —  [Date Index]  |  [Thread Index]  |  [Month Index]  |  [List Home]