[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]
Subject: RE: [virtio-dev] [PATCH v3] virtio-net: Mention VIRTIO_NET_F_HASH_REPORT dependency on VIRTIO_NET_F_CTRL_VQ
> From: virtio-dev@lists.oasis-open.org <virtio-dev@lists.oasis-open.org> On > Behalf Of Alvaro Karsz > --- > v2: Use SHOULD instead of Feature bit requirement, version 1.2 is already > out and doesn't include this depencency. > > v3: Explain the dependency in a less confusing way. > > Signed-off-by: Alvaro Karsz <alvaro.karsz@solid-run.com> > > device-types/net/description.tex | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/device-types/net/description.tex b/device-types/net/description.tex > index 700a1cb..1741c79 100644 > --- a/device-types/net/description.tex > +++ b/device-types/net/description.tex > @@ -256,6 +256,9 @@ \subsection{Device configuration > layout}\label{sec:Device Types / Network Device \field{duplex} fields as long as > VIRTIO_NET_S_LINK_UP is set in the \field{status}. > > +The device SHOULD NOT offer VIRTIO_NET_F_HASH_REPORT if it does not > +offer VIRTIO_NET_F_CTRL_VQ. > + I believe this applies to all the configurations done through the control VQ. So we should write a generic description instead of per feature statement like above. > \drivernormative{\subsubsection}{Device configuration layout}{Device Types / > Network Device / Device configuration layout} > > A driver SHOULD negotiate VIRTIO_NET_F_MAC if the device offers it. > @@ -289,6 +292,9 @@ \subsection{Device configuration > layout}\label{sec:Device Types / Network Device SHOULD re-read \field{speed} > and \field{duplex} after a configuration change notification. > > +A driver SHOULD NOT negotiate VIRTIO_NET_F_HASH_REPORT if it does not > +negotiate VIRTIO_NET_F_CTRL_VQ. > + Same for the driver too. Like below. The driver should not negotiate a feature that requires to use a control VQ when VIRTIO_NET_F_CTRL_VQ is not negotiated. Or driver must not .. .. Exception to that is VIRTIO_NET_F_HASH_REPORT, which driver should avoid negotiation ...
[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]