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 F099D454AA; Wed, 19 Jun 2024 18:27:35 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 2733F402C6; Wed, 19 Jun 2024 18:27:35 +0200 (CEST) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mails.dpdk.org (Postfix) with ESMTP id A26FA40289 for ; Wed, 19 Jun 2024 18:27:33 +0200 (CEST) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 58B1D367; Wed, 19 Jun 2024 09:27:57 -0700 (PDT) Received: from ampere-altra-2-3.austin.arm.com (ampere-altra-2-3.austin.arm.com [10.118.14.97]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 972A23F64C; Wed, 19 Jun 2024 09:27:32 -0700 (PDT) From: Paul Szczepanek To: david.marchand@redhat.com Cc: dev@dpdk.org, Paul Szczepanek , Nick Connolly Subject: [PATCH v1] ptr_compress: fix offset to use portable type Date: Wed, 19 Jun 2024 16:27:13 +0000 Message-Id: <20240619162713.2489866-1-paul.szczepanek@arm.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Fix the type of offset to use portable uintptr_t instead of uint64_t. Fixes: 077596a4b077 ("ptr_compress: add pointer compression library") Reviewed-by: Nick Connolly Signed-off-by: Paul Szczepanek --- lib/ptr_compress/rte_ptr_compress.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/ptr_compress/rte_ptr_compress.h b/lib/ptr_compress/rte_ptr_compress.h index ca746970c0..9742a9594a 100644 --- a/lib/ptr_compress/rte_ptr_compress.h +++ b/lib/ptr_compress/rte_ptr_compress.h @@ -141,7 +141,7 @@ rte_ptr_compress_32_shift(void *ptr_base, void * const *src_table, i += svcntd(); } while (i < n); #elif defined __ARM_NEON && !defined RTE_ARCH_ARMv8_AARCH32 - uint64_t ptr_diff; + uintptr_t ptr_diff; uint64x2_t v_ptr_table; /* right shift is done by left shifting by negative int */ int64x2_t v_shift = vdupq_n_s64(-bit_shift); @@ -202,7 +202,7 @@ rte_ptr_decompress_32_shift(void *ptr_base, uint32_t const *src_table, i += svcntd(); } while (i < n); #elif defined __ARM_NEON && !defined RTE_ARCH_ARMv8_AARCH32 - uint64_t ptr_diff; + uintptr_t ptr_diff; uint64x2_t v_ptr_table; int64x2_t v_shift = vdupq_n_s64(bit_shift); uint64x2_t v_ptr_base = vdupq_n_u64((uint64_t)ptr_base); @@ -215,7 +215,7 @@ rte_ptr_decompress_32_shift(void *ptr_base, uint32_t const *src_table, } /* process leftover single item in case of odd number of n */ if (unlikely(n & 0x1)) { - ptr_diff = ((uint64_t) src_table[i]) << bit_shift; + ptr_diff = ((uintptr_t) src_table[i]) << bit_shift; dest_table[i] = RTE_PTR_ADD(ptr_base, ptr_diff); } #else -- 2.25.1