Discussion of the VIRTIO specification
 help / color / mirror / Atom feed
From: Max Gurtovoy <mgurtovoy@nvidia.com>
To: Jason Wang <jasowang@redhat.com>
Cc: virtio-comment@lists.oasis-open.org
Subject: Re: [virtio-comment] [PATCH v2 1/4] Add virtio Admin Virtqueue
Date: Sun, 30 Jan 2022 12:31:06 +0200	[thread overview]
Message-ID: <2b3bd770-7ea4-e99b-9b00-3a2c386a3a04@nvidia.com> (raw)
In-Reply-To: <CACGkMEv_E7h2dJ1Wu0f0+ABwjHcRzarWSQwQbPi=MEJQpyXZ9w@mail.gmail.com>


On 1/28/2022 5:18 AM, Jason Wang wrote:
> On Thu, Jan 27, 2022 at 10:47 PM Max Gurtovoy <mgurtovoy@nvidia.com> wrote:
>>
>> On 1/27/2022 3:03 PM, Jason Wang wrote:
>>> On Thu, Jan 27, 2022 at 6:25 PM Max Gurtovoy <mgurtovoy@nvidia.com> wrote:
>>>> On 1/27/2022 5:14 AM, Jason Wang wrote:
>>>>> 在 2022/1/24 下午5:39, Max Gurtovoy 写道:
>>>>>> In one of the many use cases a user wants to manipulate features and
>>>>>> configuration of the virtio devices regardless of the device type
>>>>>> (net/block/console). Some of this configuration is generic enough. i.e
>>>>>> Number of MSI-X vectors of a virtio PCI VF device. There is a need to do
>>>>>> such features query and manipulation by its parent PCI PF.
>>>>>>
>>>>>> Currently virtio specification defines control virtqueue to manipulate
>>>>>> features and configuration of the device it operates on. However,
>>>>>> control virtqueue commands are device type specific, which makes it very
>>>>>> difficult to extend for device agnostic commands.
>>>>>>
>>>>>> To support this requirement in elegant way, this patch introduces a new
>>>>>> admin virtqueue. Admin virtqueue will use the same command format for
>>>>>> all
>>>>>> types of virtio devices.
>>>>>>
>>>>>> Manipulate features via admin virtqueue is asynchronous, scalable, easy
>>>>>> to extend and doesn't require additional and expensive on-die resources
>>>>>> to be allocated for every new feature that will be added in the future.
>>>>>>
>>>>>> Subsequent patches make use of this admin virtqueue.
>>>>>>
>>>>>> Reviewed-by: Parav Pandit <parav@nvidia.com>
>>>>>> Signed-off-by: Max Gurtovoy <mgurtovoy@nvidia.com>
>>>>>> ---
>>>>>>     admin-virtq.tex | 89 +++++++++++++++++++++++++++++++++++++++++++++++++
>>>>>>     content.tex     |  9 +++--
>>>>>>     2 files changed, 96 insertions(+), 2 deletions(-)
>>>>>>     create mode 100644 admin-virtq.tex
>>>>>>
>>>>>> diff --git a/admin-virtq.tex b/admin-virtq.tex
>>>>>> new file mode 100644
>>>>>> index 0000000..1a41c22
>>>>>> --- /dev/null
>>>>>> +++ b/admin-virtq.tex
>>>>>> @@ -0,0 +1,89 @@
>>>>>> +\section{Admin Virtqueues}\label{sec:Basic Facilities of a Virtio
>>>>>> Device / Admin Virtqueues}
>>>>>> +
>>>>>> +Admin virtqueue is used to send administrative commands to manipulate
>>>>>> +various features of the device and/or to manipulate various features,
>>>>>> +if possible, of another device within the same group (e.g. PCI VFs of
>>>>>> +a parent PCI PF device are grouped together. These devices can be
>>>>>> +optionally managed by its parent PCI PF using its admin virtqueue.).
>>>>>> +
>>>>>> +Use of Admin virtqueue is negotiated by the VIRTIO_F_ADMIN_VQ
>>>>>> +feature bit.
>>>>>> +
>>>>>> +Admin virtqueue index may vary among different device types.
>>>>>> +
>>>>>> +Regardless of device offering VIRTIO_F_IN_ORDER, admin queue command
>>>>>> buffers
>>>>>> +are used by the device in out of order manner.
>>>>>> +
>>>>>> +The Admin command set defines the commands that may be issued only
>>>>>> to the admin
>>>>>> +virtqueue. Each virtio device that advertises the VIRTIO_F_ADMIN_VQ
>>>>>> feature, MUST
>>>>>> +support all the mandatory admin commands. A device MAY support also
>>>>>> one or more
>>>>>> +optional admin commands. All commands are of the following form:
>>>>> Do we need a way for advertising the supported optional commands (or
>>>>> features bits)? Otherwise dealing with the compatibility will result
>>>>> of per command probing.
>>>>>
>>>>>
>>>>>> +
>>>>>> +\begin{lstlisting}
>>>>>> +struct virtio_admin_cmd {
>>>>>> +        /* Device-readable part */
>>>>>> +        u16 command;
>>>>> Two questions:
>>>>>
>>>>> 1) It looks to me it's better to have a explicit device_id here other
>>>>> than embed it in each command
>>>> this was already discussed.
>>>>
>>>> We agreed that some commands are not referring to different device so
>>>> no, we don't need to put it in generic structure.
>>>>
>>>> I'm not against putting such id but we don't have an exact idea how will
>>>> SF device_id will look like.
>>>>
>>>> Maybe it will be some 128 bit uuid ? maybe u64 ?
>>> How do you know u16 is sufficient for command?
>> it's sufficient for VFs and this is the subject of the proposed commands.
> So VIRTIO_ADMIN_CAPS_IDENTIFY maps to 3 commands:
>
> VIRTIO_ADMIN_PCI_SRIOV_ATTRS,
> VIRTIO_ADMIN_PCI_VF_INTERRUPT_CONFIG_SET,
> VIRTIO_ADMIN_PCI_VF_INTERRUPT_CONFIG_GET
>
> Unless you do 1:1 mapping between cap bit and command, you will be
> short of commands if you're using u16.

