RE: [virtio-comment] [PATCH v8 5/8] transport-ccw: Rename queue depth/size to other transports

[David Edmondson <david.edmondson@oracle.com>]

Parav Pandit <parav@nvidia.com> writes:

>> From: David Edmondson <david.edmondson@oracle.com>
>> Sent: Tuesday, March 28, 2023 10:07 AM
>> > --- a/transport-ccw.tex
>> > +++ b/transport-ccw.tex
>> > @@ -237,12 +237,12 @@ \subsubsection{Configuring a
>> > Virtqueue}\label{sec:Virtio Transport Options / Vir
>> > \begin{lstlisting}  struct vq_config_block {
>> >          be16 index;
>> > -        be16 max_num;
>> > +        be16 max_queue_size; /* Previously named as max_num */
>> 
>> "named as" doesn't feel like typical language use - you can remove "as", though
>> I'd be inclined to use some kind of decoration for the name:
>> 
>> /* Previously named 'max_num'. */
>> 
>> >  };
>> >  \end{lstlisting}
>
> I really don't see any value in putting this change log material in the spec section,
> But anyways, Michael finds its useful for some users,
>
> So how about, 
>
> /* Previously known as max_num. */

Seems fine.
-- 
And now I know what every step is for.

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/