From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yuanhan Liu Subject: Re: [PATCH v2] virtio: use zeroed memory for simple TX header Date: Tue, 5 Apr 2016 12:26:54 +0800 Message-ID: <20160405042654.GN3080@yliu-dev.sh.intel.com> References: <1459822261-95284-1-git-send-email-rlane@bigswitch.com> <57032EE5.4050408@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Rich Lane , dev@dpdk.org, Huawei Xie , Stephen Hemminger To: "Tan, Jianfeng" Return-path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id A6CDB2BFD for ; Tue, 5 Apr 2016 06:25:29 +0200 (CEST) Content-Disposition: inline In-Reply-To: <57032EE5.4050408@intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Tue, Apr 05, 2016 at 11:20:05AM +0800, Tan, Jianfeng wrote: > Hi, > > On 4/5/2016 10:11 AM, Rich Lane wrote: > >For simple TX the virtio-net header must be zeroed, but it was using memory > >that had been initialized with indirect descriptor tables. This resulted in > >"unsupported gso type" errors from librte_vhost. > > > >We can use the same memory for every descriptor to save cachelines in the > >vswitch. > > Pointing all virtio_net_hdr into the same memory may brings performance, but > how much? It also introduces difficulty to adding tso in future? simple rxtx will not be enabled when TSO is enabled. --yliu