On Mon, Jan 17, 2022 at 01:57:47PM +0800, Jason Wang wrote:
> How about something like:
>
> "
> Except for the MSI transactions, the device MUST use PASID TLP prefix
> for the following memory transactions initiated by the virtqueue that
> belong to a virtqueue group if a valid PASID is assigned and PASID is
> enabled in the PASID extended capability:
>
> \begin{itemize}
> \item Memory Requests (including AtomicOp Requests) with Untranslated
> Addresses
> \item Address Translation Requests
> \item Page Request Messages
> \end{itemize}
>
> The device MUST NOT use PASID TLP prefix for the MSI memory
> transactions.
>
Sure, that looks good. Maybe splitting it would make it easier to explain.
If we force the driver to enable the PASID cap before using PASID:
"The driver MUST NOT assign a PASID to a virtqueue group if the PASID
extended capability is not enabled."
Then the device description is not concerned about the PASID cap:
"If a valid PASID is assigned to a virtqueue group, the device MUST use a
PASID TLP prefix for memory transactions of the following types initiated
by any virtqueue in the group:
...
The device MUST NOT use a PASID TLP prefix for MSI memory transactions.
"
Thanks,
Jean
---------------------------------------------------------------------
To unsubscribe, e-mail: virtio-dev-unsubscr...@lists.oasis-open.org
For additional commands, e-mail: virtio-dev-h...@lists.oasis-open.org
