From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id 84CB96829 for ; Fri, 21 Nov 2014 17:57:10 +0100 (CET) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga102.jf.intel.com with ESMTP; 21 Nov 2014 09:04:58 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.07,432,1413270000"; d="scan'208";a="611899713" Received: from irsmsx152.ger.corp.intel.com ([163.33.192.66]) by orsmga001.jf.intel.com with ESMTP; 21 Nov 2014 09:07:13 -0800 Received: from irsmsx101.ger.corp.intel.com ([169.254.1.105]) by IRSMSX152.ger.corp.intel.com ([169.254.6.56]) with mapi id 14.03.0195.001; Fri, 21 Nov 2014 17:07:13 +0000 From: "Doherty, Declan" To: "dev@dpdk.org" Thread-Topic: [PATCH v6 0/8] link bonding Thread-Index: AQHP+oWZ4U4tHVspyUOTZ4uPWbecpJxVXiEAgBYHBmA= Date: Fri, 21 Nov 2014 17:07:12 +0000 Message-ID: <345C63BAECC1AD42A2EC8C63AFFC3ADC27422D1C@IRSMSX101.ger.corp.intel.com> References: <1413291597-27326-1-git-send-email-declan.doherty@intel.com> <1415362978-6306-1-git-send-email-declan.doherty@intel.com> In-Reply-To: Accept-Language: en-IE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [163.33.239.180] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Cc: "homas.monjalon@6wind.com" Subject: Re: [dpdk-dev] [PATCH v6 0/8] link bonding 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: Fri, 21 Nov 2014 16:57:11 -0000 Hey Thomas, have you had a chance to have a look at this patchset, if you h= ave any comments or issues can you let me know asap, as the patch sets for = link bonding modes 4 & 5 are dependent on this set and I don't want to be b= locking them from the next release candidate. Thanks=20 Declan > -----Original Message----- > From: De Lara Guarch, Pablo > Sent: Friday, November 7, 2014 4:41 PM > To: Doherty, Declan; dev@dpdk.org > Cc: thomas.monjalon@6wind.com > Subject: RE: [PATCH v6 0/8] link bonding >=20 >=20 >=20 > > -----Original Message----- > > From: Doherty, Declan > > Sent: Friday, November 07, 2014 12:23 PM > > To: dev@dpdk.org > > Cc: De Lara Guarch, Pablo; thomas.monjalon@6wind.com; Doherty, Declan > > Subject: [PATCH v6 0/8] link bonding > > > > v6: > > - Re-based to dpdk.org addressing associated issues for MBUF_REFCNT > > - Added details to testpmd user guide for new command to set link statu= s > > polling interval. > > > > v5: > > - Fix uninitialized variable in broadcast_tx_burst function which cause= d a > > build error in 32-bit build > > - Address unit test issue which is exposed by new test in mode 4/5 patc= h sets > > > > v4: > > - Rebased to account for changes in master. > > - Fix for rte_eth_bond_slaves_get() introduced in v3 patch set > > - Addressed issue around disabling/enabling link status polling around > > adding/ > > removing slaves devices. > > > > v3 : > > - Typo fix for the bond free mbufs patch. > > - Rebased to account for changes in the mbuf patches. > > - Add support for slave devices which don't support link status interru= pts > > - Tidy up the link bonding unit test so that all tests use the new test= macros. > > > > v2 : > > Addresses issues with the logic around the handling of fail transmissio= ns. > > In this version all modes behave in a manner similar to a standard PMD, > > returning the number of successfully transmitted mbufs and with the fai= ling > > mbufs at the end of bufs array for freeing / retransmission by the > > application software > > > > v1: > > > > This patch set adds support for link status interrupt in the link bondi= ng > > pmd. It also contains some patches to tidy up the code structure and to > > of the link bonding code and to fix bugs relating to transmission > > failures in the under lying slave pmd which could lead to leaked mbufs. > > > > > > Declan Doherty (8): > > bond: link status interrupt support > > bond: removing switch statement from rx burst method > > bond: fix naming inconsistency in tx_burst_round_robin > > bond: free mbufs if transmission fails in bonding tx_burst functions > > test app: adding support for generating variable sized packet > > testpmd: adding parameter to reconfig method to set socket_id when > > adding new port to portlist > > bond: lsc polling support > > bond: unit test test macro refactor > > > > app/test-pmd/cmdline.c | 65 +- > > app/test-pmd/testpmd.c | 3 +- > > app/test-pmd/testpmd.h | 2 +- > > app/test/packet_burst_generator.c | 25 +- > > app/test/packet_burst_generator.h | 6 +- > > app/test/test.h | 7 +- > > app/test/test_link_bonding.c | 3347 ++++++++++++++-----= -------- > > app/test/virtual_pmd.c | 96 +- > > app/test/virtual_pmd.h | 53 +- > > doc/guides/testpmd_app_ug/testpmd_funcs.rst | 19 + > > lib/librte_pmd_bond/rte_eth_bond.h | 80 + > > lib/librte_pmd_bond/rte_eth_bond_api.c | 319 ++- > > lib/librte_pmd_bond/rte_eth_bond_args.c | 28 +- > > lib/librte_pmd_bond/rte_eth_bond_pmd.c | 553 +++-- > > lib/librte_pmd_bond/rte_eth_bond_private.h | 71 +- > > 15 files changed, 2715 insertions(+), 1959 deletions(-) > > > > -- > > 1.7.12.2 >=20 > Acked-by: Pablo de Lara