OASIS Mailing List ArchivesView the OASIS mailing list archive below
or browse/search using MarkMail.

 


Help: OASIS Mailing Lists Help | MarkMail Help

virtio-comment message

[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]


Subject: Re: [PATCH v6] virtio-net: Improve introductory description


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.)



[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]