[dpdk-test-report] |WARNING| pw61441 [PATCH v5 2/3] test/ring: add test cases for configurable element size ring

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Oct 17 22:09:10 CEST 2019


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

_coding style issues_


WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#95: FILE: app/test/test_ring_perf_elem.c:34:
+static const volatile unsigned bulk_sizes[] = { 8, 32 };

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#98: FILE: app/test/test_ring_perf_elem.c:37:
+	unsigned c1, c2;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#101: FILE: app/test/test_ring_perf_elem.c:40:
+static volatile unsigned lcore_count;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#108: FILE: app/test/test_ring_perf_elem.c:47:
+	unsigned id1, id2;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#109: FILE: app/test/test_ring_perf_elem.c:48:
+	unsigned c1, c2, s1, s2;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#136: FILE: app/test/test_ring_perf_elem.c:75:
+	unsigned id1, id2;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#137: FILE: app/test/test_ring_perf_elem.c:76:
+	unsigned c1, c2, s1, s2;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#160: FILE: app/test/test_ring_perf_elem.c:99:
+	unsigned id1, id2;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#161: FILE: app/test/test_ring_perf_elem.c:100:
+	unsigned s1, s2;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#182: FILE: app/test/test_ring_perf_elem.c:121:
+	const unsigned iter_shift = 26;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#183: FILE: app/test/test_ring_perf_elem.c:122:
+	const unsigned iterations = 1<<iter_shift;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#184: FILE: app/test/test_ring_perf_elem.c:123:
+	unsigned i = 0;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#209: FILE: app/test/test_ring_perf_elem.c:148:
+	unsigned size;        /* input value, the burst size */

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#220: FILE: app/test/test_ring_perf_elem.c:159:
+	const unsigned iter_shift = 23;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#221: FILE: app/test/test_ring_perf_elem.c:160:
+	const unsigned iterations = 1<<iter_shift;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#224: FILE: app/test/test_ring_perf_elem.c:163:
+	const unsigned size = params->size;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#225: FILE: app/test/test_ring_perf_elem.c:164:
+	unsigned i;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#262: FILE: app/test/test_ring_perf_elem.c:201:
+	const unsigned iter_shift = 23;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#263: FILE: app/test/test_ring_perf_elem.c:202:
+	const unsigned iterations = 1<<iter_shift;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#266: FILE: app/test/test_ring_perf_elem.c:205:
+	const unsigned size = params->size;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#267: FILE: app/test/test_ring_perf_elem.c:206:
+	unsigned i;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#306: FILE: app/test/test_ring_perf_elem.c:245:
+	unsigned i;

WARNING:ARRAY_SIZE: Prefer ARRAY_SIZE(bulk_sizes)
#307: FILE: app/test/test_ring_perf_elem.c:246:
+	for (i = 0; i < sizeof(bulk_sizes)/sizeof(bulk_sizes[0]); i++) {

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#335: FILE: app/test/test_ring_perf_elem.c:274:
+	const unsigned iter_shift = 24;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#336: FILE: app/test/test_ring_perf_elem.c:275:
+	const unsigned iterations = 1<<iter_shift;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#337: FILE: app/test/test_ring_perf_elem.c:276:
+	unsigned i = 0;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#368: FILE: app/test/test_ring_perf_elem.c:307:
+	const unsigned iter_shift = 23;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#369: FILE: app/test/test_ring_perf_elem.c:308:
+	const unsigned iterations = 1<<iter_shift;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#370: FILE: app/test/test_ring_perf_elem.c:309:
+	unsigned sz, i = 0;

WARNING:ARRAY_SIZE: Prefer ARRAY_SIZE(bulk_sizes)
#373: FILE: app/test/test_ring_perf_elem.c:312:
+	for (sz = 0; sz < sizeof(bulk_sizes)/sizeof(bulk_sizes[0]); sz++) {

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#408: FILE: app/test/test_ring_perf_elem.c:347:
+	const unsigned iter_shift = 23;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#409: FILE: app/test/test_ring_perf_elem.c:348:
+	const unsigned iterations = 1<<iter_shift;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#410: FILE: app/test/test_ring_perf_elem.c:349:
+	unsigned sz, i = 0;

WARNING:ARRAY_SIZE: Prefer ARRAY_SIZE(bulk_sizes)
#413: FILE: app/test/test_ring_perf_elem.c:352:
+	for (sz = 0; sz < sizeof(bulk_sizes)/sizeof(bulk_sizes[0]); sz++) {

total: 0 errors, 34 warnings, 433 lines checked


More information about the test-report mailing list