From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <test-report-bounces@dpdk.org>
Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124])
	by inbox.dpdk.org (Postfix) with ESMTP id 49E83A00C4
	for <public@inbox.dpdk.org>; Thu, 28 Jul 2022 00:37:37 +0200 (CEST)
Received: from [217.70.189.124] (localhost [127.0.0.1])
	by mails.dpdk.org (Postfix) with ESMTP id 44A8241156;
	Thu, 28 Jul 2022 00:37:37 +0200 (CEST)
Received: from dpdk.org (dpdk.org [92.243.24.197])
 by mails.dpdk.org (Postfix) with ESMTP id 3A31140DF7
 for <test-report@dpdk.org>; Thu, 28 Jul 2022 00:37:36 +0200 (CEST)
Received: by dpdk.org (Postfix, from userid 65534)
 id 2CE89120765; Thu, 28 Jul 2022 00:37:36 +0200 (CEST)
Subject: |WARNING| pw114284 [PATCH V3 06/17] pipeline: add support for
 pipeline I/O specification
In-Reply-To: <20220727223639.598932-6-cristian.dumitrescu@intel.com>
References: <20220727223639.598932-6-cristian.dumitrescu@intel.com>
To: test-report@dpdk.org
From: checkpatch@dpdk.org
Cc: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
Message-Id: <20220727223736.2CE89120765@dpdk.org>
Date: Thu, 28 Jul 2022 00:37:36 +0200 (CEST)
X-BeenThere: test-report@dpdk.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: automatic DPDK test reports <test-report.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/test-report>,
 <mailto:test-report-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/test-report/>
List-Post: <mailto:test-report@dpdk.org>
List-Help: <mailto:test-report-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/test-report>,
 <mailto:test-report-request@dpdk.org?subject=subscribe>
Errors-To: test-report-bounces@dpdk.org

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

_coding style issues_


WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#193: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3672:
+		if (err_msg)
+		    *err_msg = "Invalid statement.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#202: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3681:
+		if (err_msg)
+		    *err_msg = "Invalid <n_slots> parameter.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#211: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3690:
+		if (err_msg)
+		    *err_msg = "Invalid <n_sessions> parameter.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#227: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3706:
+		if (err_msg)
+		    *err_msg = "Invalid statement.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#236: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3715:
+		if (err_msg)
+		    *err_msg = "Invalid <queue_id> parameter.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#245: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3724:
+		if (err_msg)
+		    *err_msg = "Invalid <burst_size> parameter.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#257: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3736:
+		if (err_msg)
+		    *err_msg = "Memory allocation failed.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#278: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3757:
+		if (err_msg)
+		    *err_msg = "Invalid statement.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#287: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3766:
+		if (err_msg)
+		    *err_msg = "Invalid <burst_size> parameter.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#299: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3778:
+		if (err_msg)
+		    *err_msg = "Memory allocation failed.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#324: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3803:
+		if (err_msg)
+		    *err_msg = "Invalid statement.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#332: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3811:
+		if (err_msg)
+		    *err_msg = "Invalid <mempool_name> parameter.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#341: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3820:
+		if (err_msg)
+		    *err_msg = "Invalid <n_loops> parameter.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#350: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3829:
+		if (err_msg)
+		    *err_msg = "Invalid <n_pkts_max> parameter.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#362: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3841:
+		if (err_msg)
+		    *err_msg = "Memory allocation failed.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#391: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3870:
+		if (err_msg)
+		    *err_msg = "Invalid statement.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#400: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3879:
+		if (err_msg)
+		    *err_msg = "Invalid <file_descriptor> parameter.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#409: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3888:
+		if (err_msg)
+		    *err_msg = "Invalid <mtu> parameter.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#417: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3896:
+		if (err_msg)
+		    *err_msg = "Invalid <mempool_name> parameter.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#426: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3905:
+		if (err_msg)
+		    *err_msg = "Invalid <burst_size> parameter.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#434: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3913:
+		if (err_msg)
+		    *err_msg = "Memory allocation failed.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#456: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3935:
+		if (err_msg)
+		    *err_msg = "Invalid statement.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#465: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3944:
+		if (err_msg)
+		    *err_msg = "Invalid <queue_id> parameter.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#474: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3953:
+		if (err_msg)
+		    *err_msg = "Invalid <burst_size> parameter.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#486: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3965:
+		if (err_msg)
+		    *err_msg = "Memory allocation failed.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#507: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3986:
+		if (err_msg)
+		    *err_msg = "Invalid statement.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#516: FILE: lib/pipeline/rte_swx_pipeline_spec.c:3995:
+		if (err_msg)
+		    *err_msg = "Invalid <burst_size> parameter.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#528: FILE: lib/pipeline/rte_swx_pipeline_spec.c:4007:
+		if (err_msg)
+		    *err_msg = "Memory allocation failed.";

ERROR:SPACING: space required before the open brace '{'
#547: FILE: lib/pipeline/rte_swx_pipeline_spec.c:4026:
+	if ((n_tokens != 2) || strcmp(tokens[0], "file")){

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#548: FILE: lib/pipeline/rte_swx_pipeline_spec.c:4027:
+		if (err_msg)
+		    *err_msg = "Invalid statement.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#564: FILE: lib/pipeline/rte_swx_pipeline_spec.c:4043:
+		if (err_msg)
+		    *err_msg = "Memory allocation failed.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#586: FILE: lib/pipeline/rte_swx_pipeline_spec.c:4065:
+		if (err_msg)
+		    *err_msg = "Invalid statement.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#595: FILE: lib/pipeline/rte_swx_pipeline_spec.c:4074:
+		if (err_msg)
+		    *err_msg = "Invalid <file_descriptor> parameter.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#604: FILE: lib/pipeline/rte_swx_pipeline_spec.c:4083:
+		if (err_msg)
+		    *err_msg = "Invalid <burst_size> parameter.";

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 20)
#612: FILE: lib/pipeline/rte_swx_pipeline_spec.c:4091:
+		if (err_msg)
+		    *err_msg = "Memory allocation failed.";

WARNING:TYPO_SPELLING: 'defaut' may be misspelled - perhaps 'default'?
#651: FILE: lib/pipeline/rte_swx_pipeline_spec.c:4130:
+	/* Initialize with the defaut values. */

total: 1 errors, 35 warnings, 934 lines checked