[dpdk-test-report] [PatchWork]|ERROR| pw12685 app/test: isolate lpm performance cases

[dpdk-test-report] [PatchWork]|ERROR| pw12685 app/test: isolate lpm performance cases

[sys_stv]

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


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

Patchwork ID: 12685
http://www.dpdk.org/dev/patchwork/patch/12685/
Submitter: Thomas Monjalon <thomas.monjalon@6wind.com>
Date: Wed, 11 May 2016 14:16:51 +0200
DPDK git baseline: 213af31e0960e5086bce280364b2eb0d9c6cd261

Check patch error:
12685: 
WARNING: unnecessary whitespace before a quoted newline
#577: FILE: app/test/test_lpm6_perf.c:49:
+printf("Error at line %d: 
", __LINE__);                     
ERROR: space required before the open parenthesis '('
#580: FILE: app/test/test_lpm6_perf.c:52:
+} while(0)

WARNING: unnecessary whitespace before a quoted newline
#591: FILE: app/test/test_lpm6_perf.c:63:
+printf("Route distribution per prefix width: 
");

WARNING: unnecessary whitespace before a quoted newline
#593: FILE: app/test/test_lpm6_perf.c:65:
+printf("--------------------------- 
");

ERROR: space required before the open parenthesis '('
#596: FILE: app/test/test_lpm6_perf.c:68:
+for(i = 1; i <= 128; i++) {

WARNING: line over 80 characters
#629: FILE: app/test/test_lpm6_perf.c:101:
+print_route_distribution(large_route_table, (uint32_t) 
+NUM_ROUTE_ENTRIES);

ERROR: space prohibited before that '++' (ctx:WxB)
#653: FILE: app/test/test_lpm6_perf.c:125:
+for (i = 0; i < ITERATIONS; i ++) {
                              ^

  ERROR: space prohibited before that '++' (ctx:WxB)
  #656: FILE: app/test/test_lpm6_perf.c:128:
  +for (j = 0; j < NUM_IPS_ENTRIES; j ++) {
                                   ^

   ERROR: space prohibited before that '++' (ctx:WxB)
   #679: FILE: app/test/test_lpm6_perf.c:151:
   +for (i = 0; i < ITERATIONS; i ++) {
                              ^

  WARNING: line over 80 characters
  #683: FILE: app/test/test_lpm6_perf.c:155:
  +rte_lpm6_lookup_bulk_func(lpm, ip_batch, next_hops, NUM_IPS_ENTRIES);

  WARNING: unnecessary whitespace before a quoted newline
  #774: FILE: app/test/test_lpm_perf.c:49:
  +printf("Error at line %d: 
", __LINE__);                     
  ERROR: space required before the open parenthesis '('
  #777: FILE: app/test/test_lpm_perf.c:52:
  +} while(0)

  WARNING: unnecessary whitespace before a quoted newline
  #788: FILE: app/test/test_lpm_perf.c:63:
  +printf("Route distribution per prefix width: 
");

  WARNING: unnecessary whitespace before a quoted newline
  #790: FILE: app/test/test_lpm_perf.c:65:
  +printf("--------------------------- 
");

  WARNING: line over 80 characters
  #827: FILE: app/test/test_lpm_perf.c:102:
  +print_route_distribution(large_route_table, (uint32_t) 
  +NUM_ROUTE_ENTRIES);

  WARNING: line over 80 characters
  #861: FILE: app/test/test_lpm_perf.c:136:
  +(unsigned) cache_line_counter, (unsigned) cache_line_counter * 64);

  WARNING: line over 80 characters
  #880: FILE: app/test/test_lpm_perf.c:155:
  +if (rte_lpm_lookup(lpm, ip_batch[j], &next_hop_return) != 0)

  WARNING: line over 80 characters
  #906: FILE: app/test/test_lpm_perf.c:181:
  +rte_lpm_lookup_bulk(lpm, &ip_batch[j], next_hops, BULK_SIZE);

  WARNING: line over 80 characters
  #908: FILE: app/test/test_lpm_perf.c:183:
  +if (unlikely(!(next_hops[k] & RTE_LPM_LOOKUP_SUCCESS)))

  total: 6 errors, 13 warnings, 904 lines checked

  /home/patchWorkOrg/patches/dpdk-dev-v2-app-test-isolate-lpm-performance-cases.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
  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-clang: compile pass
  x86_64-native-linuxapp-gcc-shared: compile pass
  x86_64-native-linuxapp-gcc-debug: compile pass



  DPDK STV team