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 17D3148AFC for ; Thu, 13 Nov 2025 18:50:56 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0D78D40151; Thu, 13 Nov 2025 18:50:56 +0100 (CET) Received: from mail-oi1-f172.google.com (mail-oi1-f172.google.com [209.85.167.172]) by mails.dpdk.org (Postfix) with ESMTP id A098140151 for ; Thu, 13 Nov 2025 18:50:55 +0100 (CET) Received: by mail-oi1-f172.google.com with SMTP id 5614622812f47-45038ffac4fso504479b6e.2 for ; Thu, 13 Nov 2025 09:50:55 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1763056255; x=1763661055; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=mnKDErVQ+YG09GyWiL7Omc32wmV/ekyfp8Jqe6Ty+a4=; b=P1V4/BGhG3vSny570OAuLn4CVsXmB/NT8Z3vDQv9cL+IRY1sQf5GkObN/5FcYSMuIf ejRTEVnTPiAt6YWlPv12nyW+Brw4ic1DyTjhKrX0mgyYCVcRDYcIxDHWiB4aHeJNyr4E NH6M38kyk4qhdCvK1Ci+8M0nNJHPgMZS/AgTyCAQa7VcJdR2UFv3Z/gpMst/9B+IiarE n6zcR8O1VZfBxpZF38ZjfGBvw6Q0ErgraMB9OC/IuWGLM2WA8pZTh1LnLko72jEnvBW5 YPWSCz9b/vv7v4w0Ji5NKhR5CyzWIkK+eaQwNKFVzUONItBi9xubNldpg7dYHijd2Bjs lGXQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763056255; x=1763661055; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=mnKDErVQ+YG09GyWiL7Omc32wmV/ekyfp8Jqe6Ty+a4=; b=LMkyUGag3JGoInMVlUCwcVEW6I2khI+Ku9Tg5t57BfYSNbrV2Lneza0Y58xHZ9Bpug kTJ/qCUgNvsHuY5bB3jtsGUbMzBmZ/CLN0BauPQid3O05SrTK51a8CDaSrVQxsaQ/t6Y G5zte4QIQSFf0kho/QJNgCZZhoOfpvopWhx2Uurw7T9Lp0qPWtQdL00PWZqE4IBIXB3N MYFZJ1imPrzlExtJT2v853bBi4Ik1WWnFMRYqX4QAUphnbwmiCEVF+NXyj0nnuJPq/yv q4QHwbfodtWVAEPG6+xynHadr8ZfISvzVY9yiZeO0rugx4SdHy0/FTVYEXGUWEHNVuBm 4ktg== X-Gm-Message-State: AOJu0Yx/f+dUdoSlYvT6rGT1QzGl6CkRPACgoh8kjBTxTlet77wc9uLm p5iCqOeaZQRjXawnKnmxo6mTjFD+fCt7ueK1aM4B6J/uFyCGiMponp75Vpj8V8wbjXc= X-Gm-Gg: ASbGncuv9VcAED3Vh4xXfX4Vz/8NhAKyE5qUugBGTq4GogrhynibyjfBivMEayC8l1U xcAfixOfuH3uaBLvoWQKMBZxSEfgK6fBXCyDRmWbdgL06pmPrt10Uq8dWTaID5K3x9JCPHjtAu7 lCrFigzx1BEgPjn9AAQFAjZK8B3bOTAKGYi50JoK9yPhEAcg5NdI2gZ7z0CF8wUAVKE2q1Fp9XQ QXuYHKzs6qgHde8V9fI6q2XzYZbxkZBHcLf9IeE/CoagV2D90n4ngEcaEo+XnneMJK4+KvPcP0P yjfRbGskb7n8Cs/I29vUdaicAfbHR8WhXdS8rnNWhgPt13z7xcDtLk9iNadI39Xjpc98ZPclKwT 8t8kEBDDwHUgCCFlBXV4N24vDDZU1HALx0QhaJ8u6Zoef7s1i6V4kqy0jsi1wsqCdVRAQUjoHGT IU/N40eCtxD+WSimkFAYxIZP5W6+RFjN+jUhXlTgLsW1ro X-Google-Smtp-Source: AGHT+IHFw97Tg2XUzPWxpgpfpfyTfsssN8SFngf7RIleYqTdU7OALPXcJPhR2zm+74HcZz3zl0DhGg== X-Received: by 2002:a05:6808:6713:b0:441:cabe:907 with SMTP id 5614622812f47-45097349b60mr60038b6e.14.1763056254811; Thu, 13 Nov 2025 09:50:54 -0800 (PST) Received: from phoenix (204-195-96-226.wavecable.com. [204.195.96.226]) by smtp.gmail.com with ESMTPSA id 006d021491bc7-65724cb8732sm1308160eaf.1.2025.11.13.09.50.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 13 Nov 2025 09:50:54 -0800 (PST) Date: Thu, 13 Nov 2025 09:50:49 -0800 From: Stephen Hemminger To: dev@dpdk.org Cc: stable@dpdk.org, Dariusz Sosnowski , Viacheslav Ovsiienko , Bing Zhao , Ori Kam , Suanming Mou , Matan Azrad , Ophir Munk Subject: Re: [PATCH] net/mlx5: fix uninitialized warning Message-ID: <20251113095049.008bbf9b@phoenix> In-Reply-To: <20251113163301.285550-1-stephen@networkplumber.org> References: <20251113163301.285550-1-stephen@networkplumber.org> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org On Thu, 13 Nov 2025 08:33:01 -0800 Stephen Hemminger wrote: > Gcc-16 detects use of uninitialized variable. > ./drivers/net/mlx5/linux/mlx5_ethdev_os.c: In function =E2=80=98mlx5_link= _update=E2=80=99: > ../drivers/net/mlx5/linux/mlx5_ethdev_os.c:539:15: warning: =E2=80=98*(lo= ng unsigned int *)((char *)&dev_link + offsetof(struct rte_eth_link, ))=E2=80=99 may be used uninitialized [-Wmaybe-uninitialized] > 539 | ret =3D !!memcmp(&dev->data->dev_link, &dev_link, > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > 540 | sizeof(struct rte_eth_link)); > | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > ../drivers/net/mlx5/linux/mlx5_ethdev_os.c:515:29: note: =E2=80=98*(long = unsigned int *)((char *)&dev_link + offsetof(struct rte_eth_link, ))= =E2=80=99 was declared here > 515 | struct rte_eth_link dev_link; > | >=20 > If the retry loop exits the code would do memcmp against unintialized > stack value. Resolve by intializing to zero. >=20 > Fixes: 1256805dd54d ("net/mlx5: move Linux-specific functions") > Cc: stable@dpdk.org >=20 > Signed-off-by: Stephen Hemminger > --- The build fails with clang because of the (annoying) use of pedantic in thi= s driver. I will let mlx5 maintainers fix it then. ../drivers/net/mlx5/linux/mlx5_ethdev_os.c:515:33: error: use of GNU empty = initializer extension [-Werror,-Wgnu-empty-initializer] struct rte_eth_link dev_link =3D { };