From: checkpatch@dpdk.org
To: test-report@dpdk.org
Cc: Xiaoyun Li <xiaoyun.li@intel.com>
Subject: [dpdk-test-report] |WARNING| pw29187 [PATCH v3 1/3] eal/x86: run-time dispatch over memcpy
Date: Tue, 26 Sep 2017 09:44:32 +0200 (CEST) [thread overview]
Message-ID: <20170926074432.BBE811B1EB@dpdk.org> (raw)
In-Reply-To: <1506411689-94690-2-git-send-email-xiaoyun.li@intel.com>
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
parent reply other threads:[~2017-09-26 7:44 UTC|newest]
Thread overview: expand[flat|nested] mbox.gz Atom feed
[parent not found: <1506411689-94690-2-git-send-email-xiaoyun.li@intel.com>]
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=20170926074432.BBE811B1EB@dpdk.org \
--to=checkpatch@dpdk.org \
--cc=test-report@dpdk.org \
--cc=xiaoyun.li@intel.com \
/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).