Re: [PATCH v5] virtio: introduce SUSPEND bit in device status

David Stevens
2024-06-12T07:43:47Z

> >> +\item The driver MUST NOT make any more buffers available to the device.
> >> +\item The driver MUST NOT access any fields of any virtqueues or notify any virtqueues.
> > "send notifications for any virtqueues"?
> OK
> >
> >> +\item The driver MUST NOT access Device Configuration Space.
> > ...except for the status field, if it is part of the config space?
> Yes, that is the original design, then I am convinced that the status fields is
> not a part of the config space because it has a dedicated section in the spec.
>
> I will add this sentence in the next version.

In the discussion on v3 (unarchived due to the migration), Michael stated:

> by the way device status is not in device configuration space at all
> so this sentence does not make sense either.

Doesn't that assertion directly contradict Cornelia's feedback on this
series, which seems to be saying that the device status field can be
part of the config space? Can we get some clarity here on what exactly
the existing spec is?

-David