From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: virtio-dev-return-4733-cohuck=redhat.com@lists.oasis-open.org 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 D9BC5985C80 for ; Tue, 24 Jul 2018 12:43:57 +0000 (UTC) Date: Tue, 24 Jul 2018 14:43:51 +0200 From: Cornelia Huck Message-ID: <20180724144351.5f8dd113.cohuck@redhat.com> In-Reply-To: <1532365355-45245-1-git-send-email-sridhar.samudrala@intel.com> References: <1532365355-45245-1-git-send-email-sridhar.samudrala@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: [virtio-dev] Re: [PATCH v2] content: Introduce VIRTIO_NET_F_STANDBY feature To: Sridhar Samudrala Cc: mst@redhat.com, virtio-dev@lists.oasis-open.org List-ID: On Mon, 23 Jul 2018 10:02:35 -0700 Sridhar Samudrala wrote: > VIRTIO_NET_F_STANDBY feature enables hypervisor to indicate virtio_net > device to act as a standby for a primary device with the same MAC address. > > Signed-off-by: Sridhar Samudrala --- > content.tex | 8 ++++++++ > 1 file changed, 8 insertions(+) > > v2: updated standby description based on Cornelia's feedback. > > diff --git a/content.tex b/content.tex > index be18234..b729857 100644 > --- a/content.tex > +++ b/content.tex > @@ -2525,6 +2525,9 @@ features. > > \item[VIRTIO_NET_F_CTRL_MAC_ADDR(23)] Set MAC address through control > channel. > + > +\item[VIRTIO_NET_F_STANDBY(62)] Device MAY act as a standby for a primary > + device with the same MAC I don't think you should use MAY etc. outside a normative section, so s/MAY/may/ > \end{description} > > \subsubsection{Feature bit requirements}\label{sec:Device Types / Network Device / Feature bits / Feature bit requirements} > @@ -2636,6 +2639,11 @@ If the driver negotiates VIRTIO_NET_F_MTU, it MUST NOT transmit packets of > size exceeding the value of \field{mtu} (plus low level ethernet header length) > with \field{gso_type} NONE or ECN. > > +A driver SHOULD negotiate VIRTIO_NET_F_STANDBY feature if the device offers it. s/VIRTIO_NET_F_STANDBY feature/the VIRTIO_NET_F_STANDBY feature/ > + > +If the driver negotiates VIRTIO_NET_F_STANDBY, the device MAY act as a standby > +device for a primary device with the same MAC address. I think the first statement needs to go into a driver normative section, while the second needs to go into a device normative section. > + > \subsubsection{Legacy Interface: Device configuration layout}\label{sec:Device Types / Network Device / Device configuration layout / Legacy Interface: Device configuration layout} > \label{sec:Device Types / Block Device / Feature bits / Device configuration layout / Legacy Interface: Device configuration layout} > When using the legacy interface, transitional devices and drivers I still think we need a more detailed description of how this is supposed to work elsewhere (i.e., outside of the normative section). But we can probably merge an updated version of this patch to get at least the feature bit reserved and documented. Thoughts? --------------------------------------------------------------------- To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org