* [virtio-dev] [PATCH] split-ring: Fix typo in used ring description
@ 2018-05-22 19:11 Christoffer Dall
2018-05-22 20:09 ` [virtio-dev] " Laszlo Ersek
2018-05-23 10:03 ` Stefan Hajnoczi
0 siblings, 2 replies; 3+ messages in thread
From: Christoffer Dall @ 2018-05-22 19:11 UTC (permalink / raw)
To: virtio-dev; +Cc: felipe, stefanha, pbonzini, mst, lersek, Christoffer Dall
Only the device updates the used ring.
This looks like a typo as a result of copy-paste from the description of
the Available Ring.
Fixes: https://github.com/oasis-tcs/virtio-spec/issues/12
Signed-off-by: Christoffer Dall <christoffer.dall@arm.com>
---
split-ring.tex | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/split-ring.tex b/split-ring.tex
index df278fe..2267863 100644
--- a/split-ring.tex
+++ b/split-ring.tex
@@ -417,7 +417,7 @@ overwritten the bytes which were in that buffer, this could leak the
contents of freed memory from other processes to the application.
\end{note}
-\field{idx} field indicates where the driver would put the next descriptor
+\field{idx} field indicates where the device would put the next descriptor
entry in the ring (modulo the queue size). This starts at 0, and increases.
\begin{note}
--
2.15.1 (Apple Git-101)
---------------------------------------------------------------------
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 related [flat|nested] 3+ messages in thread
* [virtio-dev] Re: [PATCH] split-ring: Fix typo in used ring description
2018-05-22 19:11 [virtio-dev] [PATCH] split-ring: Fix typo in used ring description Christoffer Dall
@ 2018-05-22 20:09 ` Laszlo Ersek
2018-05-23 10:03 ` Stefan Hajnoczi
1 sibling, 0 replies; 3+ messages in thread
From: Laszlo Ersek @ 2018-05-22 20:09 UTC (permalink / raw)
To: Christoffer Dall, virtio-dev; +Cc: felipe, stefanha, pbonzini, mst
On 05/22/18 21:11, Christoffer Dall wrote:
> Only the device updates the used ring.
>
> This looks like a typo as a result of copy-paste from the description of
> the Available Ring.
>
> Fixes: https://github.com/oasis-tcs/virtio-spec/issues/12
> Signed-off-by: Christoffer Dall <christoffer.dall@arm.com>
> ---
> split-ring.tex | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/split-ring.tex b/split-ring.tex
> index df278fe..2267863 100644
> --- a/split-ring.tex
> +++ b/split-ring.tex
> @@ -417,7 +417,7 @@ overwritten the bytes which were in that buffer, this could leak the
> contents of freed memory from other processes to the application.
> \end{note}
>
> -\field{idx} field indicates where the driver would put the next descriptor
> +\field{idx} field indicates where the device would put the next descriptor
> entry in the ring (modulo the queue size). This starts at 0, and increases.
>
> \begin{note}
>
Good point about this likely being a copypasta error; when I first tried
to look at the context of your patch in "split-ring.tex", I found the
*correct* instance (currently on line 308) and got confused. But, the
line numbers in the hunk header above are OK; on line 420, "driver" is
wrong.
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Thanks
Laszlo
---------------------------------------------------------------------
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] 3+ messages in thread
* [virtio-dev] Re: [PATCH] split-ring: Fix typo in used ring description
2018-05-22 19:11 [virtio-dev] [PATCH] split-ring: Fix typo in used ring description Christoffer Dall
2018-05-22 20:09 ` [virtio-dev] " Laszlo Ersek
@ 2018-05-23 10:03 ` Stefan Hajnoczi
1 sibling, 0 replies; 3+ messages in thread
From: Stefan Hajnoczi @ 2018-05-23 10:03 UTC (permalink / raw)
To: Christoffer Dall; +Cc: virtio-dev, felipe, pbonzini, mst, lersek
[-- Attachment #1: Type: text/plain, Size: 476 bytes --]
On Tue, May 22, 2018 at 09:11:05PM +0200, Christoffer Dall wrote:
> Only the device updates the used ring.
>
> This looks like a typo as a result of copy-paste from the description of
> the Available Ring.
>
> Fixes: https://github.com/oasis-tcs/virtio-spec/issues/12
> Signed-off-by: Christoffer Dall <christoffer.dall@arm.com>
> ---
> split-ring.tex | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 455 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-05-23 10:04 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-22 19:11 [virtio-dev] [PATCH] split-ring: Fix typo in used ring description Christoffer Dall
2018-05-22 20:09 ` [virtio-dev] " Laszlo Ersek
2018-05-23 10:03 ` Stefan Hajnoczi
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.