From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH v2 0/5] virtio: Tx performance improvements Date: Fri, 4 Mar 2016 10:17:32 -0800 Message-ID: <20160304101732.144d8901@xeon-e3> References: <1445231772-17467-1-git-send-email-stephen@networkplumber.org> <2330408.2AHZ8BdDek@xps13> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: "dev@dpdk.org" To: "Xie, Huawei" Return-path: Received: from mail-pf0-f182.google.com (mail-pf0-f182.google.com [209.85.192.182]) by dpdk.org (Postfix) with ESMTP id 6FDC42B96 for ; Fri, 4 Mar 2016 19:17:19 +0100 (CET) Received: by mail-pf0-f182.google.com with SMTP id 4so39320724pfd.1 for ; Fri, 04 Mar 2016 10:17:19 -0800 (PST) In-Reply-To: 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 Fri, 4 Mar 2016 06:18:17 +0000 "Xie, Huawei" wrote: > On 1/14/2016 9:49 PM, Xie, Huawei wrote: > > On 1/6/2016 8:04 PM, Thomas Monjalon wrote: > >> 2016-01-05 08:10, Xie, Huawei: > >>> On 10/26/2015 10:06 PM, Xie, Huawei wrote: > >>>> On 10/19/2015 1:16 PM, Stephen Hemminger wrote: > >>>>> This is a tested version of the virtio Tx performance improvements > >>>>> that I posted earlier on the list, and described at the DPDK Userspace > >>>>> meeting in Dublin. Together they get a 25% performance improvement for > >>>>> both small packet and large multi-segment packet case when testing > >>>>> from DPDK guest application to Linux KVM host. > >>>>> > >>>>> Stephen Hemminger (5): > >>>>> virtio: clean up space checks on xmit > >>>>> virtio: don't use unlikely for normal tx stuff > >>>>> virtio: use indirect ring elements > >>>>> virtio: use any layout on transmit > >>>>> virtio: optimize transmit enqueue > >>>> There is one open why merge-able header is used in tx path. Since old > >>>> implementation is also using the merge-able header in tx path if this > >>>> feature is negotiated, i choose to ack the patch and address this later > >>>> if not now. > >>>> > >>>> Acked-by: Huawei Xie > >>> Thomas: Resending them now. I don't understand the issue with merge-able header. Virtio negotiation is symmetric, if receiver is using merge-able header then the transmitter needs to send it also.