Test-Label: Intel Niantic on Fedora Test-Status: ERROR Patchwork ID: 15067 http://www.dpdk.org/dev/patchwork/patch/15067/ Submitter: Steffen.Bauch@rohde-schwarz.com Date: Mon, 1 Aug 2016 11:21:04 +0200Mon, 25 Jul 2016 16:13:02 +0200 DPDK git baseline: b0a1419a88efebac3a1d52987b7e634848f9c85e Check patch error: 15067: WARNING: please, no spaces at the start of a line #47: FILE: lib/librte_eal/linuxapp/eal/eal.c:696: + int fctret;$ WARNING: please, no spaces at the start of a line #59: FILE: lib/librte_eal/linuxapp/eal/eal.c:700: + logid = NULL;$ WARNING: suspect code indent for conditional statements (7, 15) #61: FILE: lib/librte_eal/linuxapp/eal/eal.c:702: if (rte_eal_log_early_init() < 0) + return -1; ERROR: code indent should use tabs where possible #63: FILE: lib/librte_eal/linuxapp/eal/eal.c:703: + return -1;$ WARNING: please, no spaces at the start of a line #63: FILE: lib/librte_eal/linuxapp/eal/eal.c:703: + return -1;$ WARNING: suspect code indent for conditional statements (7, 15) #65: FILE: lib/librte_eal/linuxapp/eal/eal.c:705: if (rte_eal_cpu_init() < 0) + return -1; ERROR: code indent should use tabs where possible #67: FILE: lib/librte_eal/linuxapp/eal/eal.c:706: + return -1;$ WARNING: please, no spaces at the start of a line #67: FILE: lib/librte_eal/linuxapp/eal/eal.c:706: + return -1;$ ERROR: patch seems to be corrupt (line wrapped?) #73: FILE: lib/librte_eal/linuxapp/eal/eal.c:716: && ERROR: code indent should use tabs where possible #77: FILE: lib/librte_eal/linuxapp/eal/eal.c:719: + return -1;$ WARNING: please, no spaces at the start of a line #77: FILE: lib/librte_eal/linuxapp/eal/eal.c:719: + return -1;$ WARNING: suspect code indent for conditional statements (7, 15) #85: FILE: lib/librte_eal/linuxapp/eal/eal.c:743: if (rte_eal_pci_init() < 0) + return -1; ERROR: code indent should use tabs where possible #87: FILE: lib/librte_eal/linuxapp/eal/eal.c:744: + return -1;$ WARNING: please, no spaces at the start of a line #87: FILE: lib/librte_eal/linuxapp/eal/eal.c:744: + return -1;$ WARNING: suspect code indent for conditional statements (7, 15) #90: FILE: lib/librte_eal/linuxapp/eal/eal.c:747: if (rte_eal_ivshmem_init() < 0) + return -1; ERROR: code indent should use tabs where possible #92: FILE: lib/librte_eal/linuxapp/eal/eal.c:748: + return -1;$ WARNING: please, no spaces at the start of a line #92: FILE: lib/librte_eal/linuxapp/eal/eal.c:748: + return -1;$ WARNING: suspect code indent for conditional statements (7, 15) #95: FILE: lib/librte_eal/linuxapp/eal/eal.c:751: if (rte_eal_memory_init() < 0) + return -1; ERROR: code indent should use tabs where possible #97: FILE: lib/librte_eal/linuxapp/eal/eal.c:752: + return -1;$ WARNING: please, no spaces at the start of a line #97: FILE: lib/librte_eal/linuxapp/eal/eal.c:752: + return -1;$ WARNING: suspect code indent for conditional statements (7, 15) #102: FILE: lib/librte_eal/linuxapp/eal/eal.c:757: if (rte_eal_memzone_init() < 0) + return -1; ERROR: code indent should use tabs where possible #104: FILE: lib/librte_eal/linuxapp/eal/eal.c:758: + return -1;$ WARNING: please, no spaces at the start of a line #104: FILE: lib/librte_eal/linuxapp/eal/eal.c:758: + return -1;$ WARNING: suspect code indent for conditional statements (7, 15) #106: FILE: lib/librte_eal/linuxapp/eal/eal.c:760: if (rte_eal_tailqs_init() < 0) + return -1; ERROR: code indent should use tabs where possible #108: FILE: lib/librte_eal/linuxapp/eal/eal.c:761: + return -1;$ WARNING: please, no spaces at the start of a line #108: FILE: lib/librte_eal/linuxapp/eal/eal.c:761: + return -1;$ WARNING: suspect code indent for conditional statements (7, 15) #111: FILE: lib/librte_eal/linuxapp/eal/eal.c:764: if (rte_eal_ivshmem_obj_init() < 0) + return -1; ERROR: code indent should use tabs where possible #113: FILE: lib/librte_eal/linuxapp/eal/eal.c:765: + return -1;$ WARNING: please, no spaces at the start of a line #113: FILE: lib/librte_eal/linuxapp/eal/eal.c:765: + return -1;$ WARNING: suspect code indent for conditional statements (7, 15) #116: FILE: lib/librte_eal/linuxapp/eal/eal.c:768: if (rte_eal_log_init(logid, internal_config.syslog_facility) < 0) + return -1; ERROR: code indent should use tabs where possible #118: FILE: lib/librte_eal/linuxapp/eal/eal.c:769: + return -1;$ WARNING: please, no spaces at the start of a line #118: FILE: lib/librte_eal/linuxapp/eal/eal.c:769: + return -1;$ WARNING: suspect code indent for conditional statements (7, 15) #120: FILE: lib/librte_eal/linuxapp/eal/eal.c:771: if (rte_eal_alarm_init() < 0) + return -1; ERROR: code indent should use tabs where possible #122: FILE: lib/librte_eal/linuxapp/eal/eal.c:772: + return -1;$ WARNING: please, no spaces at the start of a line #122: FILE: lib/librte_eal/linuxapp/eal/eal.c:772: + return -1;$ ERROR: trailing whitespace #124: FILE: lib/librte_eal/linuxapp/eal/eal.c:774: + /* interrupt handling will be initialized but the thread is $ ERROR: code indent should use tabs where possible #128: FILE: lib/librte_eal/linuxapp/eal/eal.c:776: + return -1;$ ERROR: trailing whitespace #130: FILE: lib/librte_eal/linuxapp/eal/eal.c:778: + /* timer stuff is initialized but hpet should be disabled in $ ERROR: code indent should use tabs where possible #134: FILE: lib/librte_eal/linuxapp/eal/eal.c:780: + return -1;$ ERROR: code indent should use tabs where possible #181: FILE: lib/librte_eal/linuxapp/eal/eal.c:794: + return -1;$ WARNING: please, no spaces at the start of a line #181: FILE: lib/librte_eal/linuxapp/eal/eal.c:794: + return -1;$ WARNING: suspect code indent for conditional statements (7, 15) #184: FILE: lib/librte_eal/linuxapp/eal/eal.c:797: if (rte_eal_pci_probe()) + return -1; ERROR: code indent should use tabs where possible #186: FILE: lib/librte_eal/linuxapp/eal/eal.c:798: + return -1;$ WARNING: please, no spaces at the start of a line #186: FILE: lib/librte_eal/linuxapp/eal/eal.c:798: + return -1;$ ERROR: trailing whitespace #199: FILE: lib/librte_eal/linuxapp/eal/eal_interrupts.c:777: + /* we do not need the interrupt handling and do not want the + extra $ ERROR: trailing whitespace #216: FILE: lib/librte_eal/linuxapp/eal/eal_log.c:94: +rte_eal_log_init(__attribute__((unused)) const char *id, $ ERROR: Missing Signed-off-by: line(s) total: 21 errors, 26 warnings, 184 lines checked NOTE: whitespace errors detected, you may wish to use scripts/cleanpatch or scripts/cleanfile /home/patchWorkOrg/patches/dpdk-dev-Application-framework-vs.-library.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