[dpdk-test-report] |WARNING| pw24214 [PATCH 6/6] eal: fix warning seen with armv8a clang

checkpatch at dpdk.org checkpatch at dpdk.org
Thu May 11 07:54:53 CEST 2017


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

_coding style issues_


WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#35: 
> diff --git a/lib/librte_eal/linuxapp/eal/Makefile b/lib/librte_eal/linuxapp/eal/Makefile

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (7, 15)
#69: FILE: lib/librte_eal/common/eal_common_launch.c:56:
        while (lcore_config[slave_id].state != WAIT &&
[...]
+               rte_pause();

ERROR:CODE_INDENT: code indent should use tabs where possible
#71: FILE: lib/librte_eal/common/eal_common_launch.c:57:
+              lcore_config[slave_id].state != FINISHED)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#71: FILE: lib/librte_eal/common/eal_common_launch.c:57:
+              lcore_config[slave_id].state != FINISHED)$

ERROR:CODE_INDENT: code indent should use tabs where possible
#72: FILE: lib/librte_eal/common/eal_common_launch.c:58:
+               rte_pause();$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#72: FILE: lib/librte_eal/common/eal_common_launch.c:58:
+               rte_pause();$

ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)

total: 3 errors, 4 warnings, 9 lines checked


More information about the test-report mailing list