[dpdk-test-report] |WARNING| pw60720 [PATCH v3 2/2] test/ring: add test cases for configurable element size ring

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Oct 8 21:20:17 CEST 2019


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

_coding style issues_


WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#94: 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'
#97: FILE: app/test/test_ring_perf_elem.c:37:
+	unsigned c1, c2;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#100: 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'
#107: FILE: app/test/test_ring_perf_elem.c:47:
+	unsigned id1, id2;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#108: 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'
#135: FILE: app/test/test_ring_perf_elem.c:75:
+	unsigned id1, id2;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#136: 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'
#159: FILE: app/test/test_ring_perf_elem.c:99:
+	unsigned id1, id2;

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

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#181: 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'
#182: 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'
#183: FILE: app/test/test_ring_perf_elem.c:123:
+	unsigned i = 0;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#208: 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'
#219: 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'
#220: 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'
#223: 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'
#224: FILE: app/test/test_ring_perf_elem.c:164:
+	unsigned i;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#261: 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'
#262: 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'
#265: 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'
#266: FILE: app/test/test_ring_perf_elem.c:206:
+	unsigned i;

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

WARNING:ARRAY_SIZE: Prefer ARRAY_SIZE(bulk_sizes)
#306: 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'
#334: 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'
#335: 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'
#336: FILE: app/test/test_ring_perf_elem.c:276:
+	unsigned i = 0;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#367: 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'
#368: 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'
#369: FILE: app/test/test_ring_perf_elem.c:309:
+	unsigned sz, i = 0;

WARNING:ARRAY_SIZE: Prefer ARRAY_SIZE(bulk_sizes)
#372: 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'
#407: 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'
#408: 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'
#409: FILE: app/test/test_ring_perf_elem.c:349:
+	unsigned sz, i = 0;

WARNING:ARRAY_SIZE: Prefer ARRAY_SIZE(bulk_sizes)
#412: 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