Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

5.1.6.2 Packet Transmission; clarify "output descriptor" #165

Open
gstew5 opened this issue Feb 27, 2023 · 0 comments
Open

5.1.6.2 Packet Transmission; clarify "output descriptor" #165

gstew5 opened this issue Feb 27, 2023 · 0 comments

Comments

@gstew5
Copy link

gstew5 commented Feb 27, 2023

https://lists.oasis-open.org/archives/virtio-comment/202302/msg00640.html

Section 5.1.6.2 Packet Transmission, subsection 5, states that:

“The header and packet are added as one output descriptor to the transmitq, and the device is notified of the new entry (see 5.1.5 Device Initialization).

The following diff clarifies that the header and packet can be chained across multiple descriptors, in accordance with Section 2.6.4 Message Framing:

From: Gordon Stewart <[gordon@bedrocksystems.com](mailto:gordon@bedrocksystems.com)>
Date: Mon, 27 Feb 2023 13:59:45 -0500
Subject: [PATCH] 5.1.6.2 Packet Transmission; clarify "output descriptor"

---
 device-types/net/description.tex | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/device-types/net/description.tex b/device-types/net/description.tex
index 0500bb6..98f0f20 100644
--- a/device-types/net/description.tex
+++ b/device-types/net/description.tex
@@ -487,8 +487,8 @@ \subsubsection{Packet Transmission}\label{sec:Device Types / Network Device / De
 
 \item \field{num_buffers} is set to zero.  This field is unused on transmitted packets.
 
-\item The header and packet are added as one output descriptor to the
-  transmitq, and the device is notified of the new entry
+\item The header and packet output descriptors are added as one buffer to
+  the transmitq, and the device is notified of the new entry
   (see \ref{sec:Device Types / Network Device / Device Initialization}~\nameref{sec:Device Types / Network Device / Device Initialization}).
 \end{enumerate}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant