automatic DPDK test reports
 help / color / mirror / Atom feed
* [dpdk-test-report] |WARNING| pw29848 [PATCH] Mellanox ConnectX-5 crashes and mbuf leak
       [not found] <374F8C13-CFB0-42FD-8993-BF7F0401F891@mellanox.com>
@ 2017-10-06 22:32 ` checkpatch
  0 siblings, 0 replies; only message in thread
From: checkpatch @ 2017-10-06 22:32 UTC (permalink / raw)
  To: test-report; +Cc: Yongseok Koh

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

_coding style issues_


WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#19: 
I figured out deadlock could happen if the Rx queue size is smaller. It is 128

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#202: FILE: drivers/net/mlx5/mlx5_rxtx_vec_sse.c:552:
+       uint16_t elts_idx = rxq->rq_ci & q_mask;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#211: FILE: drivers/net/mlx5/mlx5_rxtx_vec_sse.c:571:
+       elts_idx = rxq->rq_ci & q_mask;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#212: FILE: drivers/net/mlx5/mlx5_rxtx_vec_sse.c:572:
+       for (i = 0; i < MLX5_VPMD_DESCS_PER_LOOP; i += 2)$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (7, 15)
#212: FILE: drivers/net/mlx5/mlx5_rxtx_vec_sse.c:572:
+       for (i = 0; i < MLX5_VPMD_DESCS_PER_LOOP; i += 2)
+               _mm_storeu_si128((__m128i *)&(*rxq->elts)[elts_idx + i],

ERROR:CODE_INDENT: code indent should use tabs where possible
#213: FILE: drivers/net/mlx5/mlx5_rxtx_vec_sse.c:573:
+               _mm_storeu_si128((__m128i *)&(*rxq->elts)[elts_idx + i],$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#213: FILE: drivers/net/mlx5/mlx5_rxtx_vec_sse.c:573:
+               _mm_storeu_si128((__m128i *)&(*rxq->elts)[elts_idx + i],$

ERROR:CODE_INDENT: code indent should use tabs where possible
#214: FILE: drivers/net/mlx5/mlx5_rxtx_vec_sse.c:574:
+                                _mm_set1_epi64x((uintptr_t)&rxq->fake_mbuf));$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#214: FILE: drivers/net/mlx5/mlx5_rxtx_vec_sse.c:574:
+                                _mm_set1_epi64x((uintptr_t)&rxq->fake_mbuf));$

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

total: 3 errors, 7 warnings, 0 checks, 19 lines checked

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2017-10-06 22:32 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <374F8C13-CFB0-42FD-8993-BF7F0401F891@mellanox.com>
2017-10-06 22:32 ` [dpdk-test-report] |WARNING| pw29848 [PATCH] Mellanox ConnectX-5 crashes and mbuf leak checkpatch

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).