virtio-comment

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

diff --git a/content.tex b/content.tex
index df5ce16..874714f 100644
--- a/content.tex
+++ b/content.tex
@@ -396,7 +396,7 @@ \subsubsection{Virtqueue Re-enable}\label{sec:Basic Facilities of a Virtio Devic

\input{packed-ring.tex}

-\section{Driver Notifications} \label{sec:Virtqueues / Driver notifications}
+\section{Driver Notifications} \label{sec:Basic Facilities of a Virtio Device / Driver notifications}
The driver is sometimes required to send an available buffer
notification to the device.

@@ -1624,7 +1624,7 @@ \subsubsection{Available Buffer Notifications}\label{sec:Virtio Transport Option
the following 32-bit value to the Queue Notify address:
\lstinputlisting{notifications-le.c}

-See \ref{sec:Virtqueues / Driver notifications}~\nameref{sec:Virtqueues / Driver notifications}
+See \ref{sec:Basic Facilities of a Virtio Device / Driver notifications}~\nameref{sec:Basic Facilities of a Virtio Device / Driver notifications}
for the definition of the components.

See \ref{sec:Virtio Transport Options / Virtio Over PCI Bus / PCI Device Layout / Notification capability}
@@ -1879,7 +1879,7 @@ \subsection{MMIO Device Register Layout}\label{sec:Virtio Transport Options / Vi

\lstinputlisting{notifications-le.c}

- See \ref{sec:Virtqueues / Driver notifications}~\nameref{sec:Virtqueues / Driver notifications}
+ See \ref{sec:Basic Facilities of a Virtio Device / Driver notifications}~\nameref{sec:Basic Facilities of a Virtio Device / Driver notifications}
for the definition of the components.
}
\hline
@@ -2123,7 +2123,7 @@ \subsubsection{Available Buffer Notifications}\label{sec:Virtio Transport Option
the following 32-bit value to \field{QueueNotify}:
\lstinputlisting{notifications-le.c}

-See \ref{sec:Virtqueues / Driver notifications}~\nameref{sec:Virtqueues / Driver notifications}
+See \ref{sec:Basic Facilities of a Virtio Device / Driver notifications}~\nameref{sec:Basic Facilities of a Virtio Device / Driver notifications}
for the definition of the components.

\subsubsection{Notifications From The Device}\label{sec:Virtio Transport Options / Virtio Over MMIO / MMIO-specific Initialization And Device Operation / Notifications From The Device}
@@ -2845,7 +2845,7 @@ \subsubsection{Guest->Host Notification}\label{sec:Virtio Transport Options / Vi
the value has the following format:
\lstinputlisting{notifications-be.c}

-See \ref{sec:Virtqueues / Driver notifications}~\nameref{sec:Virtqueues / Driver notifications}
+See \ref{sec:Basic Facilities of a Virtio Device / Driver notifications}~\nameref{sec:Basic Facilities of a Virtio Device / Driver notifications}
for the definition of the components.

\devicenormative{\paragraph}{Guest->Host Notification}{Virtio Transport Options / Virtio over channel I/O / Device Operation / Guest->Host Notification}
@@ -3091,12 +3091,12 @@ \chapter{Reserved Feature Bits}\label{sec:Reserved Feature Bits}
\item[VIRTIO_F_NOTIFICATION_DATA(38)] This feature indicates
that the driver passes extra data (besides identifying the virtqueue)
in its device notifications.
- See \ref{sec:Virtqueues / Driver notifications}~\nameref{sec:Virtqueues / Driver notifications}.
+ See \ref{sec:Basic Facilities of a Virtio Device / Driver notifications}~\nameref{sec:Basic Facilities of a Virtio Device / Driver notifications}.

\item[VIRTIO_F_NOTIF_CONFIG_DATA(39)] This feature indicates that the driver
uses the data provided by the device as a virtqueue identifier in available
buffer notifications.
- As mentioned in section \ref{sec:Virtqueues / Driver notifications}, when the
+ As mentioned in section \ref{sec:Basic Facilities of a Virtio Device / Driver notifications}, when the
driver is required to send an available buffer notification to the device, it
sends the virtqueue number to be notified. The method of delivering
notifications is transport specific.
--
2.26.2

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?

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?

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

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

On Fri, Feb 10, 2023 at 12:23:14PM +0100, Cornelia Huck wrote:
> 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?

Acked-by: Michael S. Tsirkin <mst@redhat.com>

Minor fix does not need a vote.
Cornelia are you applying this?

--
MST

On Fri, Feb 10, 2023 at 12:23:14PM +0100, Cornelia Huck wrote:
> 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?

Acked-by: Michael S. Tsirkin <mst@redhat.com>

Minor fix does not need a vote.
Cornelia are you applying this?

--
MST

On Wed, Feb 15 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:

> On Fri, Feb 10, 2023 at 12:23:14PM +0100, Cornelia Huck wrote:
>> 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?
>
> Acked-by: Michael S. Tsirkin <mst@redhat.com>
>
> Minor fix does not need a vote.
> Cornelia are you applying this?

Pushed.

On Wed, Feb 15 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:

> On Fri, Feb 10, 2023 at 12:23:14PM +0100, Cornelia Huck wrote:
>> 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?
>
> Acked-by: Michael S. Tsirkin <mst@redhat.com>
>
> Minor fix does not need a vote.
> Cornelia are you applying this?

Pushed.

On Wed, Feb 15, 2023 at 01:19:25PM +0100, Cornelia Huck wrote:
> On Wed, Feb 15 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:
>
> > On Fri, Feb 10, 2023 at 12:23:14PM +0100, Cornelia Huck wrote:
> >> 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?
> >
> > Acked-by: Michael S. Tsirkin <mst@redhat.com>
> >
> > Minor fix does not need a vote.
> > Cornelia are you applying this?
>
> Pushed.

btw did you try to generate a diff pdf? when changing labels it is
sometimes necessary to keep old ones around too as diff
needs old and new links to work.

--
MST

On Wed, Feb 15, 2023 at 01:19:25PM +0100, Cornelia Huck wrote:
> On Wed, Feb 15 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:
>
> > On Fri, Feb 10, 2023 at 12:23:14PM +0100, Cornelia Huck wrote:
> >> 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?
> >
> > Acked-by: Michael S. Tsirkin <mst@redhat.com>
> >
> > Minor fix does not need a vote.
> > Cornelia are you applying this?
>
> Pushed.

btw did you try to generate a diff pdf? when changing labels it is
sometimes necessary to keep old ones around too as diff
needs old and new links to work.

--
MST

On Wed, Feb 15 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:

> On Wed, Feb 15, 2023 at 01:19:25PM +0100, Cornelia Huck wrote:
>> On Wed, Feb 15 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:
>>
>> > On Fri, Feb 10, 2023 at 12:23:14PM +0100, Cornelia Huck wrote:
>> >> 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?
>> >
>> > Acked-by: Michael S. Tsirkin <mst@redhat.com>
>> >
>> > Minor fix does not need a vote.
>> > Cornelia are you applying this?
>>
>> Pushed.
>
> btw did you try to generate a diff pdf? when changing labels it is
> sometimes necessary to keep old ones around too as diff
> needs old and new links to work.

I'm at least not seeing obvious errors.

On Wed, Feb 15 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:

> On Wed, Feb 15, 2023 at 01:19:25PM +0100, Cornelia Huck wrote:
>> On Wed, Feb 15 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:
>>
>> > On Fri, Feb 10, 2023 at 12:23:14PM +0100, Cornelia Huck wrote:
>> >> 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?
>> >
>> > Acked-by: Michael S. Tsirkin <mst@redhat.com>
>> >
>> > Minor fix does not need a vote.
>> > Cornelia are you applying this?
>>
>> Pushed.
>
> btw did you try to generate a diff pdf? when changing labels it is
> sometimes necessary to keep old ones around too as diff
> needs old and new links to work.

I'm at least not seeing obvious errors.

> From: Michael S. Tsirkin <mst@redhat.com>
> Sent: Wednesday, February 15, 2023 8:00 AM
>
> btw did you try to generate a diff pdf? when changing labels it is sometimes
> necessary to keep old ones around too as diff needs old and new links to work.

I generated the new pdf before pushing the patch.
I couldn't generate the diff pdf using ./makediffpdf.sh

I get error on running makediff.sh virtio.tex

Warning: command \@chapapp containing @ found in
Warning: virtio.tex.
Warning: consider using --makeatletter if the result is not compilable.
Warning: command \@chapapp containing @ found in
Warning: ./main.tex.
Warning: consider using --makeatletter if the result is not compilable.
Submodule path 'latexdiff' not initialized
Maybe you want to use 'update --init'?
make: Entering directory '/images/parav/upstream/virtio-spec/latexdiff'
make: *** No rule to make target 'mkdirs'. Stop.
make: Leaving directory '/images/parav/upstream/virtio-spec/latexdiff'
./makediff.sh: line 51: ./latexdiff-fast: No such file or directory

> From: Michael S. Tsirkin <mst@redhat.com>
> Sent: Wednesday, February 15, 2023 8:00 AM
>
> btw did you try to generate a diff pdf? when changing labels it is sometimes
> necessary to keep old ones around too as diff needs old and new links to work.

I generated the new pdf before pushing the patch.
I couldn't generate the diff pdf using ./makediffpdf.sh

I get error on running makediff.sh virtio.tex

Warning: command \@chapapp containing @ found in
Warning: virtio.tex.
Warning: consider using --makeatletter if the result is not compilable.
Warning: command \@chapapp containing @ found in
Warning: ./main.tex.
Warning: consider using --makeatletter if the result is not compilable.
Submodule path 'latexdiff' not initialized
Maybe you want to use 'update --init'?
make: Entering directory '/images/parav/upstream/virtio-spec/latexdiff'
make: *** No rule to make target 'mkdirs'. Stop.
make: Leaving directory '/images/parav/upstream/virtio-spec/latexdiff'
./makediff.sh: line 51: ./latexdiff-fast: No such file or directory

On Wed, Feb 15, 2023 at 09:53:55PM +0000, Parav Pandit wrote:
>
> > From: Michael S. Tsirkin <mst@redhat.com>
> > Sent: Wednesday, February 15, 2023 8:00 AM
> >
> > btw did you try to generate a diff pdf? when changing labels it is sometimes
> > necessary to keep old ones around too as diff needs old and new links to work.
>
> I generated the new pdf before pushing the patch.
> I couldn't generate the diff pdf using ./makediffpdf.sh
>
> I get error on running makediff.sh virtio.tex
>
> Warning: command \@chapapp containing @ found in
> Warning: virtio.tex.
> Warning: consider using --makeatletter if the result is not compilable.
> Warning: command \@chapapp containing @ found in
> Warning: ./main.tex.
> Warning: consider using --makeatletter if the result is not compilable.
> Submodule path 'latexdiff' not initialized
> Maybe you want to use 'update --init'?
> make: Entering directory '/images/parav/upstream/virtio-spec/latexdiff'
> make: *** No rule to make target 'mkdirs'. Stop.
> make: Leaving directory '/images/parav/upstream/virtio-spec/latexdiff'
> ./makediff.sh: line 51: ./latexdiff-fast: No such file or directory


Hmm. does this help:


diff --git a/makediff.sh b/makediff.sh
index c925c5f..f65a2b4 100755
--- a/makediff.sh
+++ b/makediff.sh
@@ -44,7 +44,7 @@ latexpand $MAIN -o flat.tex
cd "${cur}"
#wget http://mirror.math.ku.edu/tex-archive/support/latexdiff/latexdiff-fast
#chmod +x latexdiff-fast
-git submodule update latexdiff
+git submodule update --init latexdiff
make -C latexdiff mkdirs dist/latexdiff-fast
ln -fs ./latexdiff/dist/latexdiff-fast ./latexdiff-fast
./latexdiff-fast \

On Wed, Feb 15, 2023 at 09:53:55PM +0000, Parav Pandit wrote:
>
> > From: Michael S. Tsirkin <mst@redhat.com>
> > Sent: Wednesday, February 15, 2023 8:00 AM
> >
> > btw did you try to generate a diff pdf? when changing labels it is sometimes
> > necessary to keep old ones around too as diff needs old and new links to work.
>
> I generated the new pdf before pushing the patch.
> I couldn't generate the diff pdf using ./makediffpdf.sh
>
> I get error on running makediff.sh virtio.tex
>
> Warning: command \@chapapp containing @ found in
> Warning: virtio.tex.
> Warning: consider using --makeatletter if the result is not compilable.
> Warning: command \@chapapp containing @ found in
> Warning: ./main.tex.
> Warning: consider using --makeatletter if the result is not compilable.
> Submodule path 'latexdiff' not initialized
> Maybe you want to use 'update --init'?
> make: Entering directory '/images/parav/upstream/virtio-spec/latexdiff'
> make: *** No rule to make target 'mkdirs'. Stop.
> make: Leaving directory '/images/parav/upstream/virtio-spec/latexdiff'
> ./makediff.sh: line 51: ./latexdiff-fast: No such file or directory


Hmm. does this help:


diff --git a/makediff.sh b/makediff.sh
index c925c5f..f65a2b4 100755
--- a/makediff.sh
+++ b/makediff.sh
@@ -44,7 +44,7 @@ latexpand $MAIN -o flat.tex
cd "${cur}"
#wget http://mirror.math.ku.edu/tex-archive/support/latexdiff/latexdiff-fast
#chmod +x latexdiff-fast
-git submodule update latexdiff
+git submodule update --init latexdiff
make -C latexdiff mkdirs dist/latexdiff-fast
ln -fs ./latexdiff/dist/latexdiff-fast ./latexdiff-fast
./latexdiff-fast \

> From: Michael S. Tsirkin <mst@redhat.com>
> Sent: Wednesday, February 15, 2023 7:20 PM
>
> On Wed, Feb 15, 2023 at 09:53:55PM +0000, Parav Pandit wrote:
> >
> > > From: Michael S. Tsirkin <mst@redhat.com>
> > > Sent: Wednesday, February 15, 2023 8:00 AM
> > >
> > > btw did you try to generate a diff pdf? when changing labels it is
> > > sometimes necessary to keep old ones around too as diff needs old and
> new links to work.
> >
> > I generated the new pdf before pushing the patch.
> > I couldn't generate the diff pdf using ./makediffpdf.sh
> >
> > I get error on running makediff.sh virtio.tex
> >
> > Warning: command \@chapapp containing @ found in
> > Warning: virtio.tex.
> > Warning: consider using --makeatletter if the result is not compilable.
> > Warning: command \@chapapp containing @ found in
> > Warning: ./main.tex.
> > Warning: consider using --makeatletter if the result is not compilable.
> > Submodule path 'latexdiff' not initialized Maybe you want to use
> > 'update --init'?
> > make: Entering directory '/images/parav/upstream/virtio-spec/latexdiff'
> > make: *** No rule to make target 'mkdirs'. Stop.
> > make: Leaving directory '/images/parav/upstream/virtio-spec/latexdiff'
> > ./makediff.sh: line 51: ./latexdiff-fast: No such file or directory
>
>
> Hmm. does this help:
>
Works great. Thank you.
Was able to find out fixing the change log.
I posted the fix up patch at [1].

Please review.
[1] https://lists.oasis-open.org/archives/virtio-dev/202302/msg00331.html

> From: Michael S. Tsirkin <mst@redhat.com>
> Sent: Wednesday, February 15, 2023 7:20 PM
>
> On Wed, Feb 15, 2023 at 09:53:55PM +0000, Parav Pandit wrote:
> >
> > > From: Michael S. Tsirkin <mst@redhat.com>
> > > Sent: Wednesday, February 15, 2023 8:00 AM
> > >
> > > btw did you try to generate a diff pdf? when changing labels it is
> > > sometimes necessary to keep old ones around too as diff needs old and
> new links to work.
> >
> > I generated the new pdf before pushing the patch.
> > I couldn't generate the diff pdf using ./makediffpdf.sh
> >
> > I get error on running makediff.sh virtio.tex
> >
> > Warning: command \@chapapp containing @ found in
> > Warning: virtio.tex.
> > Warning: consider using --makeatletter if the result is not compilable.
> > Warning: command \@chapapp containing @ found in
> > Warning: ./main.tex.
> > Warning: consider using --makeatletter if the result is not compilable.
> > Submodule path 'latexdiff' not initialized Maybe you want to use
> > 'update --init'?
> > make: Entering directory '/images/parav/upstream/virtio-spec/latexdiff'
> > make: *** No rule to make target 'mkdirs'. Stop.
> > make: Leaving directory '/images/parav/upstream/virtio-spec/latexdiff'
> > ./makediff.sh: line 51: ./latexdiff-fast: No such file or directory
>
>
> Hmm. does this help:
>
Works great. Thank you.
Was able to find out fixing the change log.
I posted the fix up patch at [1].

Please review.
[1] https://lists.oasis-open.org/archives/virtio-dev/202302/msg00331.html