* [dpdk-test-report] |WARNING| pw29187 [PATCH v3 1/3] eal/x86: run-time dispatch over memcpy
[not found] <1506411689-94690-2-git-send-email-xiaoyun.li@intel.com>
@ 2017-09-26 7:44 ` checkpatch
0 siblings, 0 replies; only message in thread
From: checkpatch @ 2017-09-26 7:44 UTC (permalink / raw)
To: test-report; +Cc: Xiaoyun Li
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/29187
_coding style issues_
WARNING:LONG_LINE_COMMENT: line over 90 characters
#60: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:74:
+ * Macro for copying unaligned block from one location to another with constant load offset,
WARNING:LONG_LINE_COMMENT: line over 90 characters
#66: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:80:
+ * - For <src>, make sure <offset> bit backwards & <16 - offset> bit forwards are available for loading
WARNING:LONG_LINE: line over 90 characters
#70: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:84:
+#define MOVEUNALIGNED_LEFT47_IMM(dst, src, len, offset) \
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#70: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:84:
+#define MOVEUNALIGNED_LEFT47_IMM(dst, src, len, offset) \
+__extension__ ({ \
+ int tmp; \
+ while (len >= 128 + 16 - offset) { \
+ xmm0 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 0 * 16)); \
+ len -= 128; \
+ xmm1 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 1 * 16)); \
+ xmm2 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 2 * 16)); \
+ xmm3 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 3 * 16)); \
+ xmm4 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 4 * 16)); \
+ xmm5 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 5 * 16)); \
+ xmm6 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 6 * 16)); \
+ xmm7 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 7 * 16)); \
+ xmm8 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 8 * 16)); \
+ src = (const uint8_t *)src + 128; \
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 0 * 16), _mm_alignr_epi8(xmm1, xmm0, offset)); \
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 1 * 16), _mm_alignr_epi8(xmm2, xmm1, offset)); \
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 2 * 16), _mm_alignr_epi8(xmm3, xmm2, offset)); \
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 3 * 16), _mm_alignr_epi8(xmm4, xmm3, offset)); \
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 4 * 16), _mm_alignr_epi8(xmm5, xmm4, offset)); \
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 5 * 16), _mm_alignr_epi8(xmm6, xmm5, offset)); \
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 6 * 16), _mm_alignr_epi8(xmm7, xmm6, offset)); \
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 7 * 16), _mm_alignr_epi8(xmm8, xmm7, offset)); \
+ dst = (uint8_t *)dst + 128; \
+ } \
+ tmp = len; \
+ len = ((len - 16 + offset) & 127) + 16 - offset; \
+ tmp -= len; \
+ src = (const uint8_t *)src + tmp; \
+ dst = (uint8_t *)dst + tmp; \
+ if (len >= 32 + 16 - offset) { \
+ while (len >= 32 + 16 - offset) { \
+ xmm0 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 0 * 16)); \
+ len -= 32; \
+ xmm1 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 1 * 16)); \
+ xmm2 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 2 * 16)); \
+ src = (const uint8_t *)src + 32; \
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 0 * 16), _mm_alignr_epi8(xmm1, xmm0, offset)); \
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 1 * 16), _mm_alignr_epi8(xmm2, xmm1, offset)); \
+ dst = (uint8_t *)dst + 32; \
+ } \
+ tmp = len; \
+ len = ((len - 16 + offset) & 31) + 16 - offset; \
+ tmp -= len; \
+ src = (const uint8_t *)src + tmp; \
+ dst = (uint8_t *)dst + tmp; \
+ } \
+})
WARNING:LONG_LINE: line over 90 characters
#71: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:85:
+__extension__ ({ \
WARNING:LONG_LINE: line over 90 characters
#72: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:86:
+ int tmp; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#72: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:86:
+ int tmp; \$
WARNING:LONG_LINE: line over 90 characters
#73: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:87:
+ while (len >= 128 + 16 - offset) { \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#73: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:87:
+ while (len >= 128 + 16 - offset) { \$
WARNING:LONG_LINE: line over 90 characters
#74: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:88:
+ xmm0 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 0 * 16)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#74: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:88:
+ xmm0 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 0 * 16)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#74: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:88:
+ xmm0 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 0 * 16)); \$
WARNING:LONG_LINE: line over 90 characters
#75: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:89:
+ len -= 128; \
ERROR:CODE_INDENT: code indent should use tabs where possible
#75: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:89:
+ len -= 128; \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#75: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:89:
+ len -= 128; \$
WARNING:LONG_LINE: line over 90 characters
#76: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:90:
+ xmm1 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 1 * 16)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#76: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:90:
+ xmm1 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 1 * 16)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#76: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:90:
+ xmm1 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 1 * 16)); \$
WARNING:LONG_LINE: line over 90 characters
#77: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:91:
+ xmm2 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 2 * 16)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#77: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:91:
+ xmm2 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 2 * 16)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#77: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:91:
+ xmm2 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 2 * 16)); \$
WARNING:LONG_LINE: line over 90 characters
#78: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:92:
+ xmm3 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 3 * 16)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#78: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:92:
+ xmm3 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 3 * 16)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#78: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:92:
+ xmm3 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 3 * 16)); \$
WARNING:LONG_LINE: line over 90 characters
#79: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:93:
+ xmm4 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 4 * 16)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#79: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:93:
+ xmm4 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 4 * 16)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#79: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:93:
+ xmm4 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 4 * 16)); \$
WARNING:LONG_LINE: line over 90 characters
#80: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:94:
+ xmm5 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 5 * 16)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#80: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:94:
+ xmm5 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 5 * 16)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#80: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:94:
+ xmm5 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 5 * 16)); \$
WARNING:LONG_LINE: line over 90 characters
#81: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:95:
+ xmm6 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 6 * 16)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#81: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:95:
+ xmm6 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 6 * 16)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#81: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:95:
+ xmm6 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 6 * 16)); \$
WARNING:LONG_LINE: line over 90 characters
#82: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:96:
+ xmm7 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 7 * 16)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#82: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:96:
+ xmm7 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 7 * 16)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#82: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:96:
+ xmm7 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 7 * 16)); \$
WARNING:LONG_LINE: line over 90 characters
#83: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:97:
+ xmm8 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 8 * 16)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#83: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:97:
+ xmm8 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 8 * 16)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#83: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:97:
+ xmm8 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 8 * 16)); \$
WARNING:LONG_LINE: line over 90 characters
#84: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:98:
+ src = (const uint8_t *)src + 128; \
ERROR:CODE_INDENT: code indent should use tabs where possible
#84: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:98:
+ src = (const uint8_t *)src + 128; \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#84: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:98:
+ src = (const uint8_t *)src + 128; \$
WARNING:LONG_LINE: line over 90 characters
#85: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:99:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 0 * 16), _mm_alignr_epi8(xmm1, xmm0, offset)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#85: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:99:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 0 * 16), _mm_alignr_epi8(xmm1, xmm0, offset)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#85: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:99:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 0 * 16), _mm_alignr_epi8(xmm1, xmm0, offset)); \$
WARNING:LONG_LINE: line over 90 characters
#86: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:100:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 1 * 16), _mm_alignr_epi8(xmm2, xmm1, offset)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#86: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:100:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 1 * 16), _mm_alignr_epi8(xmm2, xmm1, offset)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#86: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:100:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 1 * 16), _mm_alignr_epi8(xmm2, xmm1, offset)); \$
WARNING:LONG_LINE: line over 90 characters
#87: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:101:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 2 * 16), _mm_alignr_epi8(xmm3, xmm2, offset)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#87: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:101:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 2 * 16), _mm_alignr_epi8(xmm3, xmm2, offset)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#87: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:101:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 2 * 16), _mm_alignr_epi8(xmm3, xmm2, offset)); \$
WARNING:LONG_LINE: line over 90 characters
#88: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:102:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 3 * 16), _mm_alignr_epi8(xmm4, xmm3, offset)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#88: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:102:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 3 * 16), _mm_alignr_epi8(xmm4, xmm3, offset)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#88: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:102:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 3 * 16), _mm_alignr_epi8(xmm4, xmm3, offset)); \$
WARNING:LONG_LINE: line over 90 characters
#89: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:103:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 4 * 16), _mm_alignr_epi8(xmm5, xmm4, offset)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#89: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:103:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 4 * 16), _mm_alignr_epi8(xmm5, xmm4, offset)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#89: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:103:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 4 * 16), _mm_alignr_epi8(xmm5, xmm4, offset)); \$
WARNING:LONG_LINE: line over 90 characters
#90: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:104:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 5 * 16), _mm_alignr_epi8(xmm6, xmm5, offset)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#90: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:104:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 5 * 16), _mm_alignr_epi8(xmm6, xmm5, offset)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#90: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:104:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 5 * 16), _mm_alignr_epi8(xmm6, xmm5, offset)); \$
WARNING:LONG_LINE: line over 90 characters
#91: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:105:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 6 * 16), _mm_alignr_epi8(xmm7, xmm6, offset)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#91: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:105:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 6 * 16), _mm_alignr_epi8(xmm7, xmm6, offset)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#91: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:105:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 6 * 16), _mm_alignr_epi8(xmm7, xmm6, offset)); \$
WARNING:LONG_LINE: line over 90 characters
#92: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:106:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 7 * 16), _mm_alignr_epi8(xmm8, xmm7, offset)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#92: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:106:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 7 * 16), _mm_alignr_epi8(xmm8, xmm7, offset)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#92: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:106:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 7 * 16), _mm_alignr_epi8(xmm8, xmm7, offset)); \$
WARNING:LONG_LINE: line over 90 characters
#93: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:107:
+ dst = (uint8_t *)dst + 128; \
ERROR:CODE_INDENT: code indent should use tabs where possible
#93: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:107:
+ dst = (uint8_t *)dst + 128; \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#93: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:107:
+ dst = (uint8_t *)dst + 128; \$
WARNING:LONG_LINE: line over 90 characters
#94: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:108:
+ } \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#94: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:108:
+ } \$
WARNING:LONG_LINE: line over 90 characters
#95: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:109:
+ tmp = len; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#95: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:109:
+ tmp = len; \$
WARNING:LONG_LINE: line over 90 characters
#96: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:110:
+ len = ((len - 16 + offset) & 127) + 16 - offset; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#96: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:110:
+ len = ((len - 16 + offset) & 127) + 16 - offset; \$
WARNING:LONG_LINE: line over 90 characters
#97: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:111:
+ tmp -= len; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#97: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:111:
+ tmp -= len; \$
WARNING:LONG_LINE: line over 90 characters
#98: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:112:
+ src = (const uint8_t *)src + tmp; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#98: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:112:
+ src = (const uint8_t *)src + tmp; \$
WARNING:LONG_LINE: line over 90 characters
#99: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:113:
+ dst = (uint8_t *)dst + tmp; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#99: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:113:
+ dst = (uint8_t *)dst + tmp; \$
WARNING:LONG_LINE: line over 90 characters
#100: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:114:
+ if (len >= 32 + 16 - offset) { \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#100: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:114:
+ if (len >= 32 + 16 - offset) { \$
WARNING:LONG_LINE: line over 90 characters
#101: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:115:
+ while (len >= 32 + 16 - offset) { \
ERROR:CODE_INDENT: code indent should use tabs where possible
#101: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:115:
+ while (len >= 32 + 16 - offset) { \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#101: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:115:
+ while (len >= 32 + 16 - offset) { \$
WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#101: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:115:
+ while (len >= 32 + 16 - offset) { \
+ xmm0 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 0 * 16)); \
WARNING:LONG_LINE: line over 90 characters
#102: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:116:
+ xmm0 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 0 * 16)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#102: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:116:
+ xmm0 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 0 * 16)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#102: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:116:
+ xmm0 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 0 * 16)); \$
WARNING:LONG_LINE: line over 90 characters
#103: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:117:
+ len -= 32; \
ERROR:CODE_INDENT: code indent should use tabs where possible
#103: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:117:
+ len -= 32; \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#103: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:117:
+ len -= 32; \$
WARNING:LONG_LINE: line over 90 characters
#104: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:118:
+ xmm1 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 1 * 16)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#104: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:118:
+ xmm1 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 1 * 16)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#104: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:118:
+ xmm1 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 1 * 16)); \$
WARNING:LONG_LINE: line over 90 characters
#105: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:119:
+ xmm2 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 2 * 16)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#105: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:119:
+ xmm2 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 2 * 16)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#105: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:119:
+ xmm2 = _mm_loadu_si128((const __m128i *)((const uint8_t *)src - offset + 2 * 16)); \$
WARNING:LONG_LINE: line over 90 characters
#106: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:120:
+ src = (const uint8_t *)src + 32; \
ERROR:CODE_INDENT: code indent should use tabs where possible
#106: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:120:
+ src = (const uint8_t *)src + 32; \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#106: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:120:
+ src = (const uint8_t *)src + 32; \$
WARNING:LONG_LINE: line over 90 characters
#107: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:121:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 0 * 16), _mm_alignr_epi8(xmm1, xmm0, offset)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#107: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:121:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 0 * 16), _mm_alignr_epi8(xmm1, xmm0, offset)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#107: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:121:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 0 * 16), _mm_alignr_epi8(xmm1, xmm0, offset)); \$
WARNING:LONG_LINE: line over 90 characters
#108: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:122:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 1 * 16), _mm_alignr_epi8(xmm2, xmm1, offset)); \
ERROR:CODE_INDENT: code indent should use tabs where possible
#108: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:122:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 1 * 16), _mm_alignr_epi8(xmm2, xmm1, offset)); \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#108: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:122:
+ _mm_storeu_si128((__m128i *)((uint8_t *)dst + 1 * 16), _mm_alignr_epi8(xmm2, xmm1, offset)); \$
WARNING:LONG_LINE: line over 90 characters
#109: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:123:
+ dst = (uint8_t *)dst + 32; \
ERROR:CODE_INDENT: code indent should use tabs where possible
#109: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:123:
+ dst = (uint8_t *)dst + 32; \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#109: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:123:
+ dst = (uint8_t *)dst + 32; \$
WARNING:LONG_LINE: line over 90 characters
#110: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:124:
+ } \
ERROR:CODE_INDENT: code indent should use tabs where possible
#110: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:124:
+ } \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#110: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:124:
+ } \$
WARNING:LONG_LINE: line over 90 characters
#111: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:125:
+ tmp = len; \
ERROR:CODE_INDENT: code indent should use tabs where possible
#111: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:125:
+ tmp = len; \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#111: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:125:
+ tmp = len; \$
WARNING:LONG_LINE: line over 90 characters
#112: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:126:
+ len = ((len - 16 + offset) & 31) + 16 - offset; \
ERROR:CODE_INDENT: code indent should use tabs where possible
#112: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:126:
+ len = ((len - 16 + offset) & 31) + 16 - offset; \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#112: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:126:
+ len = ((len - 16 + offset) & 31) + 16 - offset; \$
WARNING:LONG_LINE: line over 90 characters
#113: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:127:
+ tmp -= len; \
ERROR:CODE_INDENT: code indent should use tabs where possible
#113: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:127:
+ tmp -= len; \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#113: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:127:
+ tmp -= len; \$
WARNING:LONG_LINE: line over 90 characters
#114: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:128:
+ src = (const uint8_t *)src + tmp; \
ERROR:CODE_INDENT: code indent should use tabs where possible
#114: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:128:
+ src = (const uint8_t *)src + tmp; \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#114: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:128:
+ src = (const uint8_t *)src + tmp; \$
WARNING:LONG_LINE: line over 90 characters
#115: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:129:
+ dst = (uint8_t *)dst + tmp; \
ERROR:CODE_INDENT: code indent should use tabs where possible
#115: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:129:
+ dst = (uint8_t *)dst + tmp; \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#115: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:129:
+ dst = (uint8_t *)dst + tmp; \$
WARNING:LONG_LINE: line over 90 characters
#116: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:130:
+ } \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#116: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:130:
+ } \$
WARNING:LONG_LINE_COMMENT: line over 90 characters
#123: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:137:
+ * Use switch here because the aligning instruction requires immediate value for shift count.
WARNING:LONG_LINE_COMMENT: line over 90 characters
#127: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:141:
+ * - For <src>, make sure <offset> bit backwards & <16 - offset> bit forwards are available for loading
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#131: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:145:
+#define MOVEUNALIGNED_LEFT47(dst, src, len, offset) \
+__extension__ ({ \
+ switch (offset) { \
+ case 0x01: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x01); break; \
+ case 0x02: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x02); break; \
+ case 0x03: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x03); break; \
+ case 0x04: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x04); break; \
+ case 0x05: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x05); break; \
+ case 0x06: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x06); break; \
+ case 0x07: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x07); break; \
+ case 0x08: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x08); break; \
+ case 0x09: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x09); break; \
+ case 0x0A: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0A); break; \
+ case 0x0B: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0B); break; \
+ case 0x0C: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0C); break; \
+ case 0x0D: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0D); break; \
+ case 0x0E: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0E); break; \
+ case 0x0F: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0F); break; \
+ default:; \
+ } \
+})
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#133: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:147:
+ switch (offset) { \$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#134: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:148:
+ case 0x01: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x01); break; \$
ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#134: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:148:
+ case 0x01: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x01); break; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#135: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:149:
+ case 0x02: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x02); break; \$
ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#135: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:149:
+ case 0x02: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x02); break; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#136: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:150:
+ case 0x03: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x03); break; \$
ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#136: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:150:
+ case 0x03: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x03); break; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#137: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:151:
+ case 0x04: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x04); break; \$
ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#137: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:151:
+ case 0x04: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x04); break; \
WARNING:MISSING_BREAK: Possible switch case/default not preceded by break or fallthrough comment
#137: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:151:
+ case 0x04: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x04); break; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#138: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:152:
+ case 0x05: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x05); break; \$
ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#138: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:152:
+ case 0x05: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x05); break; \
WARNING:MISSING_BREAK: Possible switch case/default not preceded by break or fallthrough comment
#138: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:152:
+ case 0x05: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x05); break; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#139: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:153:
+ case 0x06: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x06); break; \$
ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#139: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:153:
+ case 0x06: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x06); break; \
WARNING:MISSING_BREAK: Possible switch case/default not preceded by break or fallthrough comment
#139: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:153:
+ case 0x06: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x06); break; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#140: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:154:
+ case 0x07: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x07); break; \$
ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#140: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:154:
+ case 0x07: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x07); break; \
WARNING:MISSING_BREAK: Possible switch case/default not preceded by break or fallthrough comment
#140: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:154:
+ case 0x07: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x07); break; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#141: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:155:
+ case 0x08: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x08); break; \$
ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#141: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:155:
+ case 0x08: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x08); break; \
WARNING:MISSING_BREAK: Possible switch case/default not preceded by break or fallthrough comment
#141: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:155:
+ case 0x08: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x08); break; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#142: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:156:
+ case 0x09: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x09); break; \$
ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#142: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:156:
+ case 0x09: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x09); break; \
WARNING:MISSING_BREAK: Possible switch case/default not preceded by break or fallthrough comment
#142: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:156:
+ case 0x09: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x09); break; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#143: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:157:
+ case 0x0A: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0A); break; \$
ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#143: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:157:
+ case 0x0A: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0A); break; \
WARNING:MISSING_BREAK: Possible switch case/default not preceded by break or fallthrough comment
#143: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:157:
+ case 0x0A: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0A); break; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#144: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:158:
+ case 0x0B: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0B); break; \$
ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#144: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:158:
+ case 0x0B: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0B); break; \
WARNING:MISSING_BREAK: Possible switch case/default not preceded by break or fallthrough comment
#144: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:158:
+ case 0x0B: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0B); break; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#145: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:159:
+ case 0x0C: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0C); break; \$
ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#145: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:159:
+ case 0x0C: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0C); break; \
WARNING:MISSING_BREAK: Possible switch case/default not preceded by break or fallthrough comment
#145: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:159:
+ case 0x0C: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0C); break; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#146: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:160:
+ case 0x0D: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0D); break; \$
ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#146: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:160:
+ case 0x0D: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0D); break; \
WARNING:MISSING_BREAK: Possible switch case/default not preceded by break or fallthrough comment
#146: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:160:
+ case 0x0D: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0D); break; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#147: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:161:
+ case 0x0E: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0E); break; \$
ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#147: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:161:
+ case 0x0E: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0E); break; \
WARNING:MISSING_BREAK: Possible switch case/default not preceded by break or fallthrough comment
#147: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:161:
+ case 0x0E: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0E); break; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#148: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:162:
+ case 0x0F: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0F); break; \$
ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#148: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:162:
+ case 0x0F: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0F); break; \
WARNING:MISSING_BREAK: Possible switch case/default not preceded by break or fallthrough comment
#148: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:162:
+ case 0x0F: MOVEUNALIGNED_LEFT47_IMM(dst, src, n, 0x0F); break; \
ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#149: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:163:
+ default:; \
WARNING:MISSING_BREAK: Possible switch case/default not preceded by break or fallthrough comment
#149: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:163:
+ default:; \
WARNING:DEFAULT_NO_BREAK: switch default: should use break
#149: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:163:
+ default:; \
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#150: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:164:
+ } \$
WARNING:UNNECESSARY_ELSE: else is not generally useful after a break or return
#344: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:976:
+ return ret;
+ } else {
WARNING:UNNECESSARY_ELSE: else is not generally useful after a break or return
#635: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:1267:
+ return ret;
+ } else {
WARNING:UNNECESSARY_ELSE: else is not generally useful after a break or return
#914: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:1546:
+ return ret;
+ } else {
WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 8)
#1278: FILE: lib/librte_eal/common/include/arch/x86/rte_memcpy.h:1910:
+ } else
[...]
+ if (rte_cpu_get_flag_enabled(RTE_CPUFLAG_AVX2)) {
total: 53 errors, 132 warnings, 1260 lines checked
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2017-09-26 7:44 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1506411689-94690-2-git-send-email-xiaoyun.li@intel.com>
2017-09-26 7:44 ` [dpdk-test-report] |WARNING| pw29187 [PATCH v3 1/3] eal/x86: run-time dispatch over memcpy checkpatch
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).