From patchwork Wed Jul 12 19:58:46 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jan Blunck X-Patchwork-Id: 26867 X-Patchwork-Delegate: pablo.de.lara.guarch@intel.com Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id 722F37CAF; Wed, 12 Jul 2017 21:59:12 +0200 (CEST) Received: from mail-wm0-f67.google.com (mail-wm0-f67.google.com [74.125.82.67]) by dpdk.org (Postfix) with ESMTP id 180C3374C for ; Wed, 12 Jul 2017 21:59:05 +0200 (CEST) Received: by mail-wm0-f67.google.com with SMTP id y5so948571wmh.3 for ; Wed, 12 Jul 2017 12:59:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:from:to:cc:subject:date:message-id:in-reply-to:references; bh=RrqO/jvyylEIZSkEt5CeeZlum0RLxUXg3zwvcja9Cf4=; b=XheidOxHA5jMRmK5BOYXm9ZsxNZSMj6T8acK5dTJcl3CAYTnLfzNWLgL6S1ksTVGRE GJynO89SW0hpfiPxIgeF2JyiuqxrJSa8sX0a6vBV37bnqlt7Yj+NyPWDve4HYw+emwIJ Y6DY8wsrg5h5W3QYNDXjGhKM4crdAPtXWY2J+BhR+D84Mgg9jvFXLAx3CCivA3YjAjZg ofTE4DpVI3H2aclp0G2B2e8Fbg8WriKuo//T+3IF+oZxn9sQIQGR9t1zuxMvCczVXyJ0 viQHdiAs36fOiAPhd9+AlHYT3QyzV3QWUOlaoKrShmGRWCebJQIHvl5D6eOy2OTch03k ObCQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:from:to:cc:subject:date:message-id :in-reply-to:references; bh=RrqO/jvyylEIZSkEt5CeeZlum0RLxUXg3zwvcja9Cf4=; b=laXgjOv9lh5BwqWw9veA5lvVFtHPzkW3qt0VxVHXEkOgbPDR8EeLm/Zjyi3z9Irk94 uDdhq58r6+mlA/4NZRwKG8npH8Sy93B6Frgiq+7t2/Vw7z1WRiSfSHfnDrgVG5IewyIP LSns0fyLUixwA8xxI+YhZepLYU63faMptSq+OswlEahJLeO2h+mh05utpBe+2OUbJNA4 3Ex4FhrEGvLDxEtAWOoWdBdLRUn7tpZ5SOOeqYJvdsMgen25Z54oZN3os4PQTzVb6Zfb InXPzFD8+KgEW4OGEsnxCD4kpeILBNioExWNjgKoXB7PR1/Vf6m+rE8stALx3Dvdfppy Ksbg== X-Gm-Message-State: AIVw111EakhrWr61k8awMJh8sKxTjHbUIdJNvBbmxQc+dqQ5spiRztbC UjSkUxgDxYWpoojA X-Received: by 10.80.163.210 with SMTP id t18mr160590edb.158.1499889544658; Wed, 12 Jul 2017 12:59:04 -0700 (PDT) Received: from localhost.localdomain ([37.228.147.16]) by smtp.gmail.com with ESMTPSA id t3sm1420660edh.1.2017.07.12.12.59.03 (version=TLS1 cipher=AES128-SHA bits=128/128); Wed, 12 Jul 2017 12:59:03 -0700 (PDT) From: Jan Blunck To: dev@dpdk.org Cc: declan.doherty@intel.com, pablo.de.lara.guarch@intel.com Date: Wed, 12 Jul 2017 15:58:46 -0400 Message-Id: <20170712195846.65286-5-jblunck@infradead.org> X-Mailer: git-send-email 2.13.2 In-Reply-To: <20170712195846.65286-1-jblunck@infradead.org> References: <20170712075940.58559-1-jblunck@infradead.org> <20170712195846.65286-1-jblunck@infradead.org> Subject: [dpdk-dev] [PATCH v2 4/4] cryptodev: move parameter parsing to its own header 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" This moves the parameter parsing functions out of the rte_cryptodev_vdev.h header to not require the rte_vdev.h header at build time of the library. Signed-off-by: Jan Blunck --- lib/librte_cryptodev/Makefile | 1 + lib/librte_cryptodev/rte_cryptodev_pmd.c | 2 +- lib/librte_cryptodev/rte_cryptodev_vdev.h | 44 +----------- lib/librte_cryptodev/rte_cryptodev_vdev_params.h | 89 ++++++++++++++++++++++++ 4 files changed, 93 insertions(+), 43 deletions(-) create mode 100644 lib/librte_cryptodev/rte_cryptodev_vdev_params.h diff --git a/lib/librte_cryptodev/Makefile b/lib/librte_cryptodev/Makefile index 6ac331bc9..59d9a4e2e 100644 --- a/lib/librte_cryptodev/Makefile +++ b/lib/librte_cryptodev/Makefile @@ -49,6 +49,7 @@ SYMLINK-y-include += rte_crypto_sym.h SYMLINK-y-include += rte_cryptodev.h SYMLINK-y-include += rte_cryptodev_pmd.h SYMLINK-y-include += rte_cryptodev_vdev.h +SYMLINK-y-include += rte_cryptodev_vdev_params.h SYMLINK-y-include += rte_cryptodev_pci.h # versioning export map diff --git a/lib/librte_cryptodev/rte_cryptodev_pmd.c b/lib/librte_cryptodev/rte_cryptodev_pmd.c index eaeddf74b..d6eaa2a90 100644 --- a/lib/librte_cryptodev/rte_cryptodev_pmd.c +++ b/lib/librte_cryptodev/rte_cryptodev_pmd.c @@ -32,7 +32,7 @@ #include -#include "rte_cryptodev_vdev.h" +#include "rte_cryptodev_vdev_params.h" #include "rte_cryptodev_pci.h" #include "rte_cryptodev_pmd.h" diff --git a/lib/librte_cryptodev/rte_cryptodev_vdev.h b/lib/librte_cryptodev/rte_cryptodev_vdev.h index 04d0796d4..94df46cab 100644 --- a/lib/librte_cryptodev/rte_cryptodev_vdev.h +++ b/lib/librte_cryptodev/rte_cryptodev_vdev.h @@ -36,33 +36,8 @@ #include #include -#include "rte_cryptodev.h" - -#define RTE_CRYPTODEV_VDEV_DEFAULT_MAX_NB_QUEUE_PAIRS 8 -#define RTE_CRYPTODEV_VDEV_DEFAULT_MAX_NB_SESSIONS 2048 - -#define RTE_CRYPTODEV_VDEV_NAME ("name") -#define RTE_CRYPTODEV_VDEV_MAX_NB_QP_ARG ("max_nb_queue_pairs") -#define RTE_CRYPTODEV_VDEV_MAX_NB_SESS_ARG ("max_nb_sessions") -#define RTE_CRYPTODEV_VDEV_SOCKET_ID ("socket_id") - -static const char * const cryptodev_vdev_valid_params[] = { - RTE_CRYPTODEV_VDEV_NAME, - RTE_CRYPTODEV_VDEV_MAX_NB_QP_ARG, - RTE_CRYPTODEV_VDEV_MAX_NB_SESS_ARG, - RTE_CRYPTODEV_VDEV_SOCKET_ID -}; - -/** - * @internal - * Initialisation parameters for virtual crypto devices - */ -struct rte_crypto_vdev_init_params { - unsigned int max_nb_queue_pairs; - unsigned int max_nb_sessions; - uint8_t socket_id; - char name[RTE_CRYPTODEV_NAME_MAX_LEN]; -}; +#include "rte_cryptodev_pmd.h" +#include "rte_cryptodev_vdev_params.h" /** * @internal @@ -107,19 +82,4 @@ rte_cryptodev_vdev_pmd_init(const char *name, size_t dev_private_size, return cryptodev; } -/** - * @internal - * Parse virtual device initialisation parameters input arguments - * - * @params params Initialisation parameters with defaults set. - * @params input_args Command line arguments - * - * @return - * 0 on successful parse - * <0 on failure to parse - */ -int -rte_cryptodev_vdev_parse_init_params(struct rte_crypto_vdev_init_params *params, - const char *input_args); - #endif /* _RTE_CRYPTODEV_VDEV_H_ */ diff --git a/lib/librte_cryptodev/rte_cryptodev_vdev_params.h b/lib/librte_cryptodev/rte_cryptodev_vdev_params.h new file mode 100644 index 000000000..66ba0d482 --- /dev/null +++ b/lib/librte_cryptodev/rte_cryptodev_vdev_params.h @@ -0,0 +1,89 @@ +/*- + * BSD LICENSE + * + * Copyright(c) 2017 Intel Corporation. All rights reserved. + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted provided that the following conditions + * are met: + * + * * Redistributions of source code must retain the above copyright + * notice, this list of conditions and the following disclaimer. + * * Redistributions in binary form must reproduce the above copyright + * notice, this list of conditions and the following disclaimer in + * the documentation and/or other materials provided with the + * distribution. + * * Neither the name of the copyright holder nor the names of its + * contributors may be used to endorse or promote products derived + * from this software without specific prior written permission. + * + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT + * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, + * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT + * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, + * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY + * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE + * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. + */ + +#ifndef _RTE_CRYPTODEV_VDEV_PARAMS_H_ +#define _RTE_CRYPTODEV_VDEV_PARAMS_H_ + +#ifdef __cplusplus +extern "C" { +#endif + +#include + +#include "rte_cryptodev.h" + +#define RTE_CRYPTODEV_VDEV_DEFAULT_MAX_NB_QUEUE_PAIRS 8 +#define RTE_CRYPTODEV_VDEV_DEFAULT_MAX_NB_SESSIONS 2048 + +#define RTE_CRYPTODEV_VDEV_NAME ("name") +#define RTE_CRYPTODEV_VDEV_MAX_NB_QP_ARG ("max_nb_queue_pairs") +#define RTE_CRYPTODEV_VDEV_MAX_NB_SESS_ARG ("max_nb_sessions") +#define RTE_CRYPTODEV_VDEV_SOCKET_ID ("socket_id") + +static const char * const cryptodev_vdev_valid_params[] = { + RTE_CRYPTODEV_VDEV_NAME, + RTE_CRYPTODEV_VDEV_MAX_NB_QP_ARG, + RTE_CRYPTODEV_VDEV_MAX_NB_SESS_ARG, + RTE_CRYPTODEV_VDEV_SOCKET_ID +}; + +/** + * @internal + * Initialisation parameters for virtual crypto devices + */ +struct rte_crypto_vdev_init_params { + unsigned int max_nb_queue_pairs; + unsigned int max_nb_sessions; + uint8_t socket_id; + char name[RTE_CRYPTODEV_NAME_MAX_LEN]; +}; + +/** + * @internal + * Parse virtual device initialisation parameters input arguments + * + * @params params Initialisation parameters with defaults set. + * @params input_args Command line arguments + * + * @return + * 0 on successful parse + * <0 on failure to parse + */ +int +rte_cryptodev_vdev_parse_init_params(struct rte_crypto_vdev_init_params *params, + const char *input_args); + +#ifdef __cplusplus +} +#endif + +#endif /* _RTE_CRYPTODEV_VDEV_PARAMS_H_ */