Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package libvma for openSUSE:Factory checked in at 2026-09-15 12:48:37 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/libvma (Old) and /work/SRC/openSUSE:Factory/.libvma.new.383539 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "libvma" Tue Sep 15 12:48:37 2026 rev:23 rq:1377877 version:9.8.84 Changes: -------- --- /work/SRC/openSUSE:Factory/libvma/libvma.changes 2026-01-20 21:05:46.679082811 +0100 +++ /work/SRC/openSUSE:Factory/.libvma.new.383539/libvma.changes 2026-09-15 12:48:54.593157012 +0200 @@ -1,0 +2,8 @@ +Mon Sep 14 09:42:27 UTC 2026 - Nicolas Morey <[email protected]> + +- Backport upstream fixes for GCC16: + - issue-4917201-Fix-more-unused-but-set-warnings-w-enable-opt-log-no.patch + - issue-4917201-Fix-non-c-typedef-for-linkage-build-error.patch + - issue-4917201-Fix-unused-but-set-warnings-in-ring_simple-ring_tap.patch + +------------------------------------------------------------------- New: ---- issue-4917201-Fix-more-unused-but-set-warnings-w-enable-opt-log-no.patch issue-4917201-Fix-non-c-typedef-for-linkage-build-error.patch issue-4917201-Fix-unused-but-set-warnings-in-ring_simple-ring_tap.patch ----------(New B)---------- New:- Backport upstream fixes for GCC16: - issue-4917201-Fix-more-unused-but-set-warnings-w-enable-opt-log-no.patch - issue-4917201-Fix-non-c-typedef-for-linkage-build-error.patch New: - issue-4917201-Fix-more-unused-but-set-warnings-w-enable-opt-log-no.patch - issue-4917201-Fix-non-c-typedef-for-linkage-build-error.patch - issue-4917201-Fix-unused-but-set-warnings-in-ring_simple-ring_tap.patch New: - issue-4917201-Fix-non-c-typedef-for-linkage-build-error.patch - issue-4917201-Fix-unused-but-set-warnings-in-ring_simple-ring_tap.patch ----------(New E)---------- ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ libvma.spec ++++++ --- /var/tmp/diff_new_pack.MHs2pN/_old 2026-09-15 12:48:56.648242683 +0200 +++ /var/tmp/diff_new_pack.MHs2pN/_new 2026-09-15 12:48:56.654242933 +0200 @@ -30,6 +30,9 @@ Source1: vma.service Source100: README.md Patch1: harden_vma.service.patch +Patch2: issue-4917201-Fix-non-c-typedef-for-linkage-build-error.patch +Patch3: issue-4917201-Fix-more-unused-but-set-warnings-w-enable-opt-log-no.patch +Patch4: issue-4917201-Fix-unused-but-set-warnings-in-ring_simple-ring_tap.patch URL: https://github.com/Mellanox/libvma BuildRequires: autoconf BuildRequires: automake ++++++ _scmsync.obsinfo ++++++ --- /var/tmp/diff_new_pack.MHs2pN/_old 2026-09-15 12:48:56.792248686 +0200 +++ /var/tmp/diff_new_pack.MHs2pN/_new 2026-09-15 12:48:56.801249061 +0200 @@ -1,6 +1,7 @@ -mtime: 1768464231 -commit: af5bbc0059284df5abe7390c5173c82afed681038d5dbf2c0e5fda5ca42682c4 -url: https://src.opensuse.org/HPC/libvma.git -revision: af5bbc0059284df5abe7390c5173c82afed681038d5dbf2c0e5fda5ca42682c4 +mtime: 1789378976 +commit: c0f6aae84c29f09db676db5cfb289ca9cb4265d339dbf2af9f847e5a72f94b62 +url: https://src.opensuse.org/HPC/libvma +revision: c0f6aae84c29f09db676db5cfb289ca9cb4265d339dbf2af9f847e5a72f94b62 +trackingbranch: main projectscmsync: https://src.opensuse.org/HPC/_ObsPrj ++++++ build.specials.obscpio ++++++ ++++++ build.specials.obscpio ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/.gitignore new/.gitignore --- old/.gitignore 1970-01-01 01:00:00.000000000 +0100 +++ new/.gitignore 2026-09-14 11:42:56.000000000 +0200 @@ -0,0 +1 @@ +.osc ++++++ issue-4917201-Fix-more-unused-but-set-warnings-w-enable-opt-log-no.patch ++++++ commit e02b9cb22e45ee66e6394c9d98e47be7ed899385 Author: Tomer Cabouly <[email protected]> Date: Mon May 11 12:09:58 2026 +0000 issue: 4917201 Fix more unused-but-set warnings w/ --enable-opt-log=no Build with --enable-opt-log=no compiles __log_funcall / __if_dbg / si_udp_logfunc to ((void)0), and the BURST variant of vma_ibv_modify_qp_rate_limit drops its mask argument, leaving variables assigned but unread and breaking -Werror=unused-but-set-variable. - verbs_extra.cpp::priv_ibv_modify_qp_ratelimit: NOT_IN_USE(attr_mask) after the modify call (mask is dropped by the macro under DEFINED_IBV_QP_SUPPORT_BURST). - io_mux_call.cpp::polling_loops: NOT_IN_USE(poll_counter) after the trailing __if_dbg (only consumer when log levels are enabled). - sockinfo_udp.cpp::rx_request_notification: NOT_IN_USE(ring_armed_count) before si_udp_logfunc (only consumer when log levels are enabled). Signed-off-by: Tomer Cabouly <[email protected]> diff --git src/vma/ib/base/verbs_extra.cpp src/vma/ib/base/verbs_extra.cpp index e7c1a8538e42..444adb949eb6 100644 --- src/vma/ib/base/verbs_extra.cpp +++ src/vma/ib/base/verbs_extra.cpp @@ -317,6 +317,7 @@ int priv_ibv_modify_qp_ratelimit(struct ibv_qp *qp, struct vma_rate_limit_t &rat return -2; } ENDIF_VERBS_FAILURE; BULLSEYE_EXCLUDE_BLOCK_END + NOT_IN_USE(attr_mask); /* vma_ibv_modify_qp_rate_limit may drop mask under DEFINED_IBV_QP_SUPPORT_BURST */ #ifdef DEFINED_IBV_QP_SUPPORT_BURST vlog_printf(VLOG_DEBUG, "qp was set to rate limit %d, burst size %d, packet size %d\n", rate_limit.rate, rate_limit.max_burst_sz, rate_limit.typical_pkt_sz); diff --git src/vma/iomux/io_mux_call.cpp src/vma/iomux/io_mux_call.cpp index 99ed4f7f2c62..39bae13d1866 100644 --- src/vma/iomux/io_mux_call.cpp +++ src/vma/iomux/io_mux_call.cpp @@ -368,6 +368,7 @@ void io_mux_call::polling_loops() } __if_dbg("2nd scenario exit (loop %d, elapsed %d)", poll_counter, m_elapsed.tv_usec); + NOT_IN_USE(poll_counter); /* to suppress warning in case VMA_MAX_DEFINED_LOG_LEVEL */ } void io_mux_call::blocking_loops() diff --git src/vma/sock/sockinfo_udp.cpp src/vma/sock/sockinfo_udp.cpp index 41d315c28ebb..8438b5f79200 100644 --- src/vma/sock/sockinfo_udp.cpp +++ src/vma/sock/sockinfo_udp.cpp @@ -1498,6 +1498,8 @@ int sockinfo_udp::rx_request_notification(uint64_t poll_sn) } m_rx_ring_map_lock.unlock(); + // to suppress warning in case si_udp_logfunc is compiled out + NOT_IN_USE(ring_armed_count); si_udp_logfunc("armed or busy %d ring(s) and %d ring are pending processing", ring_armed_count, ring_ready_count); return ring_ready_count; } ++++++ issue-4917201-Fix-non-c-typedef-for-linkage-build-error.patch ++++++ commit 9ae463b8db103a4f7a0b5ce014daaf899b04d1df Author: Tomer Cabouly <[email protected]> Date: Thu Mar 5 07:55:33 2026 +0000 issue: 4917201 Fix non-c-typedef-for-linkage build error Name anonymous structs socket_stats_t and socket_instance_block_t to fix -Werror=non-c-typedef-for-linkage with newer compilers. Structs with member functions require a tag name for C++ linkage. Matches the existing pattern used by sh_mem_t in the same file. Signed-off-by: Thomas Cabouly <[email protected]> diff --git src/vma/util/vma_stats.h src/vma/util/vma_stats.h index 3d7f6026eb72..5613843d58a4 100644 --- src/vma/util/vma_stats.h +++ src/vma/util/vma_stats.h @@ -139,7 +139,7 @@ typedef struct { uint32_t n_tx_dummy; } socket_counters_t; -typedef struct { +typedef struct socket_stats_t { int fd; uint32_t inode; uint32_t tcp_state; // enum tcp_state @@ -184,7 +184,7 @@ typedef struct { }; } socket_stats_t; -typedef struct { +typedef struct socket_instance_block_t { bool b_enabled; socket_stats_t skt_stats; ++++++ issue-4917201-Fix-unused-but-set-warnings-in-ring_simple-ring_tap.patch ++++++ commit 7bda0b56ef9f8d775eef4781bfbc1f4ad3d2ad18 Author: Tomer Cabouly <[email protected]> Date: Sun May 10 06:08:06 2026 +0000 issue: 4917201 Fix unused-but-set warnings in ring_simple/ring_tap Build with --enable-opt-log=no compiles ring_logfunc to ((void)0), leaving 'freed' / 'count' assigned but unread and breaking -Werror=unused-but-set-variable. - ring_simple.cpp::put_tx_buffers: NOT_IN_USE(freed) before logfunc - ring_tap.cpp::mem_buf_tx_release: NOT_IN_USE(freed) before logfunc - ring_tap.cpp::mem_buf_desc_return_single_to_owner_tx: drop dead 'count' (incremented but never logged or returned) Signed-off-by: Tomer Cabouly <[email protected]> diff --git src/vma/dev/ring_simple.cpp src/vma/dev/ring_simple.cpp index 9cb894527ef3..c08fd0383af9 100644 --- src/vma/dev/ring_simple.cpp +++ src/vma/dev/ring_simple.cpp @@ -833,6 +833,9 @@ int ring_simple::put_tx_buffers(mem_buf_desc_t* buff_list) count++; buff_list = next; } + + // to suppress warning in case ring_logfunc is compiled out + NOT_IN_USE(freed); ring_logfunc("buf_list: %p count: %d freed: %d\n", buff_list, count, freed); return_to_global_pool(); diff --git src/vma/dev/ring_tap.cpp src/vma/dev/ring_tap.cpp index e4f56d69581f..9e4a4b957331 100644 --- src/vma/dev/ring_tap.cpp +++ src/vma/dev/ring_tap.cpp @@ -495,8 +495,6 @@ void ring_tap::mem_buf_desc_return_single_to_owner_tx(mem_buf_desc_t* p_mem_buf_ { auto_unlocker lock(m_lock_ring_tx); - int count = 0; - if (likely(p_mem_buf_desc)) { //potential race, ref is protected here by ring_tx lock, and in dst_entry_tcp & sockinfo_tcp by tcp lock if (likely(p_mem_buf_desc->lwip_pbuf.pbuf.ref)) @@ -508,7 +506,6 @@ void ring_tap::mem_buf_desc_return_single_to_owner_tx(mem_buf_desc_t* p_mem_buf_ p_mem_buf_desc->p_next_desc = NULL; free_lwip_pbuf(&p_mem_buf_desc->lwip_pbuf); m_tx_pool.push_back(p_mem_buf_desc); - count++; } } @@ -547,6 +544,8 @@ int ring_tap::mem_buf_tx_release(mem_buf_desc_t* buff_list, bool b_accounting, b count++; buff_list = next; } + // to suppress warning in case ring_logfunc is compiled out + NOT_IN_USE(freed); ring_logfunc("buf_list: %p count: %d freed: %d\n", buff_list, count, freed); return_to_global_pool();
