[dpdk-test-report] |WARNING| pw100109 [PATCH 3/3] cryptodev: rework session framework

automatic DPDK test reports
 help / color / mirror / Atom feed

From: checkpatch@dpdk.org
To: test-report@dpdk.org
Cc: Akhil Goyal <gakhil@marvell.com>
Subject: [dpdk-test-report] |WARNING| pw100109 [PATCH 3/3] cryptodev: rework session framework
Date: Thu, 30 Sep 2021 16:52:11 +0200 (CEST)	[thread overview]
Message-ID: <20210930145211.27BE41206FE@dpdk.org> (raw)
In-Reply-To: <20210930145014.2476799-4-gakhil@marvell.com>

Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/100109

_coding style issues_


ERROR:TRAILING_WHITESPACE: trailing whitespace
#1760: FILE: drivers/crypto/cnxk/cnxk_cryptodev_ops.c:497:
+^I^I      void *sess) $

ERROR:C99_COMMENTS: do not use C99 // comments
#3446: FILE: examples/ipsec-secgw/ipsec-secgw.c:2402:
+//static void

ERROR:C99_COMMENTS: do not use C99 // comments
#3447: FILE: examples/ipsec-secgw/ipsec-secgw.c:2403:
+//session_priv_pool_init(struct socket_ctx *ctx, int32_t socket_id,

ERROR:C99_COMMENTS: do not use C99 // comments
#3448: FILE: examples/ipsec-secgw/ipsec-secgw.c:2404:
+//	size_t sess_sz)

ERROR:C99_COMMENTS: do not use C99 // comments
#3449: FILE: examples/ipsec-secgw/ipsec-secgw.c:2405:
+//{

ERROR:C99_COMMENTS: do not use C99 // comments
#3450: FILE: examples/ipsec-secgw/ipsec-secgw.c:2406:
+//	char mp_name[RTE_MEMPOOL_NAMESIZE];

ERROR:C99_COMMENTS: do not use C99 // comments
#3451: FILE: examples/ipsec-secgw/ipsec-secgw.c:2407:
+//	struct rte_mempool *sess_mp;

ERROR:C99_COMMENTS: do not use C99 // comments
#3452: FILE: examples/ipsec-secgw/ipsec-secgw.c:2408:
+//	uint32_t nb_sess;

ERROR:C99_COMMENTS: do not use C99 // comments
#3453: FILE: examples/ipsec-secgw/ipsec-secgw.c:2409:
+//

ERROR:C99_COMMENTS: do not use C99 // comments
#3454: FILE: examples/ipsec-secgw/ipsec-secgw.c:2410:
+//	snprintf(mp_name, RTE_MEMPOOL_NAMESIZE,

ERROR:C99_COMMENTS: do not use C99 // comments
#3455: FILE: examples/ipsec-secgw/ipsec-secgw.c:2411:
+//			"sess_mp_priv_%u", socket_id);

ERROR:C99_COMMENTS: do not use C99 // comments
#3456: FILE: examples/ipsec-secgw/ipsec-secgw.c:2412:
+//	nb_sess = (get_nb_crypto_sessions() + CDEV_MP_CACHE_SZ *

ERROR:C99_COMMENTS: do not use C99 // comments
#3457: FILE: examples/ipsec-secgw/ipsec-secgw.c:2413:
+//		rte_lcore_count());

ERROR:C99_COMMENTS: do not use C99 // comments
#3458: FILE: examples/ipsec-secgw/ipsec-secgw.c:2414:
+//	nb_sess = RTE_MAX(nb_sess, CDEV_MP_CACHE_SZ *

ERROR:C99_COMMENTS: do not use C99 // comments
#3459: FILE: examples/ipsec-secgw/ipsec-secgw.c:2415:
+//			CDEV_MP_CACHE_MULTIPLIER);

ERROR:C99_COMMENTS: do not use C99 // comments
#3460: FILE: examples/ipsec-secgw/ipsec-secgw.c:2416:
+//	sess_mp = rte_mempool_create(mp_name,

