From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by smtp.lore.kernel.org (Postfix) with ESMTP id A88F5C5B572 for ; Wed, 12 Aug 2026 17:10:16 +0000 (UTC) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 3E21E42C24; Wed, 12 Aug 2026 19:09:01 +0200 (CEST) Received: from agw.arknetworks.am (agw.arknetworks.am [79.141.165.80]) by mails.dpdk.org (Postfix) with ESMTP id 32018427AC for ; Wed, 12 Aug 2026 19:08:55 +0200 (CEST) Received: from localhost.localdomain (unknown [78.109.70.176]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by agw.arknetworks.am (Postfix) with ESMTPSA id 769EFE0BD3; Wed, 12 Aug 2026 21:08:54 +0400 (+04) DKIM-Filter: OpenDKIM Filter v2.11.0 agw.arknetworks.am 769EFE0BD3 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arknetworks.am; s=default; t=1786554535; bh=dAohAAIdp6nhC/tLtvCvDJvq2ufi138HmQDjQ/MTZ5U=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=fKbXrTgvvsWHloYy7uiR6XHiUE6Syk56WmJF794ixLneWDbrhxAnArUm5xKi5ZUo5 bEJj0NBcvRqH+/nuphuoGvZ7g0LOkwMOLbExE4gNFpQPkKZaG6oSGEHP6GDkLzVhUP pg59Kh+hj25dMGQviE7GKAtdN9Scz7lZ9sVsGUD3SmiML4Crl8R7DclzLrbuOimi5z mJO+BV+/IXSR3L+gmgJ85K3kDq3MEaJK9Mc7x+5R4x5/mia99g+pwNQT+ESc6LwKz7 /wveOJoJmCdbI341ltQyy4hrOchuqoF8gZw16u1s4awqg+6nPTKBMYrb8VPSBtuC8Z Rk+7xMZ6xR9zw== From: Ivan Malov To: dev@dpdk.org Cc: Andy Moreton , Viacheslav Galaktionov , Roman Zhukov , Pieter Jansen van Vuuren , Stephen Hemminger , Andrew Rybchenko Subject: [PATCH v2 14/14] common/sfc_efx/base: cleanup wider type comparisons in loops Date: Wed, 12 Aug 2026 21:08:34 +0400 Message-ID: <20260812170834.8443-15-ivan.malov@arknetworks.am> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260812170834.8443-1-ivan.malov@arknetworks.am> References: <20260811174821.8930-1-ivan.malov@arknetworks.am> <20260812170834.8443-1-ivan.malov@arknetworks.am> 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 From: Andy Moreton CodeQL reports errors for comparisons between narrow and wider types in loop conditions [cpp/infiniteloop]. Use the wider types to fix that. Signed-off-by: Andy Moreton Reviewed-by: Ivan Malov Reviewed-by: Viacheslav Galaktionov --- drivers/common/sfc_efx/base/ef10_filter.c | 2 +- drivers/common/sfc_efx/base/ef10_mcdi.c | 2 +- drivers/common/sfc_efx/base/ef10_nvram.c | 4 ++-- drivers/common/sfc_efx/base/efx_bootcfg.c | 2 +- drivers/common/sfc_efx/base/mcdi_mon.c | 2 +- 5 files changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/common/sfc_efx/base/ef10_filter.c b/drivers/common/sfc_efx/base/ef10_filter.c index 0d69ec5ba8..7d845f1446 100644 --- a/drivers/common/sfc_efx/base/ef10_filter.c +++ b/drivers/common/sfc_efx/base/ef10_filter.c @@ -1300,7 +1300,7 @@ ef10_filter_supported_filters( size_t mcdi_list_length; size_t mcdi_encap_list_length; size_t list_length; - uint32_t i; + size_t i; uint32_t next_buf_idx; size_t next_buf_length; efx_rc_t rc; diff --git a/drivers/common/sfc_efx/base/ef10_mcdi.c b/drivers/common/sfc_efx/base/ef10_mcdi.c index f852d1cde3..6f3492efe4 100644 --- a/drivers/common/sfc_efx/base/ef10_mcdi.c +++ b/drivers/common/sfc_efx/base/ef10_mcdi.c @@ -140,7 +140,7 @@ ef10_mcdi_send_request( const efx_mcdi_transport_t *emtp = enp->en_mcdi.em_emtp; efsys_mem_t *esmp = emtp->emt_dma_mem; efx_dword_t dword; - unsigned int pos; + size_t pos; EFSYS_ASSERT(EFX_FAMILY_IS_EF100(enp) || EFX_FAMILY_IS_EF10(enp)); diff --git a/drivers/common/sfc_efx/base/ef10_nvram.c b/drivers/common/sfc_efx/base/ef10_nvram.c index ce8357fa94..fd9564760a 100644 --- a/drivers/common/sfc_efx/base/ef10_nvram.c +++ b/drivers/common/sfc_efx/base/ef10_nvram.c @@ -2386,7 +2386,7 @@ ef10_nvram_type_to_partn( efx_mcdi_iface_t *emip = &(enp->en_mcdi.em_emip); ef10_parttbl_entry_t *parttbl = NULL; size_t parttbl_rows = 0; - unsigned int i; + size_t i; EFSYS_ASSERT3U(type, !=, EFX_NVRAM_INVALID); EFSYS_ASSERT3U(type, <, EFX_NVRAM_NTYPES); @@ -2418,7 +2418,7 @@ ef10_nvram_partn_to_type( efx_mcdi_iface_t *emip = &(enp->en_mcdi.em_emip); ef10_parttbl_entry_t *parttbl = NULL; size_t parttbl_rows = 0; - unsigned int i; + size_t i; EFSYS_ASSERT(typep != NULL); diff --git a/drivers/common/sfc_efx/base/efx_bootcfg.c b/drivers/common/sfc_efx/base/efx_bootcfg.c index c5b8182a3d..83379ea34c 100644 --- a/drivers/common/sfc_efx/base/efx_bootcfg.c +++ b/drivers/common/sfc_efx/base/efx_bootcfg.c @@ -140,8 +140,8 @@ efx_dhcp_csum( __in_bcount(size) uint8_t const *data, __in size_t size) { - unsigned int pos; uint8_t checksum = 0; + size_t pos; for (pos = 0; pos < size; pos++) checksum += data[pos]; diff --git a/drivers/common/sfc_efx/base/mcdi_mon.c b/drivers/common/sfc_efx/base/mcdi_mon.c index 2089840d2c..c5510e53a8 100644 --- a/drivers/common/sfc_efx/base/mcdi_mon.c +++ b/drivers/common/sfc_efx/base/mcdi_mon.c @@ -30,7 +30,7 @@ mcdi_mon_decode_stats( { efx_mcdi_iface_t *emip = &(enp->en_mcdi.em_emip); efx_mon_stat_portmask_t port_mask; - uint16_t sensor; + size_t sensor; size_t sensor_max; uint32_t stat_mask[(EFX_MON_NSTATS + 31) / 32]; uint32_t idx = 0; -- 2.47.3