All of lore.kernel.org
 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: Wed, 9 Feb 2022 13:43:45 +0200	[thread overview]
Message-ID: <28ad66f8-48f7-2734-d018-bc30746b232a@nvidia.com> (raw)
In-Reply-To: <5d081346-9c29-b508-d4a7-f31fcbd5bfd7@redhat.com>


On 2/9/2022 4:45 AM, Jason Wang wrote:
>
> 在 2022/1/30 下午6:31, Max Gurtovoy 写道:
>>
>> 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.
>
>
> You introduce 65536 cap bits via identify. And you use u16 for command.
>
> But you map 3 commands to a single cap bit. You will be short of 
> commands. Isn't it?

No we just don't use all the bits in the identify. Identify has 2^16 
bits to be able to cover all we need.

>
>
>>
>>
>>>
>>>> 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 ?
>
>
> I meant for command without and destination. You can reserve a special 
> ID for that.
>
> Regarding the bit, if you think 32bit is not sufficient, let's just 
> use 64bit, or reserving some bytes in the command.

So 64bit size is acked by TG for vdev_id ?


>
>
>>
>>>
>>>>>>    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.
>
>
> This is how virtqueue works for all the existing device. It's as 
> simple as depending the feature to another feature which indicates the 
> existing of a specific type of virtqueue.

It is not simple and not scale.

>
>
>>
>>
>> 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 ?
>
>
> Yes, but what I meant is that we've already had a feature negotiation 
> mechanism. We should improve it instead of reinventing another one.

I've explained the scale issue.


>
> Thanks
>
>
>>
>>>
>>> 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%7Ceac245eba2584b108f4308d9eb763a85%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637799715297571729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=pJ5nr6uphZoehnGnA0VSMX5eu2x%2Fgy%2BGp6eYNz6Pi0g%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%7Ceac245eba2584b108f4308d9eb763a85%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637799715297571729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=Xkaqrd0XlxyYl%2FLmxGS%2BylV7DYZr%2F7AeEQF5ht1EzAI%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%7Ceac245eba2584b108f4308d9eb763a85%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637799715297571729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=WJfH8kSdB3z14hiThyhKSU1m3xmGLRQ0UDvgwW0ITLI%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%7Ceac245eba2584b108f4308d9eb763a85%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637799715297571729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=lzexk08M1jvX3RXvA%2B56wlYc9wPV79JmSNPgvw0F8EU%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%7Ceac245eba2584b108f4308d9eb763a85%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637799715297571729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=GoUCz7VeXW9JC47bBETePKDTAAyWqlHsjnCOAhhys1k%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%7Ceac245eba2584b108f4308d9eb763a85%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637799715297571729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=pJ5nr6uphZoehnGnA0VSMX5eu2x%2Fgy%2BGp6eYNz6Pi0g%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%7Ceac245eba2584b108f4308d9eb763a85%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637799715297571729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=Xkaqrd0XlxyYl%2FLmxGS%2BylV7DYZr%2F7AeEQF5ht1EzAI%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%7Ceac245eba2584b108f4308d9eb763a85%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637799715297571729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=WJfH8kSdB3z14hiThyhKSU1m3xmGLRQ0UDvgwW0ITLI%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%7Ceac245eba2584b108f4308d9eb763a85%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637799715297571729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=lzexk08M1jvX3RXvA%2B56wlYc9wPV79JmSNPgvw0F8EU%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%7Ceac245eba2584b108f4308d9eb763a85%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637799715297571729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=GoUCz7VeXW9JC47bBETePKDTAAyWqlHsjnCOAhhys1k%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%7Ceac245eba2584b108f4308d9eb763a85%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637799715297571729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=pJ5nr6uphZoehnGnA0VSMX5eu2x%2Fgy%2BGp6eYNz6Pi0g%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%7Ceac245eba2584b108f4308d9eb763a85%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637799715297571729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=Xkaqrd0XlxyYl%2FLmxGS%2BylV7DYZr%2F7AeEQF5ht1EzAI%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%7Ceac245eba2584b108f4308d9eb763a85%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637799715297571729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=WJfH8kSdB3z14hiThyhKSU1m3xmGLRQ0UDvgwW0ITLI%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%7Ceac245eba2584b108f4308d9eb763a85%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637799715297571729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=lzexk08M1jvX3RXvA%2B56wlYc9wPV79JmSNPgvw0F8EU%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%7Ceac245eba2584b108f4308d9eb763a85%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637799715297571729%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=GoUCz7VeXW9JC47bBETePKDTAAyWqlHsjnCOAhhys1k%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-02-09 11:43 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
2022-02-09  2:45               ` Jason Wang
2022-02-09 11:43                 ` Max Gurtovoy [this message]
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=28ad66f8-48f7-2734-d018-bc30746b232a@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.