ERROR:C99_COMMENTS: do not use C99 // comments
#3461: FILE: examples/ipsec-secgw/ipsec-secgw.c:2417:
+//			nb_sess,

ERROR:C99_COMMENTS: do not use C99 // comments
#3462: FILE: examples/ipsec-secgw/ipsec-secgw.c:2418:
+//			sess_sz,

ERROR:C99_COMMENTS: do not use C99 // comments
#3463: FILE: examples/ipsec-secgw/ipsec-secgw.c:2419:
+//			CDEV_MP_CACHE_SZ,

ERROR:C99_COMMENTS: do not use C99 // comments
#3464: FILE: examples/ipsec-secgw/ipsec-secgw.c:2420:
+//			0, NULL, NULL, NULL,

ERROR:C99_COMMENTS: do not use C99 // comments
#3465: FILE: examples/ipsec-secgw/ipsec-secgw.c:2421:
+//			NULL, socket_id,

ERROR:C99_COMMENTS: do not use C99 // comments
#3466: FILE: examples/ipsec-secgw/ipsec-secgw.c:2422:
+//			0);

ERROR:C99_COMMENTS: do not use C99 // comments
#3467: FILE: examples/ipsec-secgw/ipsec-secgw.c:2423:
+//	ctx->session_priv_pool = sess_mp;

ERROR:C99_COMMENTS: do not use C99 // comments
#3468: FILE: examples/ipsec-secgw/ipsec-secgw.c:2424:
+//

ERROR:C99_COMMENTS: do not use C99 // comments
#3469: FILE: examples/ipsec-secgw/ipsec-secgw.c:2425:
+//	if (ctx->session_priv_pool == NULL)

ERROR:C99_COMMENTS: do not use C99 // comments
#3470: FILE: examples/ipsec-secgw/ipsec-secgw.c:2426:
+//		rte_exit(EXIT_FAILURE,

ERROR:C99_COMMENTS: do not use C99 // comments
#3471: FILE: examples/ipsec-secgw/ipsec-secgw.c:2427:
+//			"Cannot init session priv pool on socket %d
",

ERROR:C99_COMMENTS: do not use C99 // comments
#3472: FILE: examples/ipsec-secgw/ipsec-secgw.c:2428:
+//			socket_id);

ERROR:C99_COMMENTS: do not use C99 // comments
#3473: FILE: examples/ipsec-secgw/ipsec-secgw.c:2429:
+//	else

ERROR:C99_COMMENTS: do not use C99 // comments
#3474: FILE: examples/ipsec-secgw/ipsec-secgw.c:2430:
+//		printf("Allocated session priv pool on socket %d
",

ERROR:C99_COMMENTS: do not use C99 // comments
#3475: FILE: examples/ipsec-secgw/ipsec-secgw.c:2431:
+//			socket_id);

ERROR:C99_COMMENTS: do not use C99 // comments
#3476: FILE: examples/ipsec-secgw/ipsec-secgw.c:2432:
+//}

ERROR:C99_COMMENTS: do not use C99 // comments
#3486: FILE: examples/ipsec-secgw/ipsec-secgw.c:2935:
+//		session_priv_pool_init(&socket_ctx[socket_id], socket_id,

ERROR:C99_COMMENTS: do not use C99 // comments
#3487: FILE: examples/ipsec-secgw/ipsec-secgw.c:2936:
+//			sess_sz);

total: 34 errors, 0 warnings, 3483 lines checked

          parent reply	other threads:[~2021-09-30 14:52 UTC|newest]

Thread overview: expand[flat|nested]  mbox.gz  Atom feed
 [parent not found: <20210930145014.2476799-4-gakhil@marvell.com>]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210930145211.27BE41206FE@dpdk.org \
    --to=checkpatch@dpdk.org \
    --cc=gakhil@marvell.com \
    --cc=test-report@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Be sure your reply has a Subject: header at the top and a blank line before the message body.

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).