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&data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=I8TmvBEUVIj3BqZfjX9IG3JGchDPYH2fyoWQq4MEElk%3D&reserved=0
>>>>> Feedback License:
>>>>> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fwho%2Fipr%2Ffeedback_license.pdf&data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=22N0rKTDw%2FzO7LZjAVjU5FXKfAiF9wm%2BoKAqNk5vhg4%3D&reserved=0
>>>>> List Guidelines:
>>>>> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fpolicies-guidelines%2Fmailing-lists&data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=hVC8t%2FpBFqyJNUJnXw5pUFyr2TIfc7Y8o3Tj%2F3DSkj4%3D&reserved=0
>>>>> Committee:
>>>>> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fcommittees%2Fvirtio%2F&data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=AuNIOPqOlcH%2BB3F6cZ7cNYLFazmvRvhugTy%2B4PU4fIM%3D&reserved=0
>>>>> Join OASIS:
>>>>> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fjoin%2F&data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=W2yLv9d5YcfQ%2F5GXV9uvATbpQdamK4cRCLRlWiFLvmI%3D&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&data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=I8TmvBEUVIj3BqZfjX9IG3JGchDPYH2fyoWQq4MEElk%3D&reserved=0
>>>
>>> Feedback License: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fwho%2Fipr%2Ffeedback_license.pdf&data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=22N0rKTDw%2FzO7LZjAVjU5FXKfAiF9wm%2BoKAqNk5vhg4%3D&reserved=0
>>>
>>> List Guidelines: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fpolicies-guidelines%2Fmailing-lists&data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=hVC8t%2FpBFqyJNUJnXw5pUFyr2TIfc7Y8o3Tj%2F3DSkj4%3D&reserved=0
>>>
>>> Committee: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fcommittees%2Fvirtio%2F&data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=AuNIOPqOlcH%2BB3F6cZ7cNYLFazmvRvhugTy%2B4PU4fIM%3D&reserved=0
>>>
>>> Join OASIS: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fjoin%2F&data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=W2yLv9d5YcfQ%2F5GXV9uvATbpQdamK4cRCLRlWiFLvmI%3D&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&data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=I8TmvBEUVIj3BqZfjX9IG3JGchDPYH2fyoWQq4MEElk%3D&reserved=0
>
> Feedback License: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fwho%2Fipr%2Ffeedback_license.pdf&data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=22N0rKTDw%2FzO7LZjAVjU5FXKfAiF9wm%2BoKAqNk5vhg4%3D&reserved=0
>
> List Guidelines: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fpolicies-guidelines%2Fmailing-lists&data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=hVC8t%2FpBFqyJNUJnXw5pUFyr2TIfc7Y8o3Tj%2F3DSkj4%3D&reserved=0
>
> Committee: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fcommittees%2Fvirtio%2F&data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=AuNIOPqOlcH%2BB3F6cZ7cNYLFazmvRvhugTy%2B4PU4fIM%3D&reserved=0
>
> Join OASIS: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.oasis-open.org%2Fjoin%2F&data=04%7C01%7Cmgurtovoy%40nvidia.com%7Cecb99411085446c7d06708d9e20d260e%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637789368821237568%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=W2yLv9d5YcfQ%2F5GXV9uvATbpQdamK4cRCLRlWiFLvmI%3D&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/
next prev parent 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