From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wg0-f53.google.com (mail-wg0-f53.google.com [74.125.82.53]) by dpdk.org (Postfix) with ESMTP id AB0475A9C for ; Mon, 9 Mar 2015 11:20:36 +0100 (CET) Received: by wghn12 with SMTP id n12so19582972wgh.6 for ; Mon, 09 Mar 2015 03:20:36 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=1Psv9PSDyB/xoIum6BgxkNISj4MBMTdekXQBCJzF9vI=; b=g8GeeuY137Guu1tVwIklPZ7ZM+Y4bCP5mkWco6iBD/bQyLWtFFDgriiO5qsceqp9/n AUYUVk7Curo5mHPIl3v7fyH9PM/TYzvxiKr1C+4tgZ8C4Osm361JlOfM3cqt+fR+lK3l YjKuyc7OgSHHyFK0il++uRvuz0zfTxBbmN9T3P2ZYkV09NrkdYuDHLVihbxbteUw1aeT BFPCvyrKivTowcgOcOMljc+UHYzyCJxJdVOQ5mQ7Kd+9oNem55Yaah3A66Eus9Tt/jE7 EZXtkHNcyJKJg/RXx0YXzyoND/AiSrXKzeZ/E2egWzbkbXJ9RjfLTV3cJ+q/IjKTc1b3 Pb9w== X-Gm-Message-State: ALoCoQnUkVluRcVVHixtZUeEWEPMASicoiEFrO+2+nLEHIp3KlGsRIVRz16sjNw7sEk3bs3rqJt8 X-Received: by 10.180.90.211 with SMTP id by19mr22143223wib.27.1425896436529; Mon, 09 Mar 2015 03:20:36 -0700 (PDT) Received: from vladz-laptop.localdomain ([109.65.117.109]) by mx.google.com with ESMTPSA id md2sm18728782wic.19.2015.03.09.03.20.35 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 09 Mar 2015 03:20:35 -0700 (PDT) From: Vlad Zolotarov To: dev@dpdk.org Date: Mon, 9 Mar 2015 12:20:30 +0200 Message-Id: <1425896433-12452-1-git-send-email-vladz@cloudius-systems.com> X-Mailer: git-send-email 2.1.0 Subject: [dpdk-dev] [PATCH v5 0/3]This series adds the missing flow for enabling the LRO in the ethdev and 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: Mon, 09 Mar 2015 10:20:36 -0000 The series starts with some cleanup work in the code the final patch (the actual adding of the LRO support) is going to touch/use/change. There are still quite a few issues in the ixgbe PMD code left but they have to be a matter of a different series and I've left a few "TODO" remarks in the code. The LRO ("RSC" in Intel's context) PMD completion handling code follows the same design as the corresponding Linux and FreeBSD implementation: pass the aggregation's cluster HEAD buffer to the NEXTP entry of the software ring till EOP is met. HW configuration follows the corresponding specs: this feature is supported only by x540 and 82599 PF devices. The feature has been tested with seastar TCP stack with the following configuration on Tx side: - MTU: 400B - 100 concurrent TCP connections. The results were: - Without LRO: total throughput: 0.12Gbps, coefficient of variance: 1.41% - With LRO: total throughput: 8.21Gbps, coefficient of variance: 0.59% This is an almost factor 80 improvement. New in v5: - Split the series into "bug fixes" and "all the rest" so that the former could be integrated into a 2.0 release. - Put the RTE_ETHDEV_HAS_LRO_SUPPORT definition at the beginning of rte_ethdev.h. - Removed the "TODO: Remove me" comment near RTE_ETHDEV_HAS_LRO_SUPPORT. New in v4: - Remove CONFIG_RTE_ETHDEV_LRO_SUPPORT from config/common_linuxapp. - Define RTE_ETHDEV_HAS_LRO_SUPPORT in rte_ethdev.h. - As a result of "ixgbe: check rxd number to avoid mbuf leak" (352078e8e) Vector Rx had to get the same treatment as Rx Bulk Alloc (see PATCH4 for more details). New in v3: - ixgbe_rx_alloc_bufs(): Always reset refcnt of the buffers to 1. Otherwise rte_pktmbuf_free() won't free them. New in v2: - Removed rte_eth_dev_data.lro_bulk_alloc and added ixgbe_hw.rx_bulk_alloc_allowed instead. - Unified the rx_pkt_bulk callback setting (a separate new patch). - Fixed a few styling and spelling issues. Vlad Zolotarov (3): ixgbe: Cleanups ixgbe: Code refactoring ixgbe: Add LRO support lib/librte_ether/rte_ethdev.h | 9 +- lib/librte_pmd_ixgbe/ixgbe_ethdev.c | 6 + lib/librte_pmd_ixgbe/ixgbe_ethdev.h | 5 + lib/librte_pmd_ixgbe/ixgbe_rxtx.c | 703 ++++++++++++++++++++++++++++++++---- lib/librte_pmd_ixgbe/ixgbe_rxtx.h | 6 + 5 files changed, 662 insertions(+), 67 deletions(-) -- 2.1.0