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] content.tex Fix Driver notifications label


> From: Cornelia Huck <cohuck@redhat.com>
> Sent: Friday, February 10, 2023 6:23 AM
> To: Parav Pandit <parav@nvidia.com>; mst@redhat.com; virtio-
> dev@lists.oasis-open.org
> Cc: virtio-comment@lists.oasis-open.org; Shahaf Shuler
> <shahafs@nvidia.com>; Parav Pandit <parav@nvidia.com>
> Subject: Re: [PATCH] content.tex Fix Driver notifications label
> 
> On Fri, Feb 03 2023, Parav Pandit <parav@nvidia.com> wrote:
> 
> > Driver notifications section is under "Basic Facilities of a Virtio
> > Device". However, the label is placed under "Virtqueues" section.
> >
> > Fix the label references.
> >
> > Signed-off-by: Parav Pandit <parav@nvidia.com>
> > ---
> >  content.tex | 14 +++++++-------
> >  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> Not sure if it was originally intended to sit unter the virtqueues section (it was
> lifted up one level as it disappeared under the packet ring section), might have
> been.
> 
> It probably makes sense to not try and move it back into the virtqueues section,
> and instead do your change. Other opinions?

If there is none, can you please proceed to complete the review?
I assume this editorial fix doesn't need github issue.

Once this fix is in place, I would like to bring Notifications and driver Notifications sections close to each other as it has overlapping parts. 


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