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 v13] virtio-net: support the virtqueue coalescing moderation



> From: Cornelia Huck <cohuck@redhat.com>
> Sent: Wednesday, March 22, 2023 11:21 AM
> 
> On Wed, Mar 22 2023, Heng Qi <hengqi@linux.alibaba.com> wrote:
> 
> > +The driver MUST NOT set \field{vqn} to any value other than an enabled
> transmit or receive virtqueue number.
> 
Why do you suggest a negative statement here?
How is it better than,
The driver MUST set ...

The device will anyway have to check and apply the parameter to the right virtqueue.
And if the vq is not enabled or vq is not tx or rx vq, it needs to fail the command.

> "than the virtqueue number of an enabled transmit or receive virtqueue"
> 
> might be better -- what do others think?
> 
> Otherwise, LGTM.



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