> From: Parav Pandit
> Sent: Friday, February 3, 2023 8:46 AM
>
> Hi Michael,
>
> > From: Parav Pandit <[email protected]>
> > Sent: Monday, January 23, 2023 12:07 PM
> > To: [email protected]; [email protected];
> > [email protected]; [email protected]; [email protected]
> > Cc: [email protected]; Shahaf Shuler
> > <[email protected]>; Parav Pandit <[email protected]>
> > Subject: [PATCH v6] virtio-net: Avoid confusion between a card and a
> > device
> >
> > Historically virtio network device is documented as an Ethernet card.
> > A modern card in the industry has one to multiple ports, one to
> > multiple PCI functions. However the virtio network device is usually
> > just a single link/port network interface controller.
> >
> > Hence, avoid this confusing term 'card' and align the specification to
> > adhere to widely used specification term as 'device' used for all virtio
> > device
> types.
> >
> > Replaced 'card' with 'network interface controller'.
> >
> > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/154
> > Signed-off-by: Parav Pandit <[email protected]>
> > ---
> > This patch is on top of change [1].
> >
> > [1]
> > https://lists.oasis-open.org/archives/virtio-dev/202301/msg00210.html
>
> This is due many days now.
> I have also addressed the comment of Halil.
> Other patches depend on this change, and they need to progress too.
> This was reviewed by Jason Wang before. I dropped the Jason's reviewed-by tag
> due to one change in the wording.
>
> Can you please initiate the voting?
Michael,
Can you please initiate voting?
>
> > ---
> > changelog:
> > v5->v6:
> > - fixed 'consist' to 'consists'
> > v4->v5:
> > - replaced 'network card' with 'device' in the device initialization
> > section description
> > - net device checksum feature bit description updated
> > - rename card to network interface controller
> > v3->v4:
> > - rebased to new directory name
> > v2->v3:
> > - rebased to new file name
> > v1->v2:
> > - adapt to new file path
> > v0->v1:
> > - removed device word twice
> > - corrected grammar for changing from card to device
> > ---
> > content.tex | 6 +++---
> > device-types/net/description.tex | 22 +++++++++++-----------
> > 2 files changed, 14 insertions(+), 14 deletions(-)
> >
> > diff --git a/content.tex b/content.tex index 0118b8a..df5ce16 100644
> > --- a/content.tex
> > +++ b/content.tex
> > @@ -619,7 +619,7 @@ \subsection{PCI Device Discovery}\label{sec:Virtio
> > Transport Options / Virtio Ov \hline
> > Transitional PCI Device ID & Virtio Device \\
> > \hline \hline
> > -0x1000 & network card \\
> > +0x1000 & network device \\
> > \hline
> > 0x1001 & block device \\
> > \hline
> > @@ -635,7 +635,7 @@ \subsection{PCI Device Discovery}\label{sec:Virtio
> > Transport Options / Virtio Ov \hline \end{tabular}
> >
> > -For example, the network card device with the Virtio Device ID 1
> > +For example, the network device with the Virtio Device ID 1
> > has the PCI Device ID 0x1041 or the Transitional PCI Device ID 0x1000.
> >
> > The PCI Subsystem Vendor ID and the PCI Subsystem Device ID MAY
> > reflect @@ -2912,7 +2912,7 @@ \chapter{Device Types}\label{sec:Device
> > Types} \hline \hline
> > 0 & reserved (invalid) \\
> > \hline
> > -1 & network card \\
> > +1 & network device \\
> > \hline
> > 2 & block device \\
> > \hline
> > diff --git a/device-types/net/description.tex
> > b/device-types/net/description.tex
> > index d3f9b52..e1341e6 100644
> > --- a/device-types/net/description.tex
> > +++ b/device-types/net/description.tex
> > @@ -1,13 +1,14 @@
> > \section{Network Device}\label{sec:Device Types / Network Device}
> >
> > -The virtio network device is a virtual ethernet card, and is the
> > -most complex of the devices supported so far by virtio. It has
> > -enhanced rapidly and demonstrates clearly how support for new
> > -features are added to an existing device. Empty buffers are -placed
> > in one virtqueue for receiving packets, and outgoing -packets are enqueued
> into another for transmission in that order.
> > -A third command queue is used to control advanced filtering -features.
> > +The virtio network device is a virtual network interface controller.
> > +It consists of a virtual Ethernet link which connects the device to
> > +the Ethernet network. It is the most complex of the devices supported
> > +so far by virtio. It has enhanced rapidly and demonstrates clearly
> > +how support for new features are added to an existing device. Empty
> > +buffers are placed in one virtqueue for receiving packets, and
> > +outgoing packets are enqueued into another for transmission in that
> > +order. A third command queue is used to control advanced filtering
> features.
> >
> > \subsection{Device ID}\label{sec:Device Types / Network Device /
> > Device ID}
> >
> > @@ -32,8 +33,7 @@ \subsection{Virtqueues}\label{sec:Device Types /
> > Network Device / Virtqueues} \subsection{Feature
> > bits}\label{sec:Device Types / Network Device / Feature bits}
> >
> > \begin{description}
> > -\item[VIRTIO_NET_F_CSUM (0)] Device handles packets with partial
> > checksum. This
> > - ``checksum offload'' is a common feature on modern network cards.
> > +\item[VIRTIO_NET_F_CSUM (0)] Device handles packets with partial
> > +checksum
> > offload.
> >
> > \item[VIRTIO_NET_F_GUEST_CSUM (1)] Driver handles packets with
> > partial checksum.
> >
> > @@ -323,7 +323,7 @@ \subsection{Device
> > Initialization}\label{sec:Device Types / Network Device / Dev
> >
> > \item If the VIRTIO_NET_F_MAC feature bit is set, the configuration
> > space \field{mac} entry indicates the ``physical'' address of the
> > - network card, otherwise the driver would typically generate a
> > random
> > + device, otherwise the driver would typically generate a random
> > local MAC address.
> >
> > \item If the VIRTIO_NET_F_STATUS feature bit is negotiated, the link
> > --
> > 2.26.2
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
