From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 080D6CE7 for ; Wed, 11 Feb 2015 12:17:17 +0100 (CET) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga103.fm.intel.com with ESMTP; 11 Feb 2015 03:10:11 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.09,558,1418112000"; d="scan'208";a="664791334" Received: from irsmsx101.ger.corp.intel.com ([163.33.3.153]) by fmsmga001.fm.intel.com with ESMTP; 11 Feb 2015 03:17:15 -0800 Received: from irsmsx108.ger.corp.intel.com ([169.254.11.64]) by IRSMSX101.ger.corp.intel.com ([169.254.1.244]) with mapi id 14.03.0195.001; Wed, 11 Feb 2015 11:17:14 +0000 From: "Gonzalez Monroy, Sergio" To: Neil Horman Thread-Topic: [dpdk-dev] [PATCH v3 0/5] New Reorder Library Thread-Index: AQHQQh5/c5wcC5FMT0CsTR6ofwzVLZzmymmAgASI/bA= Date: Wed, 11 Feb 2015 11:17:13 +0000 Message-ID: <91383E96CE459D47BCE92EFBF5CE73B004F4ABB1@IRSMSX108.ger.corp.intel.com> References: <1422623694-6430-1-git-send-email-sergio.gonzalez.monroy@intel.com> <1423235164-3808-1-git-send-email-sergio.gonzalez.monroy@intel.com> <20150208135833.GC12309@neilslaptop.think-freely.org> In-Reply-To: <20150208135833.GC12309@neilslaptop.think-freely.org> Accept-Language: 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: "dev@dpdk.org" Subject: Re: [dpdk-dev] [PATCH v3 0/5] New Reorder Library 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: Wed, 11 Feb 2015 11:17:18 -0000 > From: Neil Horman [mailto:nhorman@tuxdriver.com] > Sent: Sunday, February 8, 2015 1:59 PM > To: Gonzalez Monroy, Sergio > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v3 0/5] New Reorder Library >=20 > On Fri, Feb 06, 2015 at 03:05:59PM +0000, Sergio Gonzalez Monroy wrote: > > This series introduces the new reorder library along with unit tests, > > sample app and a new entry in the programmers guide describing the > library. > > > > The library provides reordering of mbufs based on their sequence number= . > > > > As mention in the patch describing the library, one use case is the > > packet distributor. > > The distributor receives packets, assigns them a sequence number and > > sends them to the workers. > > The workers process those packets and return them to the distributor. > > The distributor collects out-of-order packets from the workers and > > uses this library to reorder the packets based on the sequence number > > they were assigned. > > > > v3: > > - fix copyright date > > - add option to sample app to disable reordering > > - add packet ordering sample guide entry > > > > v2: > > - add programmers guide entry describing the library > > - use malloc instead of memzone to allocate memory > > - modify create and init implementation, init takes a reorder buffer a= s > input > > and create reserves memory and call init. > > - update unit tests > > > > Sergio Gonzalez Monroy (5): > > reorder: new reorder library > > app: New reorder unit test > > examples: new sample app packet_ordering > > doc: new reorder library description > > doc: new packet ordering app description > > > > app/test/Makefile | 2 + > > app/test/test_reorder.c | 393 ++++++++++++++ > > config/common_bsdapp | 5 + > > config/common_linuxapp | 5 + > > doc/guides/prog_guide/index.rst | 1 + > > doc/guides/prog_guide/reorder_lib.rst | 115 ++++ > > doc/guides/sample_app_ug/index.rst | 1 + > > doc/guides/sample_app_ug/packet_ordering.rst | 102 ++++ > > examples/packet_ordering/Makefile | 50 ++ > > examples/packet_ordering/main.c | 695 > +++++++++++++++++++++++++ > > lib/Makefile | 1 + > > lib/librte_eal/common/include/rte_tailq_elem.h | 2 + > > lib/librte_mbuf/rte_mbuf.h | 3 + > > lib/librte_reorder/Makefile | 50 ++ > > lib/librte_reorder/rte_reorder.c | 416 +++++++++++++++ > > lib/librte_reorder/rte_reorder.h | 181 +++++++ > > mk/rte.app.mk | 4 + > > 17 files changed, 2026 insertions(+) > > create mode 100644 app/test/test_reorder.c create mode 100644 > > doc/guides/prog_guide/reorder_lib.rst > > create mode 100644 doc/guides/sample_app_ug/packet_ordering.rst > > create mode 100644 examples/packet_ordering/Makefile create mode > > 100644 examples/packet_ordering/main.c create mode 100644 > > lib/librte_reorder/Makefile create mode 100644 > > lib/librte_reorder/rte_reorder.c create mode 100644 > > lib/librte_reorder/rte_reorder.h > > > > -- > > 1.9.3 > > > > > Functionally it looks pretty good. You need to define an export map and > LIBABIVER macro in your makefile though. > Neil Oh yes! I did forget about the versioning. I'll send v4 shortly. Regards, Sergio