* [PATCH] content: Remove duplicate paragraph
@ 2021-09-30 12:58 Jean-Philippe Brucker
2021-10-04 9:11 ` [virtio-dev] " Stefan Hajnoczi
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Jean-Philippe Brucker @ 2021-09-30 12:58 UTC (permalink / raw)
To: virtio-dev; +Cc: cohuck, mst, Jean-Philippe Brucker
It looks like commit 356aeeb40d7a ("content: add vendor specific cfg
type") had a merge issue. It includes the device normative paragraph for
Shared memory capability, which was already added right above it by
commit 855ad7af2bd6 ("shared memory: Define PCI capability"). Remove the
duplicate paragraph.
Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
---
This applies onto "Fix copy/paste bug in PCI transport paragraph", that
fixes both paragraph titles.
https://lists.oasis-open.org/archives/virtio-comment/202108/msg00097.html
---
content.tex | 8 --------
1 file changed, 8 deletions(-)
diff --git a/content.tex b/content.tex
index 4418ac8..54f7441 100644
--- a/content.tex
+++ b/content.tex
@@ -1149,14 +1149,6 @@ \subsubsection{Shared memory capability}\label{sec:Virtio Transport Options / Vi
The \field{cap.id} MUST be unique for any one device instance.
-\devicenormative{\paragraph}{Shared memory capability}{Virtio Transport Options / Virtio Over PCI Bus / PCI Device Layout / Shared memory capability}
-
-The region defined by the combination of the \field {cap.offset},
-\field {cap.offset_hi}, and \field {cap.length}, \field
-{cap.length_hi} fields MUST be contained within the declared bar.
-
-The \field{cap.id} MUST be unique for any one device instance.
-
\subsubsection{Vendor data capability}\label{sec:Virtio
Transport Options / Virtio Over PCI Bus / PCI Device Layout /
Vendor data capability}
--
2.33.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [virtio-dev] [PATCH] content: Remove duplicate paragraph
2021-09-30 12:58 [PATCH] content: Remove duplicate paragraph Jean-Philippe Brucker
@ 2021-10-04 9:11 ` Stefan Hajnoczi
2021-10-04 9:17 ` Michael S. Tsirkin
2021-10-04 10:56 ` [virtio-dev] " Cornelia Huck
2 siblings, 0 replies; 5+ messages in thread
From: Stefan Hajnoczi @ 2021-10-04 9:11 UTC (permalink / raw)
To: Jean-Philippe Brucker; +Cc: virtio-dev, cohuck, mst
[-- Attachment #1: Type: text/plain, Size: 778 bytes --]
On Thu, Sep 30, 2021 at 01:58:48PM +0100, Jean-Philippe Brucker wrote:
> It looks like commit 356aeeb40d7a ("content: add vendor specific cfg
> type") had a merge issue. It includes the device normative paragraph for
> Shared memory capability, which was already added right above it by
> commit 855ad7af2bd6 ("shared memory: Define PCI capability"). Remove the
> duplicate paragraph.
>
> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
> ---
> This applies onto "Fix copy/paste bug in PCI transport paragraph", that
> fixes both paragraph titles.
> https://lists.oasis-open.org/archives/virtio-comment/202108/msg00097.html
> ---
> content.tex | 8 --------
> 1 file changed, 8 deletions(-)
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] content: Remove duplicate paragraph
2021-09-30 12:58 [PATCH] content: Remove duplicate paragraph Jean-Philippe Brucker
2021-10-04 9:11 ` [virtio-dev] " Stefan Hajnoczi
@ 2021-10-04 9:17 ` Michael S. Tsirkin
2021-10-04 10:56 ` [virtio-dev] " Cornelia Huck
2 siblings, 0 replies; 5+ messages in thread
From: Michael S. Tsirkin @ 2021-10-04 9:17 UTC (permalink / raw)
To: Jean-Philippe Brucker; +Cc: virtio-dev, cohuck
On Thu, Sep 30, 2021 at 01:58:48PM +0100, Jean-Philippe Brucker wrote:
> It looks like commit 356aeeb40d7a ("content: add vendor specific cfg
> type") had a merge issue. It includes the device normative paragraph for
> Shared memory capability, which was already added right above it by
> commit 855ad7af2bd6 ("shared memory: Define PCI capability"). Remove the
> duplicate paragraph.
>
> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Cornelia, I think this can go in under the trivial fixes rule.
> ---
> This applies onto "Fix copy/paste bug in PCI transport paragraph", that
> fixes both paragraph titles.
> https://lists.oasis-open.org/archives/virtio-comment/202108/msg00097.html
> ---
> content.tex | 8 --------
> 1 file changed, 8 deletions(-)
>
> diff --git a/content.tex b/content.tex
> index 4418ac8..54f7441 100644
> --- a/content.tex
> +++ b/content.tex
> @@ -1149,14 +1149,6 @@ \subsubsection{Shared memory capability}\label{sec:Virtio Transport Options / Vi
>
> The \field{cap.id} MUST be unique for any one device instance.
>
> -\devicenormative{\paragraph}{Shared memory capability}{Virtio Transport Options / Virtio Over PCI Bus / PCI Device Layout / Shared memory capability}
> -
> -The region defined by the combination of the \field {cap.offset},
> -\field {cap.offset_hi}, and \field {cap.length}, \field
> -{cap.length_hi} fields MUST be contained within the declared bar.
> -
> -The \field{cap.id} MUST be unique for any one device instance.
> -
> \subsubsection{Vendor data capability}\label{sec:Virtio
> Transport Options / Virtio Over PCI Bus / PCI Device Layout /
> Vendor data capability}
> --
> 2.33.0
^ permalink raw reply [flat|nested] 5+ messages in thread
* [virtio-dev] Re: [PATCH] content: Remove duplicate paragraph
2021-09-30 12:58 [PATCH] content: Remove duplicate paragraph Jean-Philippe Brucker
2021-10-04 9:11 ` [virtio-dev] " Stefan Hajnoczi
2021-10-04 9:17 ` Michael S. Tsirkin
@ 2021-10-04 10:56 ` Cornelia Huck
2021-10-06 11:24 ` Jean-Philippe Brucker
2 siblings, 1 reply; 5+ messages in thread
From: Cornelia Huck @ 2021-10-04 10:56 UTC (permalink / raw)
To: Jean-Philippe Brucker, virtio-dev; +Cc: mst, Jean-Philippe Brucker
On Thu, Sep 30 2021, Jean-Philippe Brucker <jean-philippe@linaro.org> wrote:
> It looks like commit 356aeeb40d7a ("content: add vendor specific cfg
> type") had a merge issue. It includes the device normative paragraph for
> Shared memory capability, which was already added right above it by
> commit 855ad7af2bd6 ("shared memory: Define PCI capability"). Remove the
> duplicate paragraph.
>
> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
> ---
> This applies onto "Fix copy/paste bug in PCI transport paragraph", that
> fixes both paragraph titles.
> https://lists.oasis-open.org/archives/virtio-comment/202108/msg00097.html
> ---
> content.tex | 8 --------
> 1 file changed, 8 deletions(-)
>
> diff --git a/content.tex b/content.tex
> index 4418ac8..54f7441 100644
> --- a/content.tex
> +++ b/content.tex
> @@ -1149,14 +1149,6 @@ \subsubsection{Shared memory capability}\label{sec:Virtio Transport Options / Vi
>
> The \field{cap.id} MUST be unique for any one device instance.
>
> -\devicenormative{\paragraph}{Shared memory capability}{Virtio Transport Options / Virtio Over PCI Bus / PCI Device Layout / Shared memory capability}
> -
> -The region defined by the combination of the \field {cap.offset},
> -\field {cap.offset_hi}, and \field {cap.length}, \field
> -{cap.length_hi} fields MUST be contained within the declared bar.
> -
> -The \field{cap.id} MUST be unique for any one device instance.
> -
> \subsubsection{Vendor data capability}\label{sec:Virtio
> Transport Options / Virtio Over PCI Bus / PCI Device Layout /
> Vendor data capability}
Hm. This statement is indeed included twice, but the two are not exactly
the same. For the first paragraph, do we want to use a combination of
the two? IOW, something like
"The region defined by the combination of the \field {cap.offset},
\field {cap.offset_hi}, and \field {cap.length}, \field
{cap.length_hi} fields MUST be contained within the BAR specified by
\field{cap.bar}."
---------------------------------------------------------------------
To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org
For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] content: Remove duplicate paragraph
2021-10-04 10:56 ` [virtio-dev] " Cornelia Huck
@ 2021-10-06 11:24 ` Jean-Philippe Brucker
0 siblings, 0 replies; 5+ messages in thread
From: Jean-Philippe Brucker @ 2021-10-06 11:24 UTC (permalink / raw)
To: Cornelia Huck; +Cc: virtio-dev, mst
On Mon, Oct 04, 2021 at 12:56:48PM +0200, Cornelia Huck wrote:
> On Thu, Sep 30 2021, Jean-Philippe Brucker <jean-philippe@linaro.org> wrote:
>
> > It looks like commit 356aeeb40d7a ("content: add vendor specific cfg
> > type") had a merge issue. It includes the device normative paragraph for
> > Shared memory capability, which was already added right above it by
> > commit 855ad7af2bd6 ("shared memory: Define PCI capability"). Remove the
> > duplicate paragraph.
> >
> > Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
> > ---
> > This applies onto "Fix copy/paste bug in PCI transport paragraph", that
> > fixes both paragraph titles.
> > https://lists.oasis-open.org/archives/virtio-comment/202108/msg00097.html
> > ---
> > content.tex | 8 --------
> > 1 file changed, 8 deletions(-)
> >
> > diff --git a/content.tex b/content.tex
> > index 4418ac8..54f7441 100644
> > --- a/content.tex
> > +++ b/content.tex
> > @@ -1149,14 +1149,6 @@ \subsubsection{Shared memory capability}\label{sec:Virtio Transport Options / Vi
> >
> > The \field{cap.id} MUST be unique for any one device instance.
> >
> > -\devicenormative{\paragraph}{Shared memory capability}{Virtio Transport Options / Virtio Over PCI Bus / PCI Device Layout / Shared memory capability}
> > -
> > -The region defined by the combination of the \field {cap.offset},
> > -\field {cap.offset_hi}, and \field {cap.length}, \field
> > -{cap.length_hi} fields MUST be contained within the declared bar.
> > -
> > -The \field{cap.id} MUST be unique for any one device instance.
> > -
> > \subsubsection{Vendor data capability}\label{sec:Virtio
> > Transport Options / Virtio Over PCI Bus / PCI Device Layout /
> > Vendor data capability}
>
> Hm. This statement is indeed included twice, but the two are not exactly
> the same. For the first paragraph, do we want to use a combination of
> the two? IOW, something like
>
> "The region defined by the combination of the \field {cap.offset},
> \field {cap.offset_hi}, and \field {cap.length}, \field
> {cap.length_hi} fields MUST be contained within the BAR specified by
> \field{cap.bar}."
>
Makes sense, although thinking more about this it was the first paragraph
that was correct, because it refers to:
struct virtio_pci_cap64 {
struct virtio_pci_cap {
...
le32 offset;
le32 length;
} cap;
u32 offset_hi;
u32 length_hi;
}
So it would be \field{cap.offset} and \field{offset_hi}.
I'll send a v2
Thanks,
Jean
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-10-06 11:24 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-30 12:58 [PATCH] content: Remove duplicate paragraph Jean-Philippe Brucker
2021-10-04 9:11 ` [virtio-dev] " Stefan Hajnoczi
2021-10-04 9:17 ` Michael S. Tsirkin
2021-10-04 10:56 ` [virtio-dev] " Cornelia Huck
2021-10-06 11:24 ` Jean-Philippe Brucker
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.