[dpdk-test-report] [PatchWork]|ERROR| pw13656-13659 test: make all lpm routes be of unsigned type

[dpdk-test-report] [PatchWork]|ERROR| pw13656-13659 test: make all lpm routes be of unsigned type

[sys_stv]

[-- Attachment #1: Type: text/plain, Size: 3497 bytes --]


Test-Label: Intel Niantic on Fedora
Test-Status: ERROR

Patchwork ID: 13656-13659
http://www.dpdk.org/dev/patchwork/patch/13659/
Submitter: Nikita Kozlov <nikita@elyzion.net>
Date: Tue, 14 Jun 2016 15:03:27 +0200
DPDK git baseline: 3a4c1dc58bc7beb67e9ca8c186b6278fb0bc5f04

Check patch error:
13658: 
WARNING: unnecessary whitespace before a quoted newline
#58: FILE: app/test/test_lpm_perf.c:61:
printf("Error at line %d: 
", __LINE__);                     
WARNING: Avoid unnecessary line continuations
#91: FILE: app/test/test_lpm_perf.c:123:
+large_route_table[num_route_entries].ip = 
WARNING: line over 80 characters
#95: FILE: app/test/test_lpm_perf.c:127:
+printf("Failed parse of %.*s
", 12, lpm_data);

WARNING: Avoid unnecessary line continuations
#100: FILE: app/test/test_lpm_perf.c:132:
+large_route_table[num_route_entries].depth = 
WARNING: line over 80 characters
#104: FILE: app/test/test_lpm_perf.c:136:
+printf("Failed parse of %.*s
",5, lpm_data);

ERROR: space required after that ',' (ctx:VxV)
#104: FILE: app/test/test_lpm_perf.c:136:
+printf("Failed parse of %.*s
",5, lpm_data);
                               ^
   
   WARNING: line over 80 characters
   #127: FILE: app/test/test_lpm_perf.c:168:
   +TEST_ASSERT_SUCCESS(load_large_route_table(), "Error loading lpm 
   +table");
   
   WARNING: line over 80 characters
   #131: FILE: app/test/test_lpm_perf.c:171:
   +print_route_distribution(large_route_table, (uint32_t) 
   +num_route_entries);
   
   total: 1 errors, 7 warnings, 135 lines checked
   
   /home/patchWorkOrg/patches/dpdk-dev-4-4-test-change-lpm-perf-test-to-use-routes-as-resource.patch has style problems, please review.
   
   If any of these errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS.
   
   13659: 
   WARNING: unnecessary whitespace before a quoted newline
   #56: FILE: app/test/test_lpm.c:59:
printf("Error at line %d: 
", __LINE__);                     
WARNING: Avoid unnecessary line continuations
#90: FILE: app/test/test_lpm.c:1262:
+large_route_table[num_route_entries].ip = 
WARNING: line over 80 characters
#94: FILE: app/test/test_lpm.c:1266:
+printf("Failed parse of %.*s
", 12, lpm_data);

WARNING: Avoid unnecessary line continuations
#99: FILE: app/test/test_lpm.c:1271:
+large_route_table[num_route_entries].depth = 
WARNING: line over 80 characters
#103: FILE: app/test/test_lpm.c:1275:
+printf("Failed parse of %.*s
",5, lpm_data);

ERROR: space required after that ',' (ctx:VxV)
#103: FILE: app/test/test_lpm.c:1275:
+printf("Failed parse of %.*s
",5, lpm_data);
                            ^

WARNING: line over 80 characters
#124: FILE: app/test/test_lpm.c:1299:
+TEST_ASSERT_SUCCESS(load_large_route_table(), "Error loading lpm 
+table");

total: 1 errors, 6 warnings, 97 lines checked

/home/patchWorkOrg/patches/dpdk-dev-3-4-test-change-lpm-test-to-use-routes-as-resource.patch has style problems, please review.

If any of these errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS.


Compilation:
OS: fedora
Nic: niantic
GCC: gcc_x86-64, 4.8.3
ICC:16.0.2
i686-native-linuxapp-icc: compile pass
x86_64-native-linuxapp-gcc-combined: compile pass
i686-native-linuxapp-gcc: compile pass
x86_64-native-linuxapp-gcc: compile pass
x86_64-native-linuxapp-icc: compile pass
x86_64-native-linuxapp-gcc-debug: compile pass
x86_64-native-linuxapp-gcc-shared: compile pass
x86_64-native-linuxapp-clang: compile pass



DPDK STV team