* [PATCH v2] content: Remove duplicate paragraph
@ 2021-10-06 11:28 Jean-Philippe Brucker
2021-10-07 8:41 ` [virtio-dev] " Stefan Hajnoczi
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Jean-Philippe Brucker @ 2021-10-06 11:28 UTC (permalink / raw)
To: virtio-dev; +Cc: cohuck, mst, stefanha, 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").
The two paragraphs differ, and the first paragraph is correct. It refers
to struct virtio_pci_cap64 which embeds struct virtio_pci_cap:
struct virtio_pci_cap64 {
struct virtio_pci_cap {
...
le32 offset;
le32 length;
} cap;
u32 offset_hi;
u32 length_hi;
}
Merge the two paragraph while keeping the best of both. Drop the spaces
after \field to stay consistent with the rest of the document.
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
---
v1: https://www.mail-archive.com/virtio-dev@lists.oasis-open.org/msg07815.html
---
content.tex | 15 ++++-----------
1 file changed, 4 insertions(+), 11 deletions(-)
diff --git a/content.tex b/content.tex
index 288e62f..39fe12b 100644
--- a/content.tex
+++ b/content.tex
@@ -1143,17 +1143,10 @@ \subsubsection{Shared memory capability}\label{sec:Virtio Transport Options / Vi
\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 \field{cap.offset},
-\field{offset_hi}, and \field {cap.length}, \field{length_hi}
-MUST be contained within the BAR specified by \field{cap.bar}.
-
-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 region defined by the combination of the \field{cap.offset},
+\field{offset_hi}, and \field{cap.length}, \field{length_hi}
+fields MUST be contained within the BAR specified by
+\field{cap.bar}.
The \field{cap.id} MUST be unique for any one device instance.
--
2.33.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [virtio-dev] [PATCH v2] content: Remove duplicate paragraph
2021-10-06 11:28 [PATCH v2] content: Remove duplicate paragraph Jean-Philippe Brucker
@ 2021-10-07 8:41 ` Stefan Hajnoczi
2021-10-07 8:43 ` [virtio-dev] " Cornelia Huck
2021-10-07 9:00 ` Cornelia Huck
2 siblings, 0 replies; 4+ messages in thread
From: Stefan Hajnoczi @ 2021-10-07 8:41 UTC (permalink / raw)
To: Jean-Philippe Brucker; +Cc: virtio-dev, cohuck, mst
[-- Attachment #1: Type: text/plain, Size: 1195 bytes --]
On Wed, Oct 06, 2021 at 12:28:19PM +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").
>
> The two paragraphs differ, and the first paragraph is correct. It refers
> to struct virtio_pci_cap64 which embeds struct virtio_pci_cap:
>
> struct virtio_pci_cap64 {
> struct virtio_pci_cap {
> ...
> le32 offset;
> le32 length;
> } cap;
> u32 offset_hi;
> u32 length_hi;
> }
>
> Merge the two paragraph while keeping the best of both. Drop the spaces
> after \field to stay consistent with the rest of the document.
>
> Acked-by: Michael S. Tsirkin <mst@redhat.com>
> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
> ---
> v1: https://www.mail-archive.com/virtio-dev@lists.oasis-open.org/msg07815.html
> ---
> content.tex | 15 ++++-----------
> 1 file changed, 4 insertions(+), 11 deletions(-)
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* [virtio-dev] Re: [PATCH v2] content: Remove duplicate paragraph
2021-10-06 11:28 [PATCH v2] content: Remove duplicate paragraph Jean-Philippe Brucker
2021-10-07 8:41 ` [virtio-dev] " Stefan Hajnoczi
@ 2021-10-07 8:43 ` Cornelia Huck
2021-10-07 9:00 ` Cornelia Huck
2 siblings, 0 replies; 4+ messages in thread
From: Cornelia Huck @ 2021-10-07 8:43 UTC (permalink / raw)
To: Jean-Philippe Brucker, virtio-dev; +Cc: mst, stefanha, Jean-Philippe Brucker
On Wed, Oct 06 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").
>
> The two paragraphs differ, and the first paragraph is correct. It refers
> to struct virtio_pci_cap64 which embeds struct virtio_pci_cap:
>
> struct virtio_pci_cap64 {
> struct virtio_pci_cap {
> ...
> le32 offset;
> le32 length;
> } cap;
> u32 offset_hi;
> u32 length_hi;
> }
>
> Merge the two paragraph while keeping the best of both. Drop the spaces
> after \field to stay consistent with the rest of the document.
>
> Acked-by: Michael S. Tsirkin <mst@redhat.com>
> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
> ---
> v1: https://www.mail-archive.com/virtio-dev@lists.oasis-open.org/msg07815.html
> ---
> content.tex | 15 ++++-----------
> 1 file changed, 4 insertions(+), 11 deletions(-)
>
> diff --git a/content.tex b/content.tex
> index 288e62f..39fe12b 100644
> --- a/content.tex
> +++ b/content.tex
> @@ -1143,17 +1143,10 @@ \subsubsection{Shared memory capability}\label{sec:Virtio Transport Options / Vi
>
> \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 \field{cap.offset},
> -\field{offset_hi}, and \field {cap.length}, \field{length_hi}
> -MUST be contained within the BAR specified by \field{cap.bar}.
> -
> -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 region defined by the combination of the \field{cap.offset},
> +\field{offset_hi}, and \field{cap.length}, \field{length_hi}
> +fields MUST be contained within the BAR specified by
> +\field{cap.bar}.
>
> The \field{cap.id} MUST be unique for any one device instance.
>
LGTM; I think this is still within the scope of editorial updates.
---------------------------------------------------------------------
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] 4+ messages in thread
* [virtio-dev] Re: [PATCH v2] content: Remove duplicate paragraph
2021-10-06 11:28 [PATCH v2] content: Remove duplicate paragraph Jean-Philippe Brucker
2021-10-07 8:41 ` [virtio-dev] " Stefan Hajnoczi
2021-10-07 8:43 ` [virtio-dev] " Cornelia Huck
@ 2021-10-07 9:00 ` Cornelia Huck
2 siblings, 0 replies; 4+ messages in thread
From: Cornelia Huck @ 2021-10-07 9:00 UTC (permalink / raw)
To: Jean-Philippe Brucker, virtio-dev; +Cc: mst, stefanha, Jean-Philippe Brucker
On Wed, Oct 06 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").
>
> The two paragraphs differ, and the first paragraph is correct. It refers
> to struct virtio_pci_cap64 which embeds struct virtio_pci_cap:
>
> struct virtio_pci_cap64 {
> struct virtio_pci_cap {
> ...
> le32 offset;
> le32 length;
> } cap;
> u32 offset_hi;
> u32 length_hi;
> }
>
> Merge the two paragraph while keeping the best of both. Drop the spaces
> after \field to stay consistent with the rest of the document.
>
> Acked-by: Michael S. Tsirkin <mst@redhat.com>
> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
> ---
> v1: https://www.mail-archive.com/virtio-dev@lists.oasis-open.org/msg07815.html
> ---
> content.tex | 15 ++++-----------
> 1 file changed, 4 insertions(+), 11 deletions(-)
Thanks, pushed as an editorial update.
[While looking at this, I noticed that the normative statements for the
vendor data capability are not at the correct level. I'll do a patch to
fix that.]
---------------------------------------------------------------------
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] 4+ messages in thread
end of thread, other threads:[~2021-10-07 9:00 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-06 11:28 [PATCH v2] content: Remove duplicate paragraph Jean-Philippe Brucker
2021-10-07 8:41 ` [virtio-dev] " Stefan Hajnoczi
2021-10-07 8:43 ` [virtio-dev] " Cornelia Huck
2021-10-07 9:00 ` Cornelia Huck
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.