From: sys_stv@intel.com
To: nikita@elyzion.net, test-report@dpdk.org
Subject: [dpdk-test-report] [PatchWork]|ERROR| pw13656-13659 test: make all lpm routes be of unsigned type
Date: 15 Jun 2016 18:50:38 -0700 [thread overview]
Message-ID: <9c8bee$tek8jr@fmsmga001.fm.intel.com> (raw)
[-- 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
reply other threads:[~2016-06-16 1:50 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='9c8bee$tek8jr@fmsmga001.fm.intel.com' \
--to=sys_stv@intel.com \
--cc=nikita@elyzion.net \
--cc=test-report@dpdk.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).