* [PATCH 1/1] virtio-blk: fix virtqueues accounting
@ 2021-07-21 12:45 Max Gurtovoy
2021-07-22 10:20 ` [virtio-comment] " Cornelia Huck
0 siblings, 1 reply; 3+ messages in thread
From: Max Gurtovoy @ 2021-07-21 12:45 UTC (permalink / raw)
To: mst, virtio-comment, virtio-dev; +Cc: cohuck, oren, Max Gurtovoy
Virtqueue index is zero based, thus virtqueue (N-1) refers to requestqN.
Signed-off-by: Max Gurtovoy <mgurtovoy@nvidia.com>
---
content.tex | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/content.tex b/content.tex
index 5c70a3c..c266fd5 100644
--- a/content.tex
+++ b/content.tex
@@ -4398,7 +4398,7 @@ \subsection{Virtqueues}\label{sec:Device Types / Block Device / Virtqueues}
\begin{description}
\item[0] requestq1
\item[\ldots]
-\item[N] requestqN
+\item[N-1] requestqN
\end{description}
N=1 if VIRTIO_BLK_F_MQ is not negotiated, otherwise N is set by
--
2.21.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [virtio-comment] Re: [PATCH 1/1] virtio-blk: fix virtqueues accounting
2021-07-21 12:45 [PATCH 1/1] virtio-blk: fix virtqueues accounting Max Gurtovoy
@ 2021-07-22 10:20 ` Cornelia Huck
2021-08-10 6:39 ` Cornelia Huck
0 siblings, 1 reply; 3+ messages in thread
From: Cornelia Huck @ 2021-07-22 10:20 UTC (permalink / raw)
To: Max Gurtovoy, mst, virtio-comment, virtio-dev; +Cc: oren, Max Gurtovoy
On Wed, Jul 21 2021, Max Gurtovoy <mgurtovoy@nvidia.com> wrote:
> Virtqueue index is zero based, thus virtqueue (N-1) refers to requestqN.
>
> Signed-off-by: Max Gurtovoy <mgurtovoy@nvidia.com>
> ---
> content.tex | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
We had an issue for a similar fix for virtio-net, so do we want to use
issue+voting here as well, or would that be overkill? I'm inclined to do
this as an editorial update.
This publicly archived list offers a means to provide input to the
OASIS Virtual I/O Device (VIRTIO) TC.
In order to verify user consent to the Feedback License terms and
to minimize spam in the list archive, subscription is required
before posting.
Subscribe: virtio-comment-subscribe@lists.oasis-open.org
Unsubscribe: virtio-comment-unsubscribe@lists.oasis-open.org
List help: virtio-comment-help@lists.oasis-open.org
List archive: https://lists.oasis-open.org/archives/virtio-comment/
Feedback License: https://www.oasis-open.org/who/ipr/feedback_license.pdf
List Guidelines: https://www.oasis-open.org/policies-guidelines/mailing-lists
Committee: https://www.oasis-open.org/committees/virtio/
Join OASIS: https://www.oasis-open.org/join/
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [virtio-comment] Re: [PATCH 1/1] virtio-blk: fix virtqueues accounting
2021-07-22 10:20 ` [virtio-comment] " Cornelia Huck
@ 2021-08-10 6:39 ` Cornelia Huck
0 siblings, 0 replies; 3+ messages in thread
From: Cornelia Huck @ 2021-08-10 6:39 UTC (permalink / raw)
To: Max Gurtovoy, mst, virtio-comment, virtio-dev; +Cc: oren, Max Gurtovoy
On Thu, Jul 22 2021, Cornelia Huck <cohuck@redhat.com> wrote:
> On Wed, Jul 21 2021, Max Gurtovoy <mgurtovoy@nvidia.com> wrote:
>
>> Virtqueue index is zero based, thus virtqueue (N-1) refers to requestqN.
>>
>> Signed-off-by: Max Gurtovoy <mgurtovoy@nvidia.com>
>> ---
>> content.tex | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> Reviewed-by: Cornelia Huck <cohuck@redhat.com>
>
> We had an issue for a similar fix for virtio-net, so do we want to use
> issue+voting here as well, or would that be overkill? I'm inclined to do
> this as an editorial update.
I just went ahead and pushed it as an editorial update. Thanks.
This publicly archived list offers a means to provide input to the
OASIS Virtual I/O Device (VIRTIO) TC.
In order to verify user consent to the Feedback License terms and
to minimize spam in the list archive, subscription is required
before posting.
Subscribe: virtio-comment-subscribe@lists.oasis-open.org
Unsubscribe: virtio-comment-unsubscribe@lists.oasis-open.org
List help: virtio-comment-help@lists.oasis-open.org
List archive: https://lists.oasis-open.org/archives/virtio-comment/
Feedback License: https://www.oasis-open.org/who/ipr/feedback_license.pdf
List Guidelines: https://www.oasis-open.org/policies-guidelines/mailing-lists
Committee: https://www.oasis-open.org/committees/virtio/
Join OASIS: https://www.oasis-open.org/join/
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-08-10 6:39 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-21 12:45 [PATCH 1/1] virtio-blk: fix virtqueues accounting Max Gurtovoy
2021-07-22 10:20 ` [virtio-comment] " Cornelia Huck
2021-08-10 6:39 ` 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.