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 E29902BB1; Fri, 14 Apr 2017 10:36:31 +0200 (CEST) Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 14 Apr 2017 01:36:30 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.37,197,1488873600"; d="scan'208";a="248873946" Received: from irsmsx109.ger.corp.intel.com ([163.33.3.23]) by fmsmga004.fm.intel.com with ESMTP; 14 Apr 2017 01:36:29 -0700 Received: from irsmsx156.ger.corp.intel.com (10.108.20.68) by IRSMSX109.ger.corp.intel.com (163.33.3.23) with Microsoft SMTP Server (TLS) id 14.3.319.2; Fri, 14 Apr 2017 09:36:29 +0100 Received: from irsmsx108.ger.corp.intel.com ([169.254.11.239]) by IRSMSX156.ger.corp.intel.com ([169.254.3.246]) with mapi id 14.03.0319.002; Fri, 14 Apr 2017 09:36:29 +0100 From: "De Lara Guarch, Pablo" To: "Zhang, Roy Fan" , "dev@dpdk.org" CC: "stable@dpdk.org" , "Mcnamara, John" Thread-Topic: [dpdk-dev] [PATCH v2] crypto/scheduler: fix Doxygen comments Thread-Index: AQHSs5Z+SRU+1QB2gEuEk2sdTph9K6HEiJyQ Date: Fri, 14 Apr 2017 08:36:28 +0000 Message-ID: References: <1492006267-51909-1-git-send-email-roy.fan.zhang@intel.com> In-Reply-To: <1492006267-51909-1-git-send-email-roy.fan.zhang@intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiMmUzMzk0NjctMzBmZi00NzhlLWIwNjAtNzc0YzEyYmYzMzQ1IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX0lDIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE1LjkuNi42IiwiVHJ1c3RlZExhYmVsSGFzaCI6ImdMdTg1YjA4Rmx0ZUF2QkEzSmZXUndBaXduVXE4M0ZjU3ZIWkdWQUVnRTA9In0= x-ctpclassification: CTP_IC x-originating-ip: [163.33.239.182] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v2] crypto/scheduler: fix Doxygen comments 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, 14 Apr 2017 08:36:32 -0000 Hi Fan, > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Fan Zhang > Sent: Wednesday, April 12, 2017 3:11 PM > To: dev@dpdk.org > Cc: De Lara Guarch, Pablo; stable@dpdk.org; Mcnamara, John > Subject: [dpdk-dev] [PATCH v2] crypto/scheduler: fix Doxygen comments >=20 > This patch adds the missing doxygen comments and updated > inline comments to cryptodev scheduler >=20 > Fixes: d58a3f312545 ("crypto/scheduler: add documentation") >=20 > Signed-off-by: Fan Zhang > --- >=20 ... > --- a/drivers/crypto/scheduler/rte_cryptodev_scheduler.h > +++ b/drivers/crypto/scheduler/rte_cryptodev_scheduler.h > @@ -34,6 +34,18 @@ > #ifndef _RTE_CRYPTO_SCHEDULER_H > #define _RTE_CRYPTO_SCHEDULER_H >=20 > +/** > + * @file rte_cryptodev_scheduler.h > + * > + * RTE Cryptodev Scheduler Device > + * > + * The RTE Cryptodev Scheduler Device allows the aggregation of multiple > (slave) > + * Cryptodevs into a single logical crypto device, and the scheduling th= e > + * crypto operations to the slaves based on the mode of the specified > mode of > + * operation specified and supported. This implementation supports 3 > modes of > + * operation: round robin, packet-size based, and fail-over. > + */ > + > #include "rte_cryptodev_scheduler_operations.h" >=20 > #ifdef __cplusplus > @@ -45,19 +57,21 @@ extern "C" { > #define RTE_CRYPTODEV_SCHEDULER_MAX_NB_SLAVES (8) > #endif Remove "<" before this macro: /** Maximum number... */). >=20 > -/* round-robin scheduling mode */ > +/** round-robin scheduling mode string */ > #define SCHEDULER_MODE_NAME_ROUND_ROBIN round-robin > -/* packet-size based distribution scheduling mode */ > +/** packet-size based distribution scheduling mode string */ > #define SCHEDULER_MODE_NAME_PKT_SIZE_DISTR packet-size-distr > -/* fail-over mode */ > +/** fail-over scheduling mode string */ > #define SCHEDULER_MODE_NAME_FAIL_OVER fail-over > -/** Start with capital letters (same for other comments in the file) ... > int > rte_cryptodev_scheduler_ordering_get(uint8_t scheduler_id); > @@ -192,15 +238,13 @@ rte_cryptodev_scheduler_ordering_get(uint8_t > scheduler_id); > * @param scheduler_id > * The target scheduler device ID > * @param slaves > - * If successful, the function will write back > - * all slaves' device IDs to it. This > - * parameter SHALL either be an uint8_t array > - * of RTE_CRYPTODEV_SCHEDULER_MAX_NB_SLAVES > - * elements or NULL. > + * If successful, the function will write back all slaves' device IDs = to it. > + * This parameter SHALL either be an uint8_t array of SHALL -> will > + * RTE_CRYPTODEV_SCHEDULER_MAX_NB_SLAVES elements or NULL. > * > * @return > * - non-negative number: the number of slaves attached > - * - negative integer if error occurs. > + * - -ENOTSUP if the operation is not supported. > */ > int > rte_cryptodev_scheduler_slaves_get(uint8_t scheduler_id, uint8_t > *slaves); > @@ -211,16 +255,21 @@ typedef uint16_t > (*rte_cryptodev_scheduler_burst_enqueue_t)(void *qp_ctx, > typedef uint16_t (*rte_cryptodev_scheduler_burst_dequeue_t)(void > *qp_ctx, > struct rte_crypto_op **ops, uint16_t nb_ops); >=20 > +/** The data structure associated with each mode of scheduler. */ > struct rte_cryptodev_scheduler { > - const char *name; > - const char *description; > - enum rte_cryptodev_scheduler_mode mode; > + const char *name; /**< scheduler name */ > + const char *description; /**< scheduler description */ > + enum rte_cryptodev_scheduler_mode mode; /**< scheduling > mode */ >=20 > + /**< pointer to scheduler operation structure */ The description has to go after the variable, not before. > struct rte_cryptodev_scheduler_ops *ops; > }; >=20 > +/** round-robin mode scheduler */ > extern struct rte_cryptodev_scheduler *roundrobin_scheduler; > +/** packet-size based distribution mode scheduler */ > extern struct rte_cryptodev_scheduler *pkt_size_based_distr_scheduler; > +/** fail-over mode scheduler */ > extern struct rte_cryptodev_scheduler *failover_scheduler; >=20 > #ifdef __cplusplus > -- > 2.7.4