virtio-comment

 View Only
Expand all | Collapse all

[PATCH v6] virtio-net: Improve introductory description

  • 1.  [PATCH v6] virtio-net: Improve introductory description

    Posted 02-14-2023 18:04
    The control VQ of the virtio network device is used beyond advance
    steering control. The control VQ dynamically changes multiple features
    of the initialized device.

    Hence, update this area of control VQ introductory description at few
    places and also place the link to its description.

    Also update the introduction section to better describe receive and
    transmit virtqueues.

    Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156
    Reviewed-by: David Edmondson <david.edmondson@oracle.com>
    Signed-off-by: Parav Pandit <parav@nvidia.com>
    ---
    changelog:
    v5->v6:
    - changed 'many' to 'various'
    - rebased
    v4->v5:
    - fixed typo from virtuqueue to virtqueue
    v3->v4:
    - write buffer add/remove, instead of enqueue/dequeue, post
    - changed 'many' to 'various'
    v2->v3:
    - further improve introduction
    v1->v2:
    - corrected/added article before transmit and receive virtqueue
    - replaced listed with described
    - updated set to negotiated for CTRL_VQ
    - improved introduction wordings
    v0->v1:
    - replaced command queue to control virtqueue to reflect current state
    - added link of control vq to its detailed section
    - removed reference of cvq from more places that limits it to steering
    - Addressed Cornelia's review comments:
    - updated introduction section to better describe rq, sq, cvq.
    - explicitly wrote that cvq is optional along with its feature bit
    related description
    - dropped few items from Cornelia's review as they do not exactly
    fit with latest 1.2 specification as below.
    - avoided listing number of queues in introduction as its linked to q
    reset which can disable all queues
    - avoid mentioning cvq as optional as there is better section for it
    - avoid talking about queue pairs as spec and use cases allows not
    operate in pairs mode
    ---
    device-types/net/description.tex | 23 +++++++++++++----------
    1 file changed, 13 insertions(+), 10 deletions(-)

    diff --git a/device-types/net/description.tex b/device-types/net/description.tex
    index 88a5770..a197e1a 100644
    --- a/device-types/net/description.tex
    +++ b/device-types/net/description.tex
    @@ -2,13 +2,15 @@ \section{Network Device}\label{sec:Device Types / Network Device}

    The virtio network device is a virtual network interface controller.
    It consists of a virtual Ethernet link which connects the device
    -to the Ethernet network. It is the most complex of the devices
    -supported so far by virtio. It has enhanced rapidly and demonstrates
    -clearly how support for new features are added to an existing
    -device. Empty buffers are placed in one virtqueue for receiving
    -packets, and outgoing packets are enqueued into another for
    -transmission in that order. A third command queue is used to
    -control advanced filtering features.
    +to the Ethernet network. The device has transmit and receive
    +queues. The driver adds empty buffers to the receive virtqueue.
    +The device receives incoming packets from the link; the device
    +places these incoming packets in the receive virtqueue buffers.
    +The driver adds outgoing packets to the transmit virtqueue. The device
    +removes these packets from the transmit virtqueue and sends them to
    +the link. The device may have a control virtqueue. The driver
    +uses the control virtqueue to dynamically manipulate various
    +features of the initialized device.

    \subsection{Device ID}\label{sec:Device Types / Network Device / Device ID}

    @@ -28,7 +30,8 @@ \subsection{Virtqueues}\label{sec:Device Types / Network Device / Virtqueues}
    N=1 if neither VIRTIO_NET_F_MQ nor VIRTIO_NET_F_RSS are negotiated, otherwise N is set by
    \field{max_virtqueue_pairs}.

    - controlq only exists if VIRTIO_NET_F_CTRL_VQ set.
    +controlq is optional; it only exists if VIRTIO_NET_F_CTRL_VQ is
    +negotiated.

    \subsection{Feature bits}\label{sec:Device Types / Network Device / Feature bits}

    @@ -389,8 +392,8 @@ \subsection{Device Operation}\label{sec:Device Types / Network Device / Device O
    };
    \end{lstlisting}

    -The controlq is used to control device features such as
    -filtering.
    +The controlq is used to control various device features described further in
    +section \ref{sec:Device Types / Network Device / Device Operation / Control Virtqueue}.

    \subsubsection{Legacy Interface: Device Operation}\label{sec:Device Types / Network Device / Device Operation / Legacy Interface: Device Operation}
    When using the legacy interface, transitional devices and drivers
    --
    2.26.2




  • 2.  Re: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-16-2023 14:10
    On Tue, Feb 14 2023, Parav Pandit <parav@nvidia.com> wrote:

    > The control VQ of the virtio network device is used beyond advance
    > steering control. The control VQ dynamically changes multiple features
    > of the initialized device.
    >
    > Hence, update this area of control VQ introductory description at few
    > places and also place the link to its description.
    >
    > Also update the introduction section to better describe receive and
    > transmit virtqueues.
    >
    > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156
    > Reviewed-by: David Edmondson <david.edmondson@oracle.com>
    > Signed-off-by: Parav Pandit <parav@nvidia.com>

    Are you requesting a vote for this?




  • 3.  Re: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-16-2023 14:10
    On Tue, Feb 14 2023, Parav Pandit <parav@nvidia.com> wrote:

    > The control VQ of the virtio network device is used beyond advance
    > steering control. The control VQ dynamically changes multiple features
    > of the initialized device.
    >
    > Hence, update this area of control VQ introductory description at few
    > places and also place the link to its description.
    >
    > Also update the introduction section to better describe receive and
    > transmit virtqueues.
    >
    > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156
    > Reviewed-by: David Edmondson <david.edmondson@oracle.com>
    > Signed-off-by: Parav Pandit <parav@nvidia.com>

    Are you requesting a vote for this?




  • 4.  RE: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-16-2023 14:30

    > From: Cornelia Huck <cohuck@redhat.com>
    > Sent: Thursday, February 16, 2023 9:10 AM
    >
    > On Tue, Feb 14 2023, Parav Pandit <parav@nvidia.com> wrote:
    >
    > > The control VQ of the virtio network device is used beyond advance
    > > steering control. The control VQ dynamically changes multiple features
    > > of the initialized device.
    > >
    > > Hence, update this area of control VQ introductory description at few
    > > places and also place the link to its description.
    > >
    > > Also update the introduction section to better describe receive and
    > > transmit virtqueues.
    > >
    > > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156
    > > Reviewed-by: David Edmondson <david.edmondson@oracle.com>
    > > Signed-off-by: Parav Pandit <parav@nvidia.com>
    >
    > Are you requesting a vote for this?

    I was not sure if its need voting as most are editorial cleanups, better description, structure movement.
    Please advise if we need voting.



  • 5.  RE: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-16-2023 14:30

    > From: Cornelia Huck <cohuck@redhat.com>
    > Sent: Thursday, February 16, 2023 9:10 AM
    >
    > On Tue, Feb 14 2023, Parav Pandit <parav@nvidia.com> wrote:
    >
    > > The control VQ of the virtio network device is used beyond advance
    > > steering control. The control VQ dynamically changes multiple features
    > > of the initialized device.
    > >
    > > Hence, update this area of control VQ introductory description at few
    > > places and also place the link to its description.
    > >
    > > Also update the introduction section to better describe receive and
    > > transmit virtqueues.
    > >
    > > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156
    > > Reviewed-by: David Edmondson <david.edmondson@oracle.com>
    > > Signed-off-by: Parav Pandit <parav@nvidia.com>
    >
    > Are you requesting a vote for this?

    I was not sure if its need voting as most are editorial cleanups, better description, structure movement.
    Please advise if we need voting.



  • 6.  Re: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-16-2023 15:50
    On Thu, Feb 16, 2023 at 02:30:03PM +0000, Parav Pandit wrote:
    >
    > > From: Cornelia Huck <cohuck@redhat.com>
    > > Sent: Thursday, February 16, 2023 9:10 AM
    > >
    > > On Tue, Feb 14 2023, Parav Pandit <parav@nvidia.com> wrote:
    > >
    > > > The control VQ of the virtio network device is used beyond advance
    > > > steering control. The control VQ dynamically changes multiple features
    > > > of the initialized device.
    > > >
    > > > Hence, update this area of control VQ introductory description at few
    > > > places and also place the link to its description.
    > > >
    > > > Also update the introduction section to better describe receive and
    > > > transmit virtqueues.
    > > >
    > > > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156
    > > > Reviewed-by: David Edmondson <david.edmondson@oracle.com>
    > > > Signed-off-by: Parav Pandit <parav@nvidia.com>
    > >
    > > Are you requesting a vote for this?
    >
    > I was not sure if its need voting as most are editorial cleanups, better description, structure movement.
    > Please advise if we need voting.


    The bylaws are here:
    https://github.com/oasis-tcs/virtio-spec/blob/master/README.md
    Minor cleanups, including editorial formatting changes, spelling and
    typo fixes can be committed directly into git for approval as part of
    the next specification approval ballot.

    This is way beyond that. And generally when in doubt vote.

    --
    MST




  • 7.  Re: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-16-2023 15:50
    On Thu, Feb 16, 2023 at 02:30:03PM +0000, Parav Pandit wrote:
    >
    > > From: Cornelia Huck <cohuck@redhat.com>
    > > Sent: Thursday, February 16, 2023 9:10 AM
    > >
    > > On Tue, Feb 14 2023, Parav Pandit <parav@nvidia.com> wrote:
    > >
    > > > The control VQ of the virtio network device is used beyond advance
    > > > steering control. The control VQ dynamically changes multiple features
    > > > of the initialized device.
    > > >
    > > > Hence, update this area of control VQ introductory description at few
    > > > places and also place the link to its description.
    > > >
    > > > Also update the introduction section to better describe receive and
    > > > transmit virtqueues.
    > > >
    > > > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156
    > > > Reviewed-by: David Edmondson <david.edmondson@oracle.com>
    > > > Signed-off-by: Parav Pandit <parav@nvidia.com>
    > >
    > > Are you requesting a vote for this?
    >
    > I was not sure if its need voting as most are editorial cleanups, better description, structure movement.
    > Please advise if we need voting.


    The bylaws are here:
    https://github.com/oasis-tcs/virtio-spec/blob/master/README.md
    Minor cleanups, including editorial formatting changes, spelling and
    typo fixes can be committed directly into git for approval as part of
    the next specification approval ballot.

    This is way beyond that. And generally when in doubt vote.

    --
    MST




  • 8.  RE: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-16-2023 16:01

    > From: Michael S. Tsirkin <mst@redhat.com>
    > Sent: Thursday, February 16, 2023 10:50 AM
    >
    > On Thu, Feb 16, 2023 at 02:30:03PM +0000, Parav Pandit wrote:
    > >
    > > > From: Cornelia Huck <cohuck@redhat.com>
    > > > Sent: Thursday, February 16, 2023 9:10 AM
    > > >
    > > > On Tue, Feb 14 2023, Parav Pandit <parav@nvidia.com> wrote:
    > > >
    > > > > The control VQ of the virtio network device is used beyond advance
    > > > > steering control. The control VQ dynamically changes multiple
    > > > > features of the initialized device.
    > > > >
    > > > > Hence, update this area of control VQ introductory description at
    > > > > few places and also place the link to its description.
    > > > >
    > > > > Also update the introduction section to better describe receive
    > > > > and transmit virtqueues.
    > > > >
    > > > > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156
    > > > > Reviewed-by: David Edmondson <david.edmondson@oracle.com>
    > > > > Signed-off-by: Parav Pandit <parav@nvidia.com>
    > > >
    > > > Are you requesting a vote for this?
    > >
    > > I was not sure if its need voting as most are editorial cleanups, better
    > description, structure movement.
    > > Please advise if we need voting.
    >
    >
    > The bylaws are here:
    > https://github.com/oasis-tcs/virtio-spec/blob/master/README.md
    > Minor cleanups, including editorial formatting changes, spelling and typo fixes
    > can be committed directly into git for approval as part of the next specification
    > approval ballot.
    >
    > This is way beyond that. And generally when in doubt vote.

    Ok. Thanks for the direction.

    Can you please create the voting ballot?
    Git hub issue of interest is: Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156



  • 9.  RE: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-16-2023 16:01

    > From: Michael S. Tsirkin <mst@redhat.com>
    > Sent: Thursday, February 16, 2023 10:50 AM
    >
    > On Thu, Feb 16, 2023 at 02:30:03PM +0000, Parav Pandit wrote:
    > >
    > > > From: Cornelia Huck <cohuck@redhat.com>
    > > > Sent: Thursday, February 16, 2023 9:10 AM
    > > >
    > > > On Tue, Feb 14 2023, Parav Pandit <parav@nvidia.com> wrote:
    > > >
    > > > > The control VQ of the virtio network device is used beyond advance
    > > > > steering control. The control VQ dynamically changes multiple
    > > > > features of the initialized device.
    > > > >
    > > > > Hence, update this area of control VQ introductory description at
    > > > > few places and also place the link to its description.
    > > > >
    > > > > Also update the introduction section to better describe receive
    > > > > and transmit virtqueues.
    > > > >
    > > > > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156
    > > > > Reviewed-by: David Edmondson <david.edmondson@oracle.com>
    > > > > Signed-off-by: Parav Pandit <parav@nvidia.com>
    > > >
    > > > Are you requesting a vote for this?
    > >
    > > I was not sure if its need voting as most are editorial cleanups, better
    > description, structure movement.
    > > Please advise if we need voting.
    >
    >
    > The bylaws are here:
    > https://github.com/oasis-tcs/virtio-spec/blob/master/README.md
    > Minor cleanups, including editorial formatting changes, spelling and typo fixes
    > can be committed directly into git for approval as part of the next specification
    > approval ballot.
    >
    > This is way beyond that. And generally when in doubt vote.

    Ok. Thanks for the direction.

    Can you please create the voting ballot?
    Git hub issue of interest is: Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156



  • 10.  Re: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-16-2023 16:03
    On Thu, Feb 16 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:

    > On Thu, Feb 16, 2023 at 02:30:03PM +0000, Parav Pandit wrote:
    >>
    >> > From: Cornelia Huck <cohuck@redhat.com>
    >> > Sent: Thursday, February 16, 2023 9:10 AM
    >> >
    >> > On Tue, Feb 14 2023, Parav Pandit <parav@nvidia.com> wrote:
    >> >
    >> > > The control VQ of the virtio network device is used beyond advance
    >> > > steering control. The control VQ dynamically changes multiple features
    >> > > of the initialized device.
    >> > >
    >> > > Hence, update this area of control VQ introductory description at few
    >> > > places and also place the link to its description.
    >> > >
    >> > > Also update the introduction section to better describe receive and
    >> > > transmit virtqueues.
    >> > >
    >> > > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156
    >> > > Reviewed-by: David Edmondson <david.edmondson@oracle.com>
    >> > > Signed-off-by: Parav Pandit <parav@nvidia.com>
    >> >
    >> > Are you requesting a vote for this?
    >>
    >> I was not sure if its need voting as most are editorial cleanups, better description, structure movement.
    >> Please advise if we need voting.
    >
    >
    > The bylaws are here:
    > https://github.com/oasis-tcs/virtio-spec/blob/master/README.md
    > Minor cleanups, including editorial formatting changes, spelling and
    > typo fixes can be committed directly into git for approval as part of
    > the next specification approval ballot.
    >
    > This is way beyond that. And generally when in doubt vote.

    Agreed.

    (Btw: the github issue needs to be updated to point to the latest
    version of the patch, it still points to v1.)




  • 11.  Re: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-16-2023 16:03
    On Thu, Feb 16 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:

    > On Thu, Feb 16, 2023 at 02:30:03PM +0000, Parav Pandit wrote:
    >>
    >> > From: Cornelia Huck <cohuck@redhat.com>
    >> > Sent: Thursday, February 16, 2023 9:10 AM
    >> >
    >> > On Tue, Feb 14 2023, Parav Pandit <parav@nvidia.com> wrote:
    >> >
    >> > > The control VQ of the virtio network device is used beyond advance
    >> > > steering control. The control VQ dynamically changes multiple features
    >> > > of the initialized device.
    >> > >
    >> > > Hence, update this area of control VQ introductory description at few
    >> > > places and also place the link to its description.
    >> > >
    >> > > Also update the introduction section to better describe receive and
    >> > > transmit virtqueues.
    >> > >
    >> > > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156
    >> > > Reviewed-by: David Edmondson <david.edmondson@oracle.com>
    >> > > Signed-off-by: Parav Pandit <parav@nvidia.com>
    >> >
    >> > Are you requesting a vote for this?
    >>
    >> I was not sure if its need voting as most are editorial cleanups, better description, structure movement.
    >> Please advise if we need voting.
    >
    >
    > The bylaws are here:
    > https://github.com/oasis-tcs/virtio-spec/blob/master/README.md
    > Minor cleanups, including editorial formatting changes, spelling and
    > typo fixes can be committed directly into git for approval as part of
    > the next specification approval ballot.
    >
    > This is way beyond that. And generally when in doubt vote.

    Agreed.

    (Btw: the github issue needs to be updated to point to the latest
    version of the patch, it still points to v1.)




  • 12.  RE: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-16-2023 16:05

    > From: Cornelia Huck <cohuck@redhat.com>
    > Sent: Thursday, February 16, 2023 11:03 AM
    >
    > On Thu, Feb 16 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:
    >
    > > On Thu, Feb 16, 2023 at 02:30:03PM +0000, Parav Pandit wrote:
    > >>
    > >> > From: Cornelia Huck <cohuck@redhat.com>
    > >> > Sent: Thursday, February 16, 2023 9:10 AM
    > >> >
    > >> > On Tue, Feb 14 2023, Parav Pandit <parav@nvidia.com> wrote:
    > >> >
    > >> > > The control VQ of the virtio network device is used beyond
    > >> > > advance steering control. The control VQ dynamically changes
    > >> > > multiple features of the initialized device.
    > >> > >
    > >> > > Hence, update this area of control VQ introductory description at
    > >> > > few places and also place the link to its description.
    > >> > >
    > >> > > Also update the introduction section to better describe receive
    > >> > > and transmit virtqueues.
    > >> > >
    > >> > > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156
    > >> > > Reviewed-by: David Edmondson <david.edmondson@oracle.com>
    > >> > > Signed-off-by: Parav Pandit <parav@nvidia.com>
    > >> >
    > >> > Are you requesting a vote for this?
    > >>
    > >> I was not sure if its need voting as most are editorial cleanups, better
    > description, structure movement.
    > >> Please advise if we need voting.
    > >
    > >
    > > The bylaws are here:
    > > https://github.com/oasis-tcs/virtio-spec/blob/master/README.md
    > > Minor cleanups, including editorial formatting changes, spelling and
    > > typo fixes can be committed directly into git for approval as part of
    > > the next specification approval ballot.
    > >
    > > This is way beyond that. And generally when in doubt vote.
    >
    > Agreed.
    >
    > (Btw: the github issue needs to be updated to point to the latest version of the
    > patch, it still points to v1.)

    Done.



  • 13.  RE: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-16-2023 16:05

    > From: Cornelia Huck <cohuck@redhat.com>
    > Sent: Thursday, February 16, 2023 11:03 AM
    >
    > On Thu, Feb 16 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:
    >
    > > On Thu, Feb 16, 2023 at 02:30:03PM +0000, Parav Pandit wrote:
    > >>
    > >> > From: Cornelia Huck <cohuck@redhat.com>
    > >> > Sent: Thursday, February 16, 2023 9:10 AM
    > >> >
    > >> > On Tue, Feb 14 2023, Parav Pandit <parav@nvidia.com> wrote:
    > >> >
    > >> > > The control VQ of the virtio network device is used beyond
    > >> > > advance steering control. The control VQ dynamically changes
    > >> > > multiple features of the initialized device.
    > >> > >
    > >> > > Hence, update this area of control VQ introductory description at
    > >> > > few places and also place the link to its description.
    > >> > >
    > >> > > Also update the introduction section to better describe receive
    > >> > > and transmit virtqueues.
    > >> > >
    > >> > > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156
    > >> > > Reviewed-by: David Edmondson <david.edmondson@oracle.com>
    > >> > > Signed-off-by: Parav Pandit <parav@nvidia.com>
    > >> >
    > >> > Are you requesting a vote for this?
    > >>
    > >> I was not sure if its need voting as most are editorial cleanups, better
    > description, structure movement.
    > >> Please advise if we need voting.
    > >
    > >
    > > The bylaws are here:
    > > https://github.com/oasis-tcs/virtio-spec/blob/master/README.md
    > > Minor cleanups, including editorial formatting changes, spelling and
    > > typo fixes can be committed directly into git for approval as part of
    > > the next specification approval ballot.
    > >
    > > This is way beyond that. And generally when in doubt vote.
    >
    > Agreed.
    >
    > (Btw: the github issue needs to be updated to point to the latest version of the
    > patch, it still points to v1.)

    Done.



  • 14.  Re: [virtio-comment] RE: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-16-2023 16:30
    On Thu, Feb 16 2023, Parav Pandit <parav@nvidia.com> wrote:

    >> From: Cornelia Huck <cohuck@redhat.com>
    >> Sent: Thursday, February 16, 2023 11:03 AM
    >>
    >> On Thu, Feb 16 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:
    >>
    >> > On Thu, Feb 16, 2023 at 02:30:03PM +0000, Parav Pandit wrote:
    >> >>
    >> >> > From: Cornelia Huck <cohuck@redhat.com>
    >> >> > Sent: Thursday, February 16, 2023 9:10 AM
    >> >> >
    >> >> > On Tue, Feb 14 2023, Parav Pandit <parav@nvidia.com> wrote:
    >> >> >
    >> >> > > The control VQ of the virtio network device is used beyond
    >> >> > > advance steering control. The control VQ dynamically changes
    >> >> > > multiple features of the initialized device.
    >> >> > >
    >> >> > > Hence, update this area of control VQ introductory description at
    >> >> > > few places and also place the link to its description.
    >> >> > >
    >> >> > > Also update the introduction section to better describe receive
    >> >> > > and transmit virtqueues.
    >> >> > >
    >> >> > > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156
    >> >> > > Reviewed-by: David Edmondson <david.edmondson@oracle.com>
    >> >> > > Signed-off-by: Parav Pandit <parav@nvidia.com>
    >> >> >
    >> >> > Are you requesting a vote for this?
    >> >>
    >> >> I was not sure if its need voting as most are editorial cleanups, better
    >> description, structure movement.
    >> >> Please advise if we need voting.
    >> >
    >> >
    >> > The bylaws are here:
    >> > https://github.com/oasis-tcs/virtio-spec/blob/master/README.md
    >> > Minor cleanups, including editorial formatting changes, spelling and
    >> > typo fixes can be committed directly into git for approval as part of
    >> > the next specification approval ballot.
    >> >
    >> > This is way beyond that. And generally when in doubt vote.
    >>
    >> Agreed.
    >>
    >> (Btw: the github issue needs to be updated to point to the latest version of the
    >> patch, it still points to v1.)
    >
    > Done.

    Do you request to withdraw the ballot, so that we can start a new ballot
    with the proper URL?

    (Sorry about being a stickler for process here.)




  • 15.  Re: [virtio-comment] RE: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-16-2023 16:30
    On Thu, Feb 16 2023, Parav Pandit <parav@nvidia.com> wrote:

    >> From: Cornelia Huck <cohuck@redhat.com>
    >> Sent: Thursday, February 16, 2023 11:03 AM
    >>
    >> On Thu, Feb 16 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:
    >>
    >> > On Thu, Feb 16, 2023 at 02:30:03PM +0000, Parav Pandit wrote:
    >> >>
    >> >> > From: Cornelia Huck <cohuck@redhat.com>
    >> >> > Sent: Thursday, February 16, 2023 9:10 AM
    >> >> >
    >> >> > On Tue, Feb 14 2023, Parav Pandit <parav@nvidia.com> wrote:
    >> >> >
    >> >> > > The control VQ of the virtio network device is used beyond
    >> >> > > advance steering control. The control VQ dynamically changes
    >> >> > > multiple features of the initialized device.
    >> >> > >
    >> >> > > Hence, update this area of control VQ introductory description at
    >> >> > > few places and also place the link to its description.
    >> >> > >
    >> >> > > Also update the introduction section to better describe receive
    >> >> > > and transmit virtqueues.
    >> >> > >
    >> >> > > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156
    >> >> > > Reviewed-by: David Edmondson <david.edmondson@oracle.com>
    >> >> > > Signed-off-by: Parav Pandit <parav@nvidia.com>
    >> >> >
    >> >> > Are you requesting a vote for this?
    >> >>
    >> >> I was not sure if its need voting as most are editorial cleanups, better
    >> description, structure movement.
    >> >> Please advise if we need voting.
    >> >
    >> >
    >> > The bylaws are here:
    >> > https://github.com/oasis-tcs/virtio-spec/blob/master/README.md
    >> > Minor cleanups, including editorial formatting changes, spelling and
    >> > typo fixes can be committed directly into git for approval as part of
    >> > the next specification approval ballot.
    >> >
    >> > This is way beyond that. And generally when in doubt vote.
    >>
    >> Agreed.
    >>
    >> (Btw: the github issue needs to be updated to point to the latest version of the
    >> patch, it still points to v1.)
    >
    > Done.

    Do you request to withdraw the ballot, so that we can start a new ballot
    with the proper URL?

    (Sorry about being a stickler for process here.)




  • 16.  RE: [virtio-dev] Re: [virtio-comment] RE: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-17-2023 03:00

    > From: virtio-dev@lists.oasis-open.org <virtio-dev@lists.oasis-open.org> On
    > Behalf Of Cornelia Huck
    >
    > On Thu, Feb 16 2023, Parav Pandit <parav@nvidia.com> wrote:
    >
    > >> From: Cornelia Huck <cohuck@redhat.com>
    > >> Sent: Thursday, February 16, 2023 11:03 AM
    > >>
    > >> On Thu, Feb 16 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:
    > >>
    > >> > On Thu, Feb 16, 2023 at 02:30:03PM +0000, Parav Pandit wrote:
    > >> >>
    > >> >> > From: Cornelia Huck <cohuck@redhat.com>
    > >> >> > Sent: Thursday, February 16, 2023 9:10 AM
    > >> >> >
    > >> >> > On Tue, Feb 14 2023, Parav Pandit <parav@nvidia.com> wrote:
    > >> >> >
    > >> >> > > The control VQ of the virtio network device is used beyond
    > >> >> > > advance steering control. The control VQ dynamically changes
    > >> >> > > multiple features of the initialized device.
    > >> >> > >
    > >> >> > > Hence, update this area of control VQ introductory description
    > >> >> > > at few places and also place the link to its description.
    > >> >> > >
    > >> >> > > Also update the introduction section to better describe
    > >> >> > > receive and transmit virtqueues.
    > >> >> > >
    > >> >> > > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156
    > >> >> > > Reviewed-by: David Edmondson <david.edmondson@oracle.com>
    > >> >> > > Signed-off-by: Parav Pandit <parav@nvidia.com>
    > >> >> >
    > >> >> > Are you requesting a vote for this?
    > >> >>
    > >> >> I was not sure if its need voting as most are editorial cleanups,
    > >> >> better
    > >> description, structure movement.
    > >> >> Please advise if we need voting.
    > >> >
    > >> >
    > >> > The bylaws are here:
    > >> > https://github.com/oasis-tcs/virtio-spec/blob/master/README.md
    > >> > Minor cleanups, including editorial formatting changes, spelling
    > >> > and typo fixes can be committed directly into git for approval as
    > >> > part of the next specification approval ballot.
    > >> >
    > >> > This is way beyond that. And generally when in doubt vote.
    > >>
    > >> Agreed.
    > >>
    > >> (Btw: the github issue needs to be updated to point to the latest
    > >> version of the patch, it still points to v1.)
    > >
    > > Done.
    >
    > Do you request to withdraw the ballot, so that we can start a new ballot with
    > the proper URL?
    >
    > (Sorry about being a stickler for process here.)

    Yes, can you please withdraw the ballot [1] as it contains stale reference to the patch?

    [1] https://www.oasis-open.org/apps/org/workgroup/virtio/ballot.php?id=3758

    Also please create new ballot as the github link is now updated for the issue [2].
    [2] https://github.com/oasis-tcs/virtio-spec/issues/156




  • 17.  RE: [virtio-dev] Re: [virtio-comment] RE: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-17-2023 03:00

    > From: virtio-dev@lists.oasis-open.org <virtio-dev@lists.oasis-open.org> On
    > Behalf Of Cornelia Huck
    >
    > On Thu, Feb 16 2023, Parav Pandit <parav@nvidia.com> wrote:
    >
    > >> From: Cornelia Huck <cohuck@redhat.com>
    > >> Sent: Thursday, February 16, 2023 11:03 AM
    > >>
    > >> On Thu, Feb 16 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:
    > >>
    > >> > On Thu, Feb 16, 2023 at 02:30:03PM +0000, Parav Pandit wrote:
    > >> >>
    > >> >> > From: Cornelia Huck <cohuck@redhat.com>
    > >> >> > Sent: Thursday, February 16, 2023 9:10 AM
    > >> >> >
    > >> >> > On Tue, Feb 14 2023, Parav Pandit <parav@nvidia.com> wrote:
    > >> >> >
    > >> >> > > The control VQ of the virtio network device is used beyond
    > >> >> > > advance steering control. The control VQ dynamically changes
    > >> >> > > multiple features of the initialized device.
    > >> >> > >
    > >> >> > > Hence, update this area of control VQ introductory description
    > >> >> > > at few places and also place the link to its description.
    > >> >> > >
    > >> >> > > Also update the introduction section to better describe
    > >> >> > > receive and transmit virtqueues.
    > >> >> > >
    > >> >> > > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/156
    > >> >> > > Reviewed-by: David Edmondson <david.edmondson@oracle.com>
    > >> >> > > Signed-off-by: Parav Pandit <parav@nvidia.com>
    > >> >> >
    > >> >> > Are you requesting a vote for this?
    > >> >>
    > >> >> I was not sure if its need voting as most are editorial cleanups,
    > >> >> better
    > >> description, structure movement.
    > >> >> Please advise if we need voting.
    > >> >
    > >> >
    > >> > The bylaws are here:
    > >> > https://github.com/oasis-tcs/virtio-spec/blob/master/README.md
    > >> > Minor cleanups, including editorial formatting changes, spelling
    > >> > and typo fixes can be committed directly into git for approval as
    > >> > part of the next specification approval ballot.
    > >> >
    > >> > This is way beyond that. And generally when in doubt vote.
    > >>
    > >> Agreed.
    > >>
    > >> (Btw: the github issue needs to be updated to point to the latest
    > >> version of the patch, it still points to v1.)
    > >
    > > Done.
    >
    > Do you request to withdraw the ballot, so that we can start a new ballot with
    > the proper URL?
    >
    > (Sorry about being a stickler for process here.)

    Yes, can you please withdraw the ballot [1] as it contains stale reference to the patch?

    [1] https://www.oasis-open.org/apps/org/workgroup/virtio/ballot.php?id=3758

    Also please create new ballot as the github link is now updated for the issue [2].
    [2] https://github.com/oasis-tcs/virtio-spec/issues/156




  • 18.  RE: [virtio-dev] Re: [virtio-comment] RE: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-20-2023 13:34
    On Fri, Feb 17 2023, Parav Pandit <parav@nvidia.com> wrote:

    > Yes, can you please withdraw the ballot [1] as it contains stale reference to the patch?
    >
    > [1] https://www.oasis-open.org/apps/org/workgroup/virtio/ballot.php?id=3758
    >
    > Also please create new ballot as the github link is now updated for the issue [2].
    > [2] https://github.com/oasis-tcs/virtio-spec/issues/156

    Done.




  • 19.  RE: [virtio-dev] Re: [virtio-comment] RE: [PATCH v6] virtio-net: Improve introductory description

    Posted 02-20-2023 13:34
    On Fri, Feb 17 2023, Parav Pandit <parav@nvidia.com> wrote:

    > Yes, can you please withdraw the ballot [1] as it contains stale reference to the patch?
    >
    > [1] https://www.oasis-open.org/apps/org/workgroup/virtio/ballot.php?id=3758
    >
    > Also please create new ballot as the github link is now updated for the issue [2].
    > [2] https://github.com/oasis-tcs/virtio-spec/issues/156

    Done.