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 B2A9A468E4 for ; Thu, 12 Jun 2025 23:07:43 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id ADFAE42DE4; Thu, 12 Jun 2025 23:07:43 +0200 (CEST) Received: from mail-wm1-f41.google.com (mail-wm1-f41.google.com [209.85.128.41]) by mails.dpdk.org (Postfix) with ESMTP id B717142E3A for ; Thu, 12 Jun 2025 23:07:42 +0200 (CEST) Received: by mail-wm1-f41.google.com with SMTP id 5b1f17b1804b1-4530921461aso13048685e9.0 for ; Thu, 12 Jun 2025 14:07:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1749762462; x=1750367262; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=9gTuI9j7+PZhsL7NOHthrNR1VhZIzRaNY2lXu7LjKPA=; b=eedlCk43TlvI7epXsy/nAT9C0q1SpJ5CWMwwpEQ36MudZ3n96QdPtpQJBxI/pclvXi seiSKxu/zBi+m56WVZCyeIuDslTxdtHvbcMOQLv0TMb9aRI0TTHDpdHY1lE7M/ihA6AD byqrDkK8w9U+wG38VmH6NnJIOUJvo59wl7IuzMLhzq6FfvBdWiHQho9ylLxulUHsW4Ho qHq5TJ1G7k/7APzZ0hXOcSgyWXngJ4E+F9M48mjRN1hRB0Do41z6dG3IyeLPonP2JJSL an8BdXpq0op5gyaRpH3HGXBsF4dqombDH6R3w9DMqKB6m9CPQP1FjA/dtOYbua3ms/gv gqBg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1749762462; x=1750367262; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=9gTuI9j7+PZhsL7NOHthrNR1VhZIzRaNY2lXu7LjKPA=; b=eYGHKvN7qQCEqphe0PCyjC32J9NvVLhZNJn5glkX0Doko+aNUYlgt3ahzTKu/8zuZt VE4vj28H7zu9OBbc5yIUazOr2b/mA4FhZkGe60AlarcTSWNOMGCy4uqiNH/Aei1FB3iT 88o935yM14rlTIIPfrKRtlwr7hFMZWmyUOHvBAMg17VcWczjP0xVlzGVIwZkQJkiwNmQ HShdhHP+9ofvP5608n/46B75f7ATgMAcQnaxYK452oCUKw4BDzRm86VAHy08fZGi1AXX BJfkObWiPzdK+CeI/tPfz2uAI05/jSdYbJSgQwdsxzXsD9f85HPp3U/ZAobSI0jd4jmE eBVw== X-Forwarded-Encrypted: i=1; AJvYcCUZe96hpt9jY9L0atBKAV+O4tz9WNbJICgJ5kXmhWhx+IhXekzB/8CYI3NC8vNZ8Mp09e9fe5M=@dpdk.org X-Gm-Message-State: AOJu0YxKg5YWPjf9H85l8Lgrpzw1Bg3aabk6gjJE1X+/VM2J+2YJJcQj lmdLsPvsjhCnx/2Ocmp/DRyLciw36j7fV/0HCrog8RW5YVNuaxlXvJ7P X-Gm-Gg: ASbGnctbzeda6EwHgFsIv3b7T44hXoONIH67nZy8L+UaVYZrVZfjHfMnoxc7vD/nZdi EeuIvF2EURZVlfpRuaZWU9Kn6E3qBJjbeThPRzBmoPR7UHHpKACGRFaeGjtMcI2JjKhGG7PwFvm NG6JFqxP8oqy1V59fa3wdL4ibRFEpLj+1OqelpScKVBHPEzsD1qhVBAp8eQ6Nv5cZP/Ti9wBW8X fbYrXxd6DEFgROM9x2X9z9u3C6aNiNbmkbC+vis+wV1NoRsRxThRjmusZAmOgJuDxcyJoXb5trU +jMIt7u7jeaepHb0Yp0Wc8yDk7zPjrcx2qWH5hqZ9QWOwKfsA6MzDmTW4+uyF9l6gpph X-Google-Smtp-Source: AGHT+IHudMwgbzribHUbLU/udQ524FwefuhBK7/hcucVDZuRzJUia1okYmmjgIWYvTJdmZEsLGAeiw== X-Received: by 2002:a05:600c:3505:b0:453:78f:fa9f with SMTP id 5b1f17b1804b1-45334a8bbbdmr4560205e9.11.1749762462060; Thu, 12 Jun 2025 14:07:42 -0700 (PDT) Received: from localhost ([2a01:4b00:d036:ae00:f2df:571a:ae4c:bef2]) by smtp.gmail.com with UTF8SMTPSA id 5b1f17b1804b1-4532e14f283sm31947215e9.27.2025.06.12.14.07.41 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 12 Jun 2025 14:07:41 -0700 (PDT) From: luca.boccassi@gmail.com To: Bruce Richardson Cc: Anatoly Burakov , dpdk stable Subject: patch 'net/fm10k/base: fix compilation warnings' has been queued to stable release 22.11.9 Date: Thu, 12 Jun 2025 22:06:20 +0100 Message-ID: <20250612210733.2506558-3-luca.boccassi@gmail.com> X-Mailer: git-send-email 2.47.2 In-Reply-To: <20250612210733.2506558-1-luca.boccassi@gmail.com> References: <20250612210733.2506558-1-luca.boccassi@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 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 Hi, FYI, your patch has been queued to stable release 22.11.9 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 06/14/25. So please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. This will indicate if there was any rebasing needed to apply to the stable branch. If there were code changes for rebasing (ie: not only metadata diffs), please double check that the rebase was correctly done. Queued patches are on a temporary branch at: https://github.com/bluca/dpdk-stable This queued commit can be viewed at: https://github.com/bluca/dpdk-stable/commit/11713936aef12cf6e66203e1ada9b58c1340542e Thanks. Luca Boccassi --- >From 11713936aef12cf6e66203e1ada9b58c1340542e Mon Sep 17 00:00:00 2001 From: Bruce Richardson Date: Fri, 28 Mar 2025 11:16:13 +0000 Subject: [PATCH] net/fm10k/base: fix compilation warnings [ upstream commit 48fc5188aa753824a4bd469f9c482e845404c963 ] The fixes required to re-enable warnings in the fm10k base code are trivial, so let's make the changes and get a clean compile without any warning disable flags. * provide definitions for the UNREFERENCED_PARAMETER macros * fix the spelling of the work "fallthrough" in comments * provide a definition of FM10K_READ_PCI_WORD in os_dep.h that marks the parameters as used. Fixes: 7223d200c227 ("fm10k: add base driver") Signed-off-by: Bruce Richardson Acked-by: Anatoly Burakov --- drivers/net/fm10k/base/fm10k_mbx.c | 2 +- drivers/net/fm10k/base/fm10k_osdep.h | 2 +- drivers/net/fm10k/base/fm10k_pf.c | 8 ++++---- drivers/net/fm10k/base/fm10k_type.h | 6 +++--- drivers/net/fm10k/base/meson.build | 14 +------------- 5 files changed, 10 insertions(+), 22 deletions(-) diff --git a/drivers/net/fm10k/base/fm10k_mbx.c b/drivers/net/fm10k/base/fm10k_mbx.c index 2bb0d82efe..9028403757 100644 --- a/drivers/net/fm10k/base/fm10k_mbx.c +++ b/drivers/net/fm10k/base/fm10k_mbx.c @@ -1602,7 +1602,7 @@ s32 fm10k_pfvf_mbx_init(struct fm10k_hw *hw, struct fm10k_mbx_info *mbx, mbx->mbmem_reg = FM10K_MBMEM_VF(id, 0); break; } - /* fallthough */ + /* fallthrough */ default: return FM10K_MBX_ERR_NO_MBX; } diff --git a/drivers/net/fm10k/base/fm10k_osdep.h b/drivers/net/fm10k/base/fm10k_osdep.h index 019fba5e25..977aa07c4b 100644 --- a/drivers/net/fm10k/base/fm10k_osdep.h +++ b/drivers/net/fm10k/base/fm10k_osdep.h @@ -67,7 +67,7 @@ typedef uint64_t u64; #define FM10K_PCI_REG_WRITE(reg, value) rte_write32((value), (reg)) /* not implemented */ -#define FM10K_READ_PCI_WORD(hw, reg) 0 +#define FM10K_READ_PCI_WORD(hw, reg) ((void)hw, (void)reg, 0) #define FM10K_WRITE_MBX(hw, reg, value) FM10K_WRITE_REG(hw, reg, value) #define FM10K_READ_MBX(hw, reg) FM10K_READ_REG(hw, reg) diff --git a/drivers/net/fm10k/base/fm10k_pf.c b/drivers/net/fm10k/base/fm10k_pf.c index 439dd224de..b54116a4b5 100644 --- a/drivers/net/fm10k/base/fm10k_pf.c +++ b/drivers/net/fm10k/base/fm10k_pf.c @@ -1362,19 +1362,19 @@ STATIC u8 fm10k_iov_supported_xcast_mode_pf(struct fm10k_vf_info *vf_info, case FM10K_XCAST_MODE_PROMISC: if (vf_flags & FM10K_VF_FLAG_PROMISC_CAPABLE) return FM10K_XCAST_MODE_PROMISC; - /* fallthough */ + /* fallthrough */ case FM10K_XCAST_MODE_ALLMULTI: if (vf_flags & FM10K_VF_FLAG_ALLMULTI_CAPABLE) return FM10K_XCAST_MODE_ALLMULTI; - /* fallthough */ + /* fallthrough */ case FM10K_XCAST_MODE_MULTI: if (vf_flags & FM10K_VF_FLAG_MULTI_CAPABLE) return FM10K_XCAST_MODE_MULTI; - /* fallthough */ + /* fallthrough */ case FM10K_XCAST_MODE_NONE: if (vf_flags & FM10K_VF_FLAG_NONE_CAPABLE) return FM10K_XCAST_MODE_NONE; - /* fallthough */ + /* fallthrough */ default: break; } diff --git a/drivers/net/fm10k/base/fm10k_type.h b/drivers/net/fm10k/base/fm10k_type.h index 84781ba9b2..437fb1c55e 100644 --- a/drivers/net/fm10k/base/fm10k_type.h +++ b/drivers/net/fm10k/base/fm10k_type.h @@ -83,9 +83,9 @@ struct fm10k_hw; #define FM10K_NOT_IMPLEMENTED 0x7FFFFFFF #define UNREFERENCED_XPARAMETER -#define UNREFERENCED_1PARAMETER(_p) (_p) -#define UNREFERENCED_2PARAMETER(_p, _q) do { (_p); (_q); } while (0) -#define UNREFERENCED_3PARAMETER(_p, _q, _r) do { (_p); (_q); (_r); } while (0) +#define UNREFERENCED_1PARAMETER(_p) (void)(_p) +#define UNREFERENCED_2PARAMETER(_p, _q) do { (void)(_p); (void)(_q); } while (0) +#define UNREFERENCED_3PARAMETER(_p, _q, _r) do { (void)(_p); (void)(_q); (void)(_r); } while (0) /* Start of PF registers */ #define FM10K_CTRL 0x0000 diff --git a/drivers/net/fm10k/base/meson.build b/drivers/net/fm10k/base/meson.build index bd19df27f7..a2640d1ee8 100644 --- a/drivers/net/fm10k/base/meson.build +++ b/drivers/net/fm10k/base/meson.build @@ -10,19 +10,7 @@ sources = [ 'fm10k_vf.c', ] -error_cflags = ['-Wno-unused-parameter', '-Wno-unused-value', - '-Wno-strict-aliasing', '-Wno-format-extra-args', - '-Wno-unused-variable', - '-Wno-implicit-fallthrough' -] -c_args = cflags -foreach flag: error_cflags - if cc.has_argument(flag) - c_args += flag - endif -endforeach - base_lib = static_library('fm10k_base', sources, dependencies: static_rte_eal, - c_args: c_args) + c_args: cflags) base_objs = base_lib.extract_all_objects(recursive: true) -- 2.47.2 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2025-06-12 22:06:24.016372433 +0100 +++ 0003-net-fm10k-base-fix-compilation-warnings.patch 2025-06-12 22:06:23.782042779 +0100 @@ -1 +1 @@ -From 48fc5188aa753824a4bd469f9c482e845404c963 Mon Sep 17 00:00:00 2001 +From 11713936aef12cf6e66203e1ada9b58c1340542e Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit 48fc5188aa753824a4bd469f9c482e845404c963 ] + @@ -16 +17,0 @@ -Cc: stable@dpdk.org @@ -21,5 +22,5 @@ - drivers/net/intel/fm10k/base/fm10k_mbx.c | 2 +- - drivers/net/intel/fm10k/base/fm10k_osdep.h | 2 +- - drivers/net/intel/fm10k/base/fm10k_pf.c | 8 ++++---- - drivers/net/intel/fm10k/base/fm10k_type.h | 6 +++--- - drivers/net/intel/fm10k/base/meson.build | 14 +------------- + drivers/net/fm10k/base/fm10k_mbx.c | 2 +- + drivers/net/fm10k/base/fm10k_osdep.h | 2 +- + drivers/net/fm10k/base/fm10k_pf.c | 8 ++++---- + drivers/net/fm10k/base/fm10k_type.h | 6 +++--- + drivers/net/fm10k/base/meson.build | 14 +------------- @@ -28 +29 @@ -diff --git a/drivers/net/intel/fm10k/base/fm10k_mbx.c b/drivers/net/intel/fm10k/base/fm10k_mbx.c +diff --git a/drivers/net/fm10k/base/fm10k_mbx.c b/drivers/net/fm10k/base/fm10k_mbx.c @@ -30,2 +31,2 @@ ---- a/drivers/net/intel/fm10k/base/fm10k_mbx.c -+++ b/drivers/net/intel/fm10k/base/fm10k_mbx.c +--- a/drivers/net/fm10k/base/fm10k_mbx.c ++++ b/drivers/net/fm10k/base/fm10k_mbx.c @@ -41,4 +42,4 @@ -diff --git a/drivers/net/intel/fm10k/base/fm10k_osdep.h b/drivers/net/intel/fm10k/base/fm10k_osdep.h -index a727a57481..5f8ff10474 100644 ---- a/drivers/net/intel/fm10k/base/fm10k_osdep.h -+++ b/drivers/net/intel/fm10k/base/fm10k_osdep.h +diff --git a/drivers/net/fm10k/base/fm10k_osdep.h b/drivers/net/fm10k/base/fm10k_osdep.h +index 019fba5e25..977aa07c4b 100644 +--- a/drivers/net/fm10k/base/fm10k_osdep.h ++++ b/drivers/net/fm10k/base/fm10k_osdep.h @@ -54 +55 @@ -diff --git a/drivers/net/intel/fm10k/base/fm10k_pf.c b/drivers/net/intel/fm10k/base/fm10k_pf.c +diff --git a/drivers/net/fm10k/base/fm10k_pf.c b/drivers/net/fm10k/base/fm10k_pf.c @@ -56,2 +57,2 @@ ---- a/drivers/net/intel/fm10k/base/fm10k_pf.c -+++ b/drivers/net/intel/fm10k/base/fm10k_pf.c +--- a/drivers/net/fm10k/base/fm10k_pf.c ++++ b/drivers/net/fm10k/base/fm10k_pf.c @@ -82 +83 @@ -diff --git a/drivers/net/intel/fm10k/base/fm10k_type.h b/drivers/net/intel/fm10k/base/fm10k_type.h +diff --git a/drivers/net/fm10k/base/fm10k_type.h b/drivers/net/fm10k/base/fm10k_type.h @@ -84,2 +85,2 @@ ---- a/drivers/net/intel/fm10k/base/fm10k_type.h -+++ b/drivers/net/intel/fm10k/base/fm10k_type.h +--- a/drivers/net/fm10k/base/fm10k_type.h ++++ b/drivers/net/fm10k/base/fm10k_type.h @@ -99,4 +100,4 @@ -diff --git a/drivers/net/intel/fm10k/base/meson.build b/drivers/net/intel/fm10k/base/meson.build -index f24e453fd0..a2640d1ee8 100644 ---- a/drivers/net/intel/fm10k/base/meson.build -+++ b/drivers/net/intel/fm10k/base/meson.build +diff --git a/drivers/net/fm10k/base/meson.build b/drivers/net/fm10k/base/meson.build +index bd19df27f7..a2640d1ee8 100644 +--- a/drivers/net/fm10k/base/meson.build ++++ b/drivers/net/fm10k/base/meson.build @@ -107,4 +108,4 @@ --error_cflags = [ -- '-Wno-unused-parameter', -- '-Wno-unused-value', -- '-Wno-implicit-fallthrough', +-error_cflags = ['-Wno-unused-parameter', '-Wno-unused-value', +- '-Wno-strict-aliasing', '-Wno-format-extra-args', +- '-Wno-unused-variable', +- '-Wno-implicit-fallthrough'