2^16 admin commands are more than enough.


>
>> When you'll add SF to virtio spec we'll create new commands for it with
>> SF_id.
>>
>> Again, this was discussed with Parav in V1 and was close. Please don't
>> re-open.
> I don't think so, you can reserve a special device id for the PF itself.

can you propose something concrete ?

are you sure by 100% that sf_id is 32 bit ? can the TG ack on that ?

>
>>>>    how can we guess ?
>>> You know it's you but not me that tries to propose this solution, right?
>> I'm proposing a solution but you ask questions that we answer but
>> somehow they are asked again.
>>>>> 2) virtio-net cvq use class/command which seems more elegant, e.g all
>>>>> commands belongs to MSI could be grouped into the same class
>>>> why can't we do it in u16 command opcode ?
>>> Where did you see I say you can't use u16?
>> you suggest to replace it to class + command.
>>
>> I don't understand why.
>>
>> If it's not mandatory, please ack.
>>
>>
>>>> are you open for changes or
>>>> should we copy/paste from net cvq ?
>>> Net cvq comes first, it's natural to ask why you do things differently.
>> I answered why.
>>
>> Admin command set has a different purpose.
>>
>>>> Let's ask differently. Why we need class/command structure ? why is it
>>>> more elegant ?
>>> Don't you see my reply above?
>>>
>>> "
>>> commands belongs to MSI could be grouped into the same class
>>> "
>> It's doesn't explain why class A + commands 1, 2 is better than opcodes
>> 1 + 2 without a class.
> Well, if you read how cvq work it's not hard to get the conclusion,
>
> It's easier to be mapped to a feature bit. You just map a class and that's all.
>
> Otherwise, you need to feature X:  command A,B,C,D ....

