* [dpdk-test-report] [PatchWork]|ERROR| pw13307-13326 thunderx/nicvf/base: add hardware API for ThunderX nicvf inbuilt NIC
@ 2016-06-12 2:13 sys_stv
0 siblings, 0 replies; only message in thread
From: sys_stv @ 2016-06-12 2:13 UTC (permalink / raw)
To: jerin.jacob, test-report
[-- Attachment #1: Type: text/plain, Size: 6313 bytes --]
Test-Label: Intel Niantic on Fedora
Test-Status: ERROR
Patchwork ID: 13307-13326
http://www.dpdk.org/dev/patchwork/patch/13326/
Submitter: Jerin Jacob <jerin.jacob@caviumnetworks.com>
Date: Tue, 7 Jun 2016 22:10:13 +0530
DPDK git baseline: ca173a909538a2f1082cd0dcb4d778a97dab69c3
Check patch error:
13307:
WARNING: networking block comments don't use an empty /* line, use /* Comment...
#53: FILE: drivers/net/thunderx/base/nicvf_hw.c:2:
+/*
+ * BSD LICENSE
ERROR: return is not a function, parentheses are not required
#320: FILE: drivers/net/thunderx/base/nicvf_hw.c:269:
+return (*(const uint32_t *)a - *(const uint32_t *)b);
WARNING: networking block comments don't use an empty /* line, use /* Comment...
#394: FILE: drivers/net/thunderx/base/nicvf_hw.c:343:
+/*
+ * Handle poll mode driver interested "mbox" and "queue-set error" interrupts.
WARNING: networking block comments don't use an empty /* line, use /* Comment...
#967: FILE: drivers/net/thunderx/base/nicvf_hw.h:2:
+/*
+ * BSD LICENSE
WARNING: networking block comments don't use an empty /* line, use /* Comment...
#1213: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:2:
+/*
+ * BSD LICENSE
WARNING: do not add new typedefs
#1750: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:539:
+typedef uint64_t nicvf_phys_addr_t;
WARNING: do not add new typedefs
#1761: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:550:
+typedef union {
WARNING: do not add new typedefs
#1806: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:595:
+typedef union {
WARNING: do not add new typedefs
#1831: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:620:
+typedef union {
WARNING: do not add new typedefs
#1848: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:637:
+typedef union {
WARNING: do not add new typedefs
#1865: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:654:
+typedef union {
WARNING: do not add new typedefs
#1882: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:671:
+typedef union {
WARNING: do not add new typedefs
#1899: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:688:
+typedef union {
ERROR: spaces prohibited around that ':' (ctx:VxW)
#2336: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:1125:
+uint64_t ena: 1;
^
WARNING: networking block comments don't use an empty /* line, use /* Comment...
#2435: FILE: drivers/net/thunderx/base/nicvf_mbox.c:2:
+/*
+ * BSD LICENSE
WARNING: networking block comments don't use an empty /* line, use /* Comment...
#2628: FILE: drivers/net/thunderx/base/nicvf_mbox.c:195:
+/*
+ * Checks if VF is able to communicate with PF
WARNING: networking block comments don't use an empty /* line, use /* Comment...
#2857: FILE: drivers/net/thunderx/base/nicvf_mbox.h:2:
+/*
+ * BSD LICENSE
WARNING: networking block comments don't use an empty /* line, use /* Comment...
#3095: FILE: drivers/net/thunderx/base/nicvf_plat.h:2:
+/*
+ * BSD LICENSE
ERROR: space prohibited before open square bracket '['
#3176: FILE: drivers/net/thunderx/base/nicvf_plat.h:83:
+ : [val] "r" (val), [addr] "r" (addr));
ERROR: space prohibited before open square bracket '['
#3186: FILE: drivers/net/thunderx/base/nicvf_plat.h:93:
+ : [val] "=r" (val)
ERROR: space prohibited before open square bracket '['
#3187: FILE: drivers/net/thunderx/base/nicvf_plat.h:94:
+ : [addr] "r" (addr));
ERROR: space prohibited before open square bracket '['
#3194: FILE: drivers/net/thunderx/base/nicvf_plat.h:101:
+: [x1]"=r"(reg1), [x0]"=r"(reg2)
ERROR: space prohibited before open square bracket '['
#3195: FILE: drivers/net/thunderx/base/nicvf_plat.h:102:
+: [p1]"r"(addr)
WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
#3205: FILE: drivers/net/thunderx/base/nicvf_plat.h:112:
+*(volatile uint64_t *)addr = val;
WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
#3211: FILE: drivers/net/thunderx/base/nicvf_plat.h:118:
+return*(volatile uint64_t *)addr;
total: 7 errors, 18 warnings, 3144 lines checked
/home/patchWorkOrg/patches/dpdk-dev-v3-01-20-thunderx-nicvf-base-add-hardware-API-for-ThunderX-nicvf-inbuilt-NIC.patch has style problems, please review.
If any of these errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS.
13318:
WARNING: networking block comments don't use an empty /* line, use /* Comment...
#83: FILE: drivers/net/thunderx/nicvf_rxtx.c:2:
+/*
+ * BSD LICENSE
ERROR: return is not a function, parentheses are not required
#221: FILE: drivers/net/thunderx/nicvf_rxtx.c:140:
+return ((sq->head - sq->tail - 1) & sq->qlen_mask);
WARNING: networking block comments don't use an empty /* line, use /* Comment...
#345: FILE: drivers/net/thunderx/nicvf_rxtx.h:2:
+/*
+ * BSD LICENSE
ERROR: code indent should use tabs where possible
#422: FILE: drivers/net/thunderx/nicvf_rxtx.h:79:
+ entry->buff[0] = (uint64_t)SQ_DESC_TYPE_GATHER << 60 |$
WARNING: please, no spaces at the start of a line
#422: FILE: drivers/net/thunderx/nicvf_rxtx.h:79:
+ entry->buff[0] = (uint64_t)SQ_DESC_TYPE_GATHER << 60 |$
ERROR: code indent should use tabs where possible
#425: FILE: drivers/net/thunderx/nicvf_rxtx.h:82:
+ entry->buff[1] = rte_mbuf_data_dma_addr(pkt);$
WARNING: please, no spaces at the start of a line
#425: FILE: drivers/net/thunderx/nicvf_rxtx.h:82:
+ entry->buff[1] = rte_mbuf_data_dma_addr(pkt);$
total: 3 errors, 4 warnings, 378 lines checked
NOTE: whitespace errors detected, you may wish to use scripts/cleanpatch or
scripts/cleanfile
/home/patchWorkOrg/patches/dpdk-dev-v3-12-20-thunderx-nicvf-add-single-and-multi-segment-tx-functions.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
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2016-06-12 2:13 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-12 2:13 [dpdk-test-report] [PatchWork]|ERROR| pw13307-13326 thunderx/nicvf/base: add hardware API for ThunderX nicvf inbuilt NIC sys_stv
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).