From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Sender: List-Post: List-Help: List-Unsubscribe: List-Subscribe: Received: from lists.oasis-open.org (oasis-open.org [10.110.1.242]) by lists.oasis-open.org (Postfix) with ESMTP id 6403B986272 for ; Mon, 9 Jan 2023 07:17:24 +0000 (UTC) Date: Mon, 9 Jan 2023 02:17:16 -0500 From: "Michael S. Tsirkin" Message-ID: <20230109021550-mutt-send-email-mst@kernel.org> References: <20230109012747.444663-1-parav@nvidia.com> MIME-Version: 1.0 In-Reply-To: <20230109012747.444663-1-parav@nvidia.com> Subject: [virtio-comment] Re: [PATCH v7] virtio-network: Clarify VLAN filter table configuration Content-Type: text/plain; charset=us-ascii Content-Disposition: inline To: Parav Pandit Cc: virtio-dev@lists.oasis-open.org, virtio-comment@lists.oasis-open.org, si-wei.liu@oracle.com List-ID: On Mon, Jan 09, 2023 at 03:27:47AM +0200, Parav Pandit wrote: > The filtering behavior of the VLAN filter commands is not very clear as > discussed in thread [1]. > > Hence, add the command description and device requirements for it. > > [1] https://www.mail-archive.com/qemu-devel@nongnu.org/msg912392.html > Fixes: https://github.com/oasis-tcs/virtio-spec/issues/147 > Suggested-by: Si-Wei Liu > Signed-off-by: Parav Pandit Another grammar correction. Otherwise ok. > --- > changelog: > v6->v7: > - Moved VLAN filter table description from requirements to device > descrption section > - Added MUST and SHOULD to device requirements > v5->v6: > - removed unwanted article > v4->v5: > - reworded 'vlan filtering table' to 'vlan filter table' to match > to the existing description about vlan filtering > - remove confusing text around VLAN_DEL command description > - added missing article > - reword device match configuration to device configuration > - changed 'found' to 'present' and 'not found' to 'absent' to > consider vlan filter table as config table rather > than search table > v3->v4: > - added description for accepting vlan tagged packets when vlan > filter is not negotiated > v2->v3: > - corrected grammar > - simplified description for untagged packets > v1->v2: > - adapt to new file path > v0->v1: > - added missing conformance section link > --- > .../virtio-network/device-conformance.tex | 1 + > device-types/virtio-network/device.tex | 22 ++++++++++++++++++- > 2 files changed, 22 insertions(+), 1 deletion(-) > > diff --git a/device-types/virtio-network/device-conformance.tex b/device-types/virtio-network/device-conformance.tex > index c686377..54f6783 100644 > --- a/device-types/virtio-network/device-conformance.tex > +++ b/device-types/virtio-network/device-conformance.tex > @@ -9,6 +9,7 @@ > \item \ref{devicenormative:Device Types / Network Device / Device Operation / Processing of Incoming Packets} > \item \ref{devicenormative:Device Types / Network Device / Device Operation / Control Virtqueue / Packet Receive Filtering} > \item \ref{devicenormative:Device Types / Network Device / Device Operation / Control Virtqueue / Setting MAC Address Filtering} > +\item \ref{devicenormative:Device Types / Network Device / Device Operation / Control Virtqueue / VLAN Filtering} > \item \ref{devicenormative:Device Types / Network Device / Device Operation / Control Virtqueue / Gratuitous Packet Sending} > \item \ref{devicenormative:Device Types / Network Device / Device Operation / Control Virtqueue / Automatic receive steering in multiqueue mode} > \item \ref{devicenormative:Device Types / Network Device / Device Operation / Control Virtqueue / Receive-side scaling (RSS) / RSS processing} > diff --git a/device-types/virtio-network/device.tex b/device-types/virtio-network/device.tex > index e7cdd6f..bff586e 100644 > --- a/device-types/virtio-network/device.tex > +++ b/device-types/virtio-network/device.tex > @@ -1194,7 +1194,11 @@ \subsubsection{Control Virtqueue}\label{sec:Device Types / Network Device / Devi > \paragraph{VLAN Filtering}\label{sec:Device Types / Network Device / Device Operation / Control Virtqueue / VLAN Filtering} > > If the driver negotiates the VIRTIO_NET_F_CTRL_VLAN feature, it > -can control a VLAN filter table in the device. > +can control a VLAN filter table in the device. The VLAN filter > +table applies only to VLAN tagged packets. > + > +When VIRTIO_NET_F_CTRL_VLAN is negotiated, the device starts with > +an empty VLAN filter table. > > \begin{note} > Similar to the MAC address based filtering, the VLAN filtering > @@ -1210,6 +1214,22 @@ \subsubsection{Control Virtqueue}\label{sec:Device Types / Network Device / Devi > Both the VIRTIO_NET_CTRL_VLAN_ADD and VIRTIO_NET_CTRL_VLAN_DEL > command take a little-endian 16-bit VLAN id as the command-specific-data. > > +VIRTIO_NET_CTRL_VLAN_ADD command adds the specified VLAN to the > +VLAN filter table. > + > +VIRTIO_NET_CTRL_VLAN_DEL command removes the specified VLAN from > +the VLAN filter table. > + > +\devicenormative{\subparagraph}{VLAN Filtering}{Device Types / Network Device / Device Operation / Control Virtqueue / VLAN Filtering} > + > +When VIRTIO_NET_F_CTRL_VLAN is not negotiated, the device MUST > +accepts all VLAN tagged packets as per the device configuration. MUST accept > + > +When VIRTIO_NET_F_CTRL_VLAN is negotiated, the device MUST > +accepts MUST accept > all VLAN tagged packets whose VLAN tag is present in > +the VLAN filter table and the device SHOULD drop all VLAN tagged and SHOULD drop (I think it's clear it's the device from context). > +packets whose VLAN tag is absent in the VLAN filter table. > + > \subparagraph{Legacy Interface: VLAN Filtering}\label{sec:Device Types / Network Device / Device Operation / Control Virtqueue / VLAN Filtering / Legacy Interface: VLAN Filtering} > When using the legacy interface, transitional devices and drivers > MUST format the VLAN id > -- > 2.26.2 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/