From mboxrd@z Thu Jan  1 00:00:00 1970
From: Thomas Monjalon <thomas@monjalon.net>
Subject: Re: [PATCH] compressdev: implement API
Date: Sun, 04 Feb 2018 15:24:40 +0100
Message-ID: <1997769.o2qW9LT7Ur@xps>
References: <1517595924-25963-1-git-send-email-fiona.trahe@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7Bit
Cc: dev@dpdk.org, ahmed.mansour@nxp.com, Shally.Verma@cavium.com,
 pablo.de.lara.guarch@intel.com
To: Fiona Trahe <fiona.trahe@intel.com>
Return-path: <dev-bounces@dpdk.org>
Received: from out4-smtp.messagingengine.com (out4-smtp.messagingengine.com
 [66.111.4.28]) by dpdk.org (Postfix) with ESMTP id 13FC91B29E
 for <dev@dpdk.org>; Sun,  4 Feb 2018 16:07:22 +0100 (CET)
In-Reply-To: <1517595924-25963-1-git-send-email-fiona.trahe@intel.com>
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

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 ++++++++++++

Why rte_comp.h instead of the more consistent rte_compress.h?

>  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(+)

Please update MAINTAINERS file and release notes.

Maybe it is worth splitting this patch in few shorter parts?


> --- a/config/common_base
> +++ b/config/common_base
> @@ -535,6 +535,12 @@ CONFIG_RTE_LIBRTE_PMD_MRVL_CRYPTO=n
>  CONFIG_RTE_LIBRTE_PMD_MRVL_CRYPTO_DEBUG=n
>  
>  #
> +# Compile generic compression device library
> +#
> +CONFIG_RTE_LIBRTE_COMPRESSDEV=y
> +CONFIG_RTE_COMPRESS_MAX_DEVS=64
> +
> +#
>  # Compile generic security library
>  #
>  CONFIG_RTE_LIBRTE_SECURITY=y
> 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),

Please move it between security and eventdev in these lists.