From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id 004602BA6 for ; Wed, 29 Jun 2016 11:42:10 +0200 (CEST) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga102.fm.intel.com with ESMTP; 29 Jun 2016 02:42:10 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.26,546,1459839600"; d="scan'208";a="1011891144" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.220.81]) by fmsmga002.fm.intel.com with SMTP; 29 Jun 2016 02:42:08 -0700 Received: by (sSMTP sendmail emulation); Wed, 29 Jun 2016 10:42:07 +0025 Date: Wed, 29 Jun 2016 10:42:07 +0100 From: Bruce Richardson To: Zhe Tao Cc: dev@dpdk.org, jingjing.wu@intel.com Message-ID: <20160629094206.GA4720@bricha3-MOBL3> References: <1467004353-7534-1-git-send-email-zhe.tao@intel.com> <1467012016-8273-1-git-send-email-zhe.tao@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1467012016-8273-1-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 v14 0/2] i40e: add floating VEB support for 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: Wed, 29 Jun 2016 09:42:11 -0000 On Mon, Jun 27, 2016 at 03:20:14PM +0800, Zhe Tao wrote: > This patch-set 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 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. > > This floating VEB only take effects on the specific version F/W which newer > than 5.0. > > Zhe Tao (2): > i40e: support floating VEB config > i40e: add floating VEB support in i40e > I think the order of the patches would be better reversed, with the feature coming first and the config after. The documentation would then move to the new second patch on config, which is where it better fits, since a lot of it describes how to configure the feature. Regards, /Bruce