But I just added one feature bit: the existence of the AQ.

in your suggestion I'll need to add for each feature:

Feature X can be negotiated only if AQ feature is negotiated.

And if feature X will have a small extension so a new feature will be 
needed and it will say:

Feature Y can be negotiated in feature X and feature AQ are negotiated.

or if feature X and feature Y are mutual exclusive then:

feature X can't be negotiated if feature Y is negotiated.

feature X can be negotiated only if feature AQ is negotiated.


and this is a small example. We will have more and more Admin 
functionality and this will be impossible to scale.

A simple identify command solve this.

>
>>
>>>>>> +        u8 command_specific_data[];
>>>>>> +
>>>>>> +        /* Device-writable part */
>>>>>> +        u8 status;
>>>>>> +        u8 command_specific_error;
>>>>> I wonder the reason why not using a single field for the above two
>>>>> fields. (Or any value for separating command specific error out, we
>>>>> don't do that for virtio-net cvq).
>>>> each command have its own specific errors.
>>>>
>>>> virtio net cvq is not generic - it's a net device cvqueue.
>>> We're talking about the command format, not its semantics, right?
>>>
>>> It's command format is pretty general.
>> in the Admin command set we would like to have better format that will
>> allow users/drivers to better understand device error.
>>
>> It is very common in many HW devices and specs out there.
>>
>> I hope it doesn't critical for you as a concept.
>>
>>>> To make it generic we need to separate.
>>>>
>>>>>> +        u8 command_specific_result[];
>>>>>> +};
>>>>>> +\end{lstlisting}
>>>>>> +
>>>>>> +The following table describes the generic Admin status codes:
>>>>>> +
>>>>>> +\begin{tabular}{|l|l|l|}
>>>>>> +\hline
>>>>>> +Opcode & Status & Description \\
>>>>>> +\hline \hline
>>>>>> +00h   & VIRTIO_ADMIN_STATUS_OK    & successful completion  \\
>>>>>> +\hline
>>>>>> +01h   & VIRTIO_ADMIN_STATUS_CS_ERR    & command specific error  \\
>>>>>> +\hline
>>>>>> +02h   & VIRTIO_ADMIN_STATUS_COMMAND_UNSUPPORTED    & unsupported or
>>>>>> invalid opcode  \\
>>>>>> +\hline
>>>>>> +\end{tabular}
>>>>>> +
>>>>>> +The \field{command} and \field{command_specific_data} are
>>>>>> +set by the driver, and the device sets the \field{status}, the
>>>>>> +\field{command_specific_error} and the \field{command_specific_result},
>>>>>> +if needed.
>>>>>> +
>>>>>> +The \field{command_specific_error} should be inspected by the driver
>>>>>> only if \field{status} is set to
>>>>>> +VIRTIO_ADMIN_STATUS_CS_ERR by the device. In this case, the content
>>>>>> of \field{command_specific_error}
>>>>>> +holds the command specific error. If \field{status} is not set to
>>>>>> VIRTIO_ADMIN_STATUS_CS_ERR, the
>>>>>> +\field{command_specific_error} value is undefined.
>>>>>> +
>>>>>> +The following table describes the Admin command set:
>>>>>> +
>>>>>> +\begin{tabular}{|l|l|l|}
>>>>>> +\hline
>>>>>> +Opcode & Command & M/O \\
>>>>>> +\hline \hline
>>>>>> +0000h   & VIRTIO_ADMIN_CAPS_IDENTIFY    & M  \\
>>>>>> +\hline
>>>>>> +0001h - 7FFFh   & Generic admin cmds    & -  \\
>>>>>> +\hline
>>>>>> +8000h - FFFFh   & Reserved    & - \\
>>>>>> +\hline
>>>>>> +\end{tabular}
>>>>> See above, I'd suggest to use class/command as virtio-net cvq.
>>>> see my comment above.
>> I answered above.
>>>>>> +
>>>>>> +\subsection{VIRTIO ADMIN CAPS IDENTIFY command}\label{sec:Basic
>>>>>> Facilities of a Virtio Device / Admin Virtqueues / VIRTIO ADMIN CAPS
>>>>>> IDENTIFY command}
>>>>>> +
>>>>>> +The VIRTIO_ADMIN_CAPS_IDENTIFY command has no command specific data
>>>>>> set by the driver.
>>>>>> +This command upon success, returns a data buffer that describes
>>>>>> information about the supported
>>>>>> +admin capabilities by the device. This information is of form:
>>>>>> +\begin{lstlisting}
>>>>>> +struct virtio_admin_caps_identify_result {
>>>>>> +       /*
>>>>>> +        * caps[0] - Bit 0x0 - Bit 0x7 are reserved
>>>>>> +        * caps[1] - Bit 0x0 - Bit 0x7 are reserved
>>>>>> +        * caps[2] - Bit 0x0 - Bit 0x7 are reserved
>>>>>> +        * ....
>>>>>> +        * caps[8191] - Bit 0x0 - Bit 0x7 are reserved
>>>>>> +        */
>>>>>> +       u8 caps[8192];
>>>>>> +};
>>>>> Ok, so I see the identify command. But I wonder if we can do that via
>>>>> the feature bits?
>>>> It doesn't scale. I tried it. It doesn't work.
>>> What prevents you from improving the scalability instead of trying to
>>> revinting a duplicated mechanism?
>> It was already discussed.
>>
>> Spec doesn't scale, too many constrains between command-A,B,C and
>> feature 44,45, 46 and for transport X, Y, Z.
> What I meant is, if you feel the current feature negotiation doesn't
> scale, you can improve it instead of reinventing it partially (e.g no
> negotiation) .

