From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by dpdk.org (Postfix) with ESMTP id C4A3EA494 for ; Fri, 23 Mar 2018 19:08:53 +0100 (CET) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga107.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 23 Mar 2018 11:08:52 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.48,351,1517904000"; d="scan'208";a="27915065" Received: from irsmsx103.ger.corp.intel.com ([163.33.3.157]) by orsmga008.jf.intel.com with ESMTP; 23 Mar 2018 11:08:51 -0700 Received: from irsmsx101.ger.corp.intel.com ([169.254.1.176]) by IRSMSX103.ger.corp.intel.com ([169.254.3.61]) with mapi id 14.03.0319.002; Fri, 23 Mar 2018 18:08:50 +0000 From: "Trahe, Fiona" To: Thomas Monjalon CC: "dev@dpdk.org" , "ahmed.mansour@nxp.com" , "Shally.Verma@cavium.com" , "De Lara Guarch, Pablo" , "fiona.trahe@gmail.com" Thread-Topic: [dpdk-dev] [PATCH] compressdev: implement API Thread-Index: AQHTnFNA34V0d0oQkkqZkaaruzarZKOUTyIAgEoWovA= Date: Fri, 23 Mar 2018 18:08:49 +0000 Message-ID: <348A99DA5F5B7549AA880327E580B43589360349@IRSMSX101.ger.corp.intel.com> References: <1517595924-25963-1-git-send-email-fiona.trahe@intel.com> <1997769.o2qW9LT7Ur@xps> In-Reply-To: <1997769.o2qW9LT7Ur@xps> Accept-Language: en-IE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiMWMwOTQzMjctZDhmOC00ODlkLWJjYjktNGNlMGU4NGUzNWZhIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE2LjUuOS4zIiwiVHJ1c3RlZExhYmVsSGFzaCI6IkgyRWQ3V2tDcTk2QUcxNjNlRGpZb2FhN0RuQmh2Uk9iV0lCZjZTYzIxZjQ9In0= x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.0.116 dlp-reaction: no-action x-originating-ip: [163.33.239.181] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH] compressdev: implement API X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 23 Mar 2018 18:08:56 -0000 Hi Tomas, Sorry for the delay in replying to this > -----Original Message----- > From: Thomas Monjalon [mailto:thomas@monjalon.net] > Sent: Sunday, February 4, 2018 2:25 PM > To: Trahe, Fiona > Cc: dev@dpdk.org; ahmed.mansour@nxp.com; Shally.Verma@cavium.com; De Lara= Guarch, Pablo > > Subject: Re: [dpdk-dev] [PATCH] compressdev: implement API >=20 > 02/02/2018 19:25, Fiona Trahe: > > config/common_base | 6 + > > doc/api/doxy-api-index.md | 1 + > > doc/api/doxy-api.conf | 1 + > > lib/Makefile | 3 + > > lib/librte_compressdev/Makefile | 29 + > > lib/librte_compressdev/rte_comp.h | 503 ++++++++++++ >=20 > Why rte_comp.h instead of the more consistent rte_compress.h? [Fiona] I did originally... but ran into difficulty with horribly names li= ke RTE_COMPRESS_COMPRESS RTE_COMPRESS_DECOMPRESS rte_compress_compress_xform rte_compress_decompress_xform So compress is both the module prefix and the name of one of the actions. I could have used compressdev - but names were very long. So decided to opt for using=20 _compressdev_ in names to do with the device and _comp_ in names to do with the compression service Also I could have used compdev instead of compressdev,=20 but I felt compress should be in the lib name >=20 > > lib/librte_compressdev/rte_compressdev.c | 902 +++++++++++++= ++++++++ > > lib/librte_compressdev/rte_compressdev.h | 757 +++++++++++++= ++++ > > lib/librte_compressdev/rte_compressdev_pmd.c | 163 ++++ > > lib/librte_compressdev/rte_compressdev_pmd.h | 439 ++++++++++ > > lib/librte_compressdev/rte_compressdev_version.map | 47 ++ > > lib/librte_eal/common/include/rte_log.h | 1 + > > mk/rte.app.mk | 1 + > > 13 files changed, 2853 insertions(+) >=20 > Please update MAINTAINERS file and release notes. >=20 > Maybe it is worth splitting this patch in few shorter parts? [Fiona] will do >=20 >=20 > > --- a/config/common_base > > +++ b/config/common_base > > @@ -535,6 +535,12 @@ CONFIG_RTE_LIBRTE_PMD_MRVL_CRYPTO=3Dn > > CONFIG_RTE_LIBRTE_PMD_MRVL_CRYPTO_DEBUG=3Dn > > > > # > > +# Compile generic compression device library > > +# > > +CONFIG_RTE_LIBRTE_COMPRESSDEV=3Dy > > +CONFIG_RTE_COMPRESS_MAX_DEVS=3D64 > > + > > +# > > # Compile generic security library > > # > > CONFIG_RTE_LIBRTE_SECURITY=3Dy > > diff --git a/doc/api/doxy-api-index.md b/doc/api/doxy-api-index.md > > index d77f205..07b8e75 100644 > > --- a/doc/api/doxy-api-index.md > > +++ b/doc/api/doxy-api-index.md > > @@ -43,6 +43,7 @@ The public API headers are grouped by topics: > > [rte_tm] (@ref rte_tm.h), > > [rte_mtr] (@ref rte_mtr.h), > > [bbdev] (@ref rte_bbdev.h), > > + [compressdev] (@ref rte_compressdev.h), > > [cryptodev] (@ref rte_cryptodev.h), > > [security] (@ref rte_security.h), > > [eventdev] (@ref rte_eventdev.h), >=20 > Please move it between security and eventdev in these lists. ok