From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id 7EEE728F3 for ; Thu, 9 Jun 2016 17:55:40 +0200 (CEST) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga101.fm.intel.com with ESMTP; 09 Jun 2016 08:55:40 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.26,445,1459839600"; d="scan'208";a="998656553" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.221.71]) by fmsmga002.fm.intel.com with SMTP; 09 Jun 2016 08:55:32 -0700 Received: by (sSMTP sendmail emulation); Thu, 09 Jun 2016 16:55:31 +0025 Date: Thu, 9 Jun 2016 16:55:31 +0100 From: Bruce Richardson To: Zhe Tao Cc: dev@dpdk.org, jingjing.wu@intel.com Message-ID: <20160609155530.GG12520@bricha3-MOBL3> References: <1458895321-21896-1-git-send-email-zhe.tao@intel.com> <1464110886-9504-1-git-send-email-zhe.tao@intel.com> <1464110886-9504-3-git-send-email-zhe.tao@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1464110886-9504-3-git-send-email-zhe.tao@intel.com> Organization: Intel Research and =?iso-8859-1?Q?De=ACvel?= =?iso-8859-1?Q?opment?= Ireland Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Subject: Re: [dpdk-dev] [PATCH v8 2/3] i40e: Add floating VEB support in i40e X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 09 Jun 2016 15:55:40 -0000 On Wed, May 25, 2016 at 01:28:05AM +0800, Zhe Tao wrote: > This patch add the support for floating VEB in i40e. > All the VFs VSIs can decide whether to connect to the legacy VEB/VEPA or > the floating VEB. When connect to the floating VEB a new floating VEB is > created. Now all the VFs need to connect to floating VEB or legacy VEB, > cannot connect to both of them. The PF and VMDQ,FD VSIs still connect to > the old legacy VEB/VEPA. > > All the VEB/VEPA concepts are not specific for FVL, they are defined in the > 802.1Qbg spec. copy reference to patch 1. > > Now the floating VEB feature is only avaiable in the specific version of FW. Please list the firmware versions rather than making the reader read through the patch. > > Signed-off-by: Zhe Tao > --- > doc/guides/nics/i40e.rst | 7 +++ > doc/guides/rel_notes/release_16_07.rst | 6 ++ > drivers/net/i40e/i40e_ethdev.c | 109 ++++++++++++++++++++++++++------- > drivers/net/i40e/i40e_ethdev.h | 2 + > drivers/net/i40e/i40e_pf.c | 11 +++- > 5 files changed, 112 insertions(+), 23 deletions(-) > > diff --git a/doc/guides/nics/i40e.rst b/doc/guides/nics/i40e.rst > index 934eb02..49a0598 100644 > --- a/doc/guides/nics/i40e.rst > +++ b/doc/guides/nics/i40e.rst > @@ -366,3 +366,10 @@ Delete all flow director rules on a port: > > testpmd> flush_flow_director 0 > > +Floating VEB > +~~~~~~~~~~~~~ > +FVL can support floating VEB feature. > +To enable this feature, the user should pass a devargs parameter to the EAL > +like "-w 84:00.0,enable_floating=1", and the application will make sure the PMD > +will use the floating VEB feature for all the VFs created by this PF device. > + I think this documentation part really need to be expanded to explain what a floating VEB is and why you'd use it. If this is well documented elsewhere, e.g. in the relevant datasheets for NICs using the i40e driver, by all means refer to that, but keep a summarised account of what it is here in the DPDK docs. Perhaps you can reuse the description from the release note below, just with an extra sentence or two about why the PF being down makes a difference, etc. > diff --git a/doc/guides/rel_notes/release_16_07.rst b/doc/guides/rel_notes/release_16_07.rst > index 30e78d4..8485b08 100644 > --- a/doc/guides/rel_notes/release_16_07.rst > +++ b/doc/guides/rel_notes/release_16_07.rst > @@ -47,6 +47,12 @@ New Features > * Dropped specific Xen Dom0 code. > * Dropped specific anonymous mempool code in testpmd. > > +* **Added floating VEB support for i40e PF driver.** > + > + Now VFs for i40e can connect to the floating VEB. > + With this feature, VFs can communicate with each other, but cannot access > + outside network. When PF is down, and VFs can still forward pkts between each > + other. This expanation belongs more in the i40e.rst document above. I think the release notes only needs a mention that the feature was added and then a link to the fuller description in the i40e doc. Regards, /Bruce