We're adding an identify command for the admin command set.

did you review it ?

>
> Thanks
>
>>> Thanks
>>>
>>>> Thanks.
>>>>
>>>>> Thanks
>>>>>
>>>>>
>>>>>> +\end{lstlisting}
>>>>>> diff --git a/content.tex b/content.tex
>>>>>> index 32de668..c524fab 100644
>>>>>> --- a/content.tex
>>>>>> +++ b/content.tex
>>>>>> @@ -99,10 +99,10 @@ \section{Feature Bits}\label{sec:Basic Facilities
>>>>>> of a Virtio Device / Feature B
>>>>>>     \begin{description}
>>>>>>     \item[0 to 23] Feature bits for the specific device type
>>>>>>     -\item[24 to 40] Feature bits reserved for extensions to the queue and
>>>>>> +\item[24 to 41] Feature bits reserved for extensions to the queue and
>>>>>>       feature negotiation mechanisms
>>>>>>     -\item[41 and above] Feature bits reserved for future extensions.
>>>>>> +\item[42 and above] Feature bits reserved for future extensions.
>>>>>>     \end{description}
>>>>>>       \begin{note}
>>>>>> @@ -449,6 +449,8 @@ \section{Exporting Objects}\label{sec:Basic
>>>>>> Facilities of a Virtio Device / Expo
>>>>>>     types. It is RECOMMENDED that devices generate version 4
>>>>>>     UUIDs as specified by \hyperref[intro:rfc4122]{[RFC4122]}.
>>>>>>     +\input{admin-virtq.tex}
>>>>>> +
>>>>>>     \chapter{General Initialization And Device
>>>>>> Operation}\label{sec:General Initialization And Device Operation}
>>>>>>       We start with an overview of device initialization, then expand
>>>>>> on the
>>>>>> @@ -6847,6 +6849,9 @@ \chapter{Reserved Feature
>>>>>> Bits}\label{sec:Reserved Feature Bits}
>>>>>>       that the driver can reset a queue individually.
>>>>>>       See \ref{sec:Basic Facilities of a Virtio Device / Virtqueues /
>>>>>> Virtqueue Reset}.
>>>>>>     +  \item[VIRTIO_F_ADMIN_VQ (41)] This feature indicates that
>>>>>> +  the device supports administration virtqueue negotiation.
>>>>>> +
>>>>>>     \end{description}
>>>>>>       \drivernormative{\section}{Reserved Feature Bits}{Reserved
>>>>>> Feature Bits}
>>>>> This publicly archived list offers a means to provide input to the
>>>>> OASIS Virtual I/O Device (VIRTIO) TC.
>>>>>
>>>>> In order to verify user consent to the Feedback License terms and
>>>>> to minimize spam in the list archive, subscription is required
>>>>> before posting.
>>>>>
>>>>> Subscribe: virtio-comment-subscribe@lists.oasis-open.org
>>>>> Unsubscribe: virtio-comment-unsubscribe@lists.oasis-open.org
>>>>> List help: virtio-comment-help@lists.oasis-open.org
>>>>> List archive:
>>>>> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.oasis-open.org%2Farchives%2Fvirtio-comment%2F&amp;data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=I8TmvBEUVIj3BqZfjX9IG3JGchDPYH2fyoWQq4MEElk%3D&amp;reserved=0
>>>>> Feedback License:
>>>>> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fwho%2Fipr%2Ffeedback_license.pdf&amp;data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=22N0rKTDw%2FzO7LZjAVjU5FXKfAiF9wm%2BoKAqNk5vhg4%3D&amp;reserved=0
>>>>> List Guidelines:
>>>>> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fpolicies-guidelines%2Fmailing-lists&amp;data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=hVC8t%2FpBFqyJNUJnXw5pUFyr2TIfc7Y8o3Tj%2F3DSkj4%3D&amp;reserved=0
>>>>> Committee:
>>>>> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fcommittees%2Fvirtio%2F&amp;data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=AuNIOPqOlcH%2BB3F6cZ7cNYLFazmvRvhugTy%2B4PU4fIM%3D&amp;reserved=0
>>>>> Join OASIS:
>>>>> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fjoin%2F&amp;data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=W2yLv9d5YcfQ%2F5GXV9uvATbpQdamK4cRCLRlWiFLvmI%3D&amp;reserved=0
>>>>>
>>> This publicly archived list offers a means to provide input to the
>>>
>>> OASIS Virtual I/O Device (VIRTIO) TC.
>>>
>>>
>>>
>>> In order to verify user consent to the Feedback License terms and
>>>
>>> to minimize spam in the list archive, subscription is required
>>>
>>> before posting.
>>>
>>>
>>>
>>> Subscribe: virtio-comment-subscribe@lists.oasis-open.org
>>>
>>> Unsubscribe: virtio-comment-unsubscribe@lists.oasis-open.org
>>>
>>> List help: virtio-comment-help@lists.oasis-open.org
>>>
>>> List archive: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.oasis-open.org%2Farchives%2Fvirtio-comment%2F&amp;data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=I8TmvBEUVIj3BqZfjX9IG3JGchDPYH2fyoWQq4MEElk%3D&amp;reserved=0
>>>
>>> Feedback License: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fwho%2Fipr%2Ffeedback_license.pdf&amp;data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=22N0rKTDw%2FzO7LZjAVjU5FXKfAiF9wm%2BoKAqNk5vhg4%3D&amp;reserved=0
>>>
>>> List Guidelines: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fpolicies-guidelines%2Fmailing-lists&amp;data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=hVC8t%2FpBFqyJNUJnXw5pUFyr2TIfc7Y8o3Tj%2F3DSkj4%3D&amp;reserved=0
>>>
>>> Committee: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fcommittees%2Fvirtio%2F&amp;data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=AuNIOPqOlcH%2BB3F6cZ7cNYLFazmvRvhugTy%2B4PU4fIM%3D&amp;reserved=0
>>>
>>> Join OASIS: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fjoin%2F&amp;data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=W2yLv9d5YcfQ%2F5GXV9uvATbpQdamK4cRCLRlWiFLvmI%3D&amp;reserved=0
>>>
>
> This publicly archived list offers a means to provide input to the
>
> OASIS Virtual I/O Device (VIRTIO) TC.
>
>
>
> In order to verify user consent to the Feedback License terms and
>
> to minimize spam in the list archive, subscription is required
>
> before posting.
>
>
>
> Subscribe: virtio-comment-subscribe@lists.oasis-open.org
>
> Unsubscribe: virtio-comment-unsubscribe@lists.oasis-open.org
>
> List help: virtio-comment-help@lists.oasis-open.org
>
> List archive: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.oasis-open.org%2Farchives%2Fvirtio-comment%2F&amp;data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=I8TmvBEUVIj3BqZfjX9IG3JGchDPYH2fyoWQq4MEElk%3D&amp;reserved=0
>
> Feedback License: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fwho%2Fipr%2Ffeedback_license.pdf&amp;data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=22N0rKTDw%2FzO7LZjAVjU5FXKfAiF9wm%2BoKAqNk5vhg4%3D&amp;reserved=0
>
> List Guidelines: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fpolicies-guidelines%2Fmailing-lists&amp;data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=hVC8t%2FpBFqyJNUJnXw5pUFyr2TIfc7Y8o3Tj%2F3DSkj4%3D&amp;reserved=0
>
> Committee: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fcommittees%2Fvirtio%2F&amp;data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=AuNIOPqOlcH%2BB3F6cZ7cNYLFazmvRvhugTy%2B4PU4fIM%3D&amp;reserved=0
>
> Join OASIS: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fjoin%2F&amp;data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=W2yLv9d5YcfQ%2F5GXV9uvATbpQdamK4cRCLRlWiFLvmI%3D&amp;reserved=0
>

This publicly archived list offers a means to provide input to the
OASIS Virtual I/O Device (VIRTIO) TC.

In order to verify user consent to the Feedback License terms and
to minimize spam in the list archive, subscription is required
before posting.

Subscribe: virtio-comment-subscribe@lists.oasis-open.org
Unsubscribe: virtio-comment-unsubscribe@lists.oasis-open.org
List help: virtio-comment-help@lists.oasis-open.org
List archive: https://lists.oasis-open.org/archives/virtio-comment/
Feedback License: https://www.oasis-open.org/who/ipr/feedback_license.pdf
List Guidelines: https://www.oasis-open.org/policies-guidelines/mailing-lists
Committee: https://www.oasis-open.org/committees/virtio/
Join OASIS: https://www.oasis-open.org/join/


  reply	other threads:[~2022-01-30 10:31 UTC|newest]

Thread overview: 69+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-24  9:39 [PATCH v2 0/4] VIRTIO: Provision maximum MSI-X vectors for a VF Max Gurtovoy
2022-01-24  9:39 ` [PATCH v2 1/4] Add virtio Admin Virtqueue Max Gurtovoy
2022-01-26 14:40   ` Michael S. Tsirkin
2022-01-26 14:54     ` [virtio-comment] " Max Gurtovoy
2022-01-26 15:03       ` Michael S. Tsirkin
2022-01-26 15:16         ` [virtio-comment] " Max Gurtovoy
2022-01-27  3:56         ` Parav Pandit
2022-01-27  3:55     ` Parav Pandit
2022-01-27  3:56       ` Jason Wang
2022-01-27  3:14   ` [virtio-comment] " Jason Wang
2022-01-27 10:25     ` Max Gurtovoy
2022-01-27 13:03       ` Jason Wang
2022-01-27 14:46         ` Max Gurtovoy
2022-01-28  3:18           ` Jason Wang
2022-01-30 10:31             ` Max Gurtovoy [this message]
2022-02-09  2:45               ` Jason Wang
2022-02-09 11:43                 ` Max Gurtovoy
2022-01-28 12:14   ` [virtio-comment] " Cornelia Huck
2022-01-28 12:47     ` Michael S. Tsirkin
2022-01-28 15:49       ` [virtio-comment] " Cornelia Huck
2022-01-28 15:52         ` Michael S. Tsirkin
2022-01-28 16:14           ` [virtio-dev] " Cornelia Huck
2022-01-28 16:16             ` Michael S. Tsirkin
2022-01-28 16:23               ` [virtio-dev] " Cornelia Huck
2022-01-29  3:53           ` Jason Wang
2022-01-30  9:13             ` Max Gurtovoy
2022-01-30  9:40               ` Michael S. Tsirkin
2022-01-30  9:56                 ` Max Gurtovoy
2022-01-30 14:41                   ` Michael S. Tsirkin
2022-01-30 15:12                     ` Max Gurtovoy
2022-01-30 15:30                       ` Michael S. Tsirkin
2022-01-30 18:23                         ` [virtio-comment] " Max Gurtovoy
2022-01-30 21:15                           ` Michael S. Tsirkin
2022-01-31  9:16                         ` [virtio-dev] " Cornelia Huck
2022-01-31 13:40                           ` Michael S. Tsirkin
2022-01-31 14:26                             ` [virtio-comment] " Cornelia Huck
2022-01-31 14:52                               ` Michael S. Tsirkin
2022-01-31 15:48                                 ` [virtio-dev] " Cornelia Huck
2022-01-31 16:00                                   ` Michael S. Tsirkin
2022-01-31 16:12                                 ` Halil Pasic
2022-01-31 17:10                                   ` [virtio-dev] " Cornelia Huck
2022-01-31 17:22                                     ` Michael S. Tsirkin
2022-02-01 11:53                                     ` [virtio-dev] " Halil Pasic
2022-02-01 17:01                                       ` Cornelia Huck
2022-02-01 18:34                                         ` Michael S. Tsirkin
2022-01-31 15:47                           ` Halil Pasic
2022-01-31 16:01                             ` Michael S. Tsirkin
2022-01-31 16:12                             ` Cornelia Huck
2022-02-09  2:27                         ` Jason Wang
2022-02-09  7:46                           ` Michael S. Tsirkin
2022-01-30  9:39             ` Michael S. Tsirkin
2022-01-30 11:21     ` [virtio-comment] " Max Gurtovoy
2022-01-30 14:37       ` Michael S. Tsirkin
2022-01-24  9:39 ` [virtio-comment] [PATCH v2 2/4] virtio-blk: add support for VIRTIO_F_ADMIN_VQ Max Gurtovoy
2022-01-24  9:39 ` [PATCH v2 3/4] virtio-net: " Max Gurtovoy
2022-01-24  9:39 ` [PATCH v2 4/4] Add support for MSI-X vectors configuration for PCI VFs Max Gurtovoy
2022-01-25 11:53   ` Michael S. Tsirkin
2022-01-26 13:03     ` Parav Pandit
2022-01-26 14:08       ` [virtio-comment] " Michael S. Tsirkin
2022-01-27  3:40         ` Parav Pandit
2022-01-27 12:39           ` Michael S. Tsirkin
2022-01-27 14:16             ` Parav Pandit
2022-01-27 17:28               ` Michael S. Tsirkin
2022-01-27  3:36   ` Jason Wang
2022-01-27  5:22     ` Parav Pandit
2022-01-28  3:23       ` [virtio-comment] " Jason Wang
2022-01-28  3:30         ` Parav Pandit
2022-01-28  3:35           ` Jason Wang
2022-01-28  3:45             ` Parav Pandit

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2b3bd770-7ea4-e99b-9b00-3a2c386a3a04@nvidia.com \
    --to=mgurtovoy@nvidia.com \
    --cc=jasowang@redhat.com \
    --cc=virtio-comment@lists.oasis-open.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox