From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id A6308A0547 for ; Thu, 21 Oct 2021 10:53:04 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 9BF02411C4; Thu, 21 Oct 2021 10:53:04 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 605E3411C4 for ; Thu, 21 Oct 2021 10:53:01 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id 581A4120E1D; Thu, 21 Oct 2021 10:53:01 +0200 (CEST) In-Reply-To: <20211021085132.12672-3-elibr@nvidia.com> References: <20211021085132.12672-3-elibr@nvidia.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Eli Britstein Message-Id: <20211021085301.581A4120E1D@dpdk.org> Date: Thu, 21 Oct 2021 10:53:01 +0200 (CEST) Subject: [dpdk-test-report] |WARNING| pw102539 [PATCH V2 3/3] eal/x86: avoid cast-align warning in x86 memcpy functions X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: automatic DPDK test reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: test-report-bounces@dpdk.org Sender: "test-report" Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/102539 _coding style issues_ WARNING:LONG_LINE: line length of 123 exceeds 100 columns #221: FILE: lib/eal/x86/include/rte_memcpy.h:592: + xmm0 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 0 * 16)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #221: FILE: lib/eal/x86/include/rte_memcpy.h:592: + xmm0 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 0 * 16)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #221: FILE: lib/eal/x86/include/rte_memcpy.h:592: + xmm0 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 0 * 16)); \$ WARNING:LONG_LINE: line length of 123 exceeds 100 columns #231: FILE: lib/eal/x86/include/rte_memcpy.h:594: + xmm1 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 1 * 16)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #231: FILE: lib/eal/x86/include/rte_memcpy.h:594: + xmm1 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 1 * 16)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #231: FILE: lib/eal/x86/include/rte_memcpy.h:594: + xmm1 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 1 * 16)); \$ WARNING:LONG_LINE: line length of 123 exceeds 100 columns #232: FILE: lib/eal/x86/include/rte_memcpy.h:595: + xmm2 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 2 * 16)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #232: FILE: lib/eal/x86/include/rte_memcpy.h:595: + xmm2 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 2 * 16)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #232: FILE: lib/eal/x86/include/rte_memcpy.h:595: + xmm2 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 2 * 16)); \$ WARNING:LONG_LINE: line length of 123 exceeds 100 columns #233: FILE: lib/eal/x86/include/rte_memcpy.h:596: + xmm3 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 3 * 16)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #233: FILE: lib/eal/x86/include/rte_memcpy.h:596: + xmm3 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 3 * 16)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #233: FILE: lib/eal/x86/include/rte_memcpy.h:596: + xmm3 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 3 * 16)); \$ WARNING:LONG_LINE: line length of 123 exceeds 100 columns #234: FILE: lib/eal/x86/include/rte_memcpy.h:597: + xmm4 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 4 * 16)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #234: FILE: lib/eal/x86/include/rte_memcpy.h:597: + xmm4 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 4 * 16)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #234: FILE: lib/eal/x86/include/rte_memcpy.h:597: + xmm4 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 4 * 16)); \$ WARNING:LONG_LINE: line length of 123 exceeds 100 columns #235: FILE: lib/eal/x86/include/rte_memcpy.h:598: + xmm5 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 5 * 16)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #235: FILE: lib/eal/x86/include/rte_memcpy.h:598: + xmm5 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 5 * 16)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #235: FILE: lib/eal/x86/include/rte_memcpy.h:598: + xmm5 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 5 * 16)); \$ WARNING:LONG_LINE: line length of 123 exceeds 100 columns #236: FILE: lib/eal/x86/include/rte_memcpy.h:599: + xmm6 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 6 * 16)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #236: FILE: lib/eal/x86/include/rte_memcpy.h:599: + xmm6 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 6 * 16)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #236: FILE: lib/eal/x86/include/rte_memcpy.h:599: + xmm6 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 6 * 16)); \$ WARNING:LONG_LINE: line length of 123 exceeds 100 columns #237: FILE: lib/eal/x86/include/rte_memcpy.h:600: + xmm7 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 7 * 16)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #237: FILE: lib/eal/x86/include/rte_memcpy.h:600: + xmm7 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 7 * 16)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #237: FILE: lib/eal/x86/include/rte_memcpy.h:600: + xmm7 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 7 * 16)); \$ WARNING:LONG_LINE: line length of 123 exceeds 100 columns #238: FILE: lib/eal/x86/include/rte_memcpy.h:601: + xmm8 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 8 * 16)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #238: FILE: lib/eal/x86/include/rte_memcpy.h:601: + xmm8 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 8 * 16)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #238: FILE: lib/eal/x86/include/rte_memcpy.h:601: + xmm8 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 8 * 16)); \$ WARNING:LONG_LINE: line length of 117 exceeds 100 columns #248: FILE: lib/eal/x86/include/rte_memcpy.h:603: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 0 * 16), _mm_alignr_epi8(xmm1, xmm0, offset)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #248: FILE: lib/eal/x86/include/rte_memcpy.h:603: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 0 * 16), _mm_alignr_epi8(xmm1, xmm0, offset)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #248: FILE: lib/eal/x86/include/rte_memcpy.h:603: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 0 * 16), _mm_alignr_epi8(xmm1, xmm0, offset)); \$ WARNING:LONG_LINE: line length of 117 exceeds 100 columns #249: FILE: lib/eal/x86/include/rte_memcpy.h:604: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 1 * 16), _mm_alignr_epi8(xmm2, xmm1, offset)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #249: FILE: lib/eal/x86/include/rte_memcpy.h:604: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 1 * 16), _mm_alignr_epi8(xmm2, xmm1, offset)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #249: FILE: lib/eal/x86/include/rte_memcpy.h:604: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 1 * 16), _mm_alignr_epi8(xmm2, xmm1, offset)); \$ WARNING:LONG_LINE: line length of 117 exceeds 100 columns #250: FILE: lib/eal/x86/include/rte_memcpy.h:605: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 2 * 16), _mm_alignr_epi8(xmm3, xmm2, offset)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #250: FILE: lib/eal/x86/include/rte_memcpy.h:605: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 2 * 16), _mm_alignr_epi8(xmm3, xmm2, offset)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #250: FILE: lib/eal/x86/include/rte_memcpy.h:605: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 2 * 16), _mm_alignr_epi8(xmm3, xmm2, offset)); \$ WARNING:LONG_LINE: line length of 117 exceeds 100 columns #251: FILE: lib/eal/x86/include/rte_memcpy.h:606: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 3 * 16), _mm_alignr_epi8(xmm4, xmm3, offset)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #251: FILE: lib/eal/x86/include/rte_memcpy.h:606: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 3 * 16), _mm_alignr_epi8(xmm4, xmm3, offset)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #251: FILE: lib/eal/x86/include/rte_memcpy.h:606: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 3 * 16), _mm_alignr_epi8(xmm4, xmm3, offset)); \$ WARNING:LONG_LINE: line length of 117 exceeds 100 columns #252: FILE: lib/eal/x86/include/rte_memcpy.h:607: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 4 * 16), _mm_alignr_epi8(xmm5, xmm4, offset)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #252: FILE: lib/eal/x86/include/rte_memcpy.h:607: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 4 * 16), _mm_alignr_epi8(xmm5, xmm4, offset)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #252: FILE: lib/eal/x86/include/rte_memcpy.h:607: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 4 * 16), _mm_alignr_epi8(xmm5, xmm4, offset)); \$ WARNING:LONG_LINE: line length of 117 exceeds 100 columns #253: FILE: lib/eal/x86/include/rte_memcpy.h:608: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 5 * 16), _mm_alignr_epi8(xmm6, xmm5, offset)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #253: FILE: lib/eal/x86/include/rte_memcpy.h:608: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 5 * 16), _mm_alignr_epi8(xmm6, xmm5, offset)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #253: FILE: lib/eal/x86/include/rte_memcpy.h:608: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 5 * 16), _mm_alignr_epi8(xmm6, xmm5, offset)); \$ WARNING:LONG_LINE: line length of 117 exceeds 100 columns #254: FILE: lib/eal/x86/include/rte_memcpy.h:609: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 6 * 16), _mm_alignr_epi8(xmm7, xmm6, offset)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #254: FILE: lib/eal/x86/include/rte_memcpy.h:609: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 6 * 16), _mm_alignr_epi8(xmm7, xmm6, offset)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #254: FILE: lib/eal/x86/include/rte_memcpy.h:609: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 6 * 16), _mm_alignr_epi8(xmm7, xmm6, offset)); \$ WARNING:LONG_LINE: line length of 117 exceeds 100 columns #255: FILE: lib/eal/x86/include/rte_memcpy.h:610: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 7 * 16), _mm_alignr_epi8(xmm8, xmm7, offset)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #255: FILE: lib/eal/x86/include/rte_memcpy.h:610: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 7 * 16), _mm_alignr_epi8(xmm8, xmm7, offset)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #255: FILE: lib/eal/x86/include/rte_memcpy.h:610: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 7 * 16), _mm_alignr_epi8(xmm8, xmm7, offset)); \$ WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12) #262: FILE: lib/eal/x86/include/rte_memcpy.h:619: while (len >= 32 + 16 - offset) { \ + xmm0 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 0 * 16)); \ WARNING:LONG_LINE: line length of 123 exceeds 100 columns #264: FILE: lib/eal/x86/include/rte_memcpy.h:620: + xmm0 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 0 * 16)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #264: FILE: lib/eal/x86/include/rte_memcpy.h:620: + xmm0 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 0 * 16)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #264: FILE: lib/eal/x86/include/rte_memcpy.h:620: + xmm0 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 0 * 16)); \$ WARNING:LONG_LINE: line length of 123 exceeds 100 columns #268: FILE: lib/eal/x86/include/rte_memcpy.h:622: + xmm1 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 1 * 16)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #268: FILE: lib/eal/x86/include/rte_memcpy.h:622: + xmm1 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 1 * 16)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #268: FILE: lib/eal/x86/include/rte_memcpy.h:622: + xmm1 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 1 * 16)); \$ WARNING:LONG_LINE: line length of 123 exceeds 100 columns #269: FILE: lib/eal/x86/include/rte_memcpy.h:623: + xmm2 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 2 * 16)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #269: FILE: lib/eal/x86/include/rte_memcpy.h:623: + xmm2 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 2 * 16)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #269: FILE: lib/eal/x86/include/rte_memcpy.h:623: + xmm2 = _mm_loadu_si128((const __m128i *)(const void *)((const uint8_t *)src - offset + 2 * 16)); \$ WARNING:LONG_LINE: line length of 117 exceeds 100 columns #273: FILE: lib/eal/x86/include/rte_memcpy.h:625: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 0 * 16), _mm_alignr_epi8(xmm1, xmm0, offset)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #273: FILE: lib/eal/x86/include/rte_memcpy.h:625: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 0 * 16), _mm_alignr_epi8(xmm1, xmm0, offset)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #273: FILE: lib/eal/x86/include/rte_memcpy.h:625: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 0 * 16), _mm_alignr_epi8(xmm1, xmm0, offset)); \$ WARNING:LONG_LINE: line length of 117 exceeds 100 columns #274: FILE: lib/eal/x86/include/rte_memcpy.h:626: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 1 * 16), _mm_alignr_epi8(xmm2, xmm1, offset)); \ ERROR:CODE_INDENT: code indent should use tabs where possible #274: FILE: lib/eal/x86/include/rte_memcpy.h:626: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 1 * 16), _mm_alignr_epi8(xmm2, xmm1, offset)); \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #274: FILE: lib/eal/x86/include/rte_memcpy.h:626: + _mm_storeu_si128((__m128i *)(void *)((uint8_t *)dst + 1 * 16), _mm_alignr_epi8(xmm2, xmm1, offset)); \$ total: 22 errors, 45 warnings, 122 lines checked