From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] examples/vhost: Fix vlan offload issue Date: Thu, 18 Dec 2014 00:47:16 +0100 Message-ID: <7354113.GC7lWJkY43@xps13> References: <1418357726-25141-1-git-send-email-changchun.ouyang@intel.com> <1589451.jCIKiaz4qv@xps13> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org To: "Ouyang, Changchun" Return-path: In-Reply-To: <1589451.jCIKiaz4qv@xps13> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" 2014-12-17 11:04, Thomas Monjalon: > 2014-12-17 00:51, Ouyang, Changchun: > > From: Thomas Monjalon [mailto:thomas.monjalon-pdR9zngts4EAvxtiuMwx3w@public.gmane.org] > > > 2014-12-12 12:15, Ouyang Changchun: > > > > The following commit break vm2vm hard mode test cases: > > > > commit db4014f2b65cb31bf209cadd5bcec778ca137fe2 > > > > Author: Huawei Xie > > > > Date: Thu Nov 13 06:34:07 2014 +0800 > > > > examples/vhost: use factorized default Rx/Tx configuration > > > > > > > > Investigation show that it needs enabling vlan offload since it is > > > > turn off by default, and Tx need it, especially when vm2vm is in hard mode. > > > > > > I missed something here. Where VLAN offload is disabled by default? > > > Could you point the code, please? > > > > Inside the function ixgbe_dev_info_get() > > The txq_flags is assigned value of "ETH_TXQ_FLAGS_MULTISEGS|ETH_TXQ_FLAGS_NOOFFLOADS", > > The ETH_TXQ_FLAGS_NOOFFLOADS contain ETH_TXQ_FLAGS_NOVLANOFFL. > > so VLAN offload is disabled. > > > > Do you think any incorrect in my original description? > > Yes. You say VLAN offload is turned off by default. > But it's the case only for ixgbe, i40e and vmxnet3. Applied with this log change "turn off by default in some drivers" Thanks -- Thomas