From nobody Sun Feb 8 17:21:44 2026 Received: from mail-wr1-f44.google.com (mail-wr1-f44.google.com [209.85.221.44]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 40BC632C31B for ; Sun, 18 Jan 2026 11:00:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.44 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768734043; cv=none; b=vD9dXfJjEm3s40xSvFrOeV4WxK/yGcoIDI+/db0loiwmKOHC4VZ1iMntfVtal8Mp+dJYlpRQt73nNF/DpBmb/LBgaQY8olNZLeCCgQFBtJWR1rZHZH3KCV+ENjAz7TUOWkz43aJDhqUjiR7QORPpW87cAEDU5ylmuGwF3f+GTq0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768734043; c=relaxed/simple; bh=4PDwzlCI+jgQG2jAxOz1RlldCa/d+sKTSpqV/8xAtWQ=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=SN35e6Fq+0N/8xt93ANJI2wYEiIlgfx2c8EumXv5w4gMKL0ASLGugPjeYs0mAoX4XQgclcN6zrMS3Q30rXDnTchxBkfinbJgEbM0cwUj5yFChrDbefgKC4GSlSOKnf1/LXJGlbTzkrZrD6XyLctLPvuJikdPGJZR+91RSGQdUy4= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=kzeORgji; arc=none smtp.client-ip=209.85.221.44 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="kzeORgji" Received: by mail-wr1-f44.google.com with SMTP id ffacd0b85a97d-42fbc305914so2842767f8f.0 for ; Sun, 18 Jan 2026 03:00:34 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1768734033; x=1769338833; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=JUk5UFagcf4IlgTEQ/iWJeI/RTRWAeLpBdR/nh29wa0=; b=kzeORgjiISvpqKR3m8sKjLUxWRqK13aUdgUcsnmPrhk6UWBGpX6b5UFKyUB5YE207n KLikJVpV/jJZZV26tn8o3OXz1hRbVbRN9a2s/La/XY54DKu3R6/XUj6iFS4U+VFAONlQ 6TDpl5DhvG8mnoypQ1CEUIozWY7raGEbAgc5SvYn2rKubHV/outZuCIZZKNLtR7+kxQl dJwnZzeEDeEF+8DlOnAz/h58i2bUSrIC/xlFo+cRogTSgpF2KCAU3dziIW/ocF2T8MWX f5qANr+ozzA+23GdKkAxYOjIAHct1RX+4VSUdqbgw6rPMHPVmftzbxaBU444hPUTI+Ew sLDA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1768734033; x=1769338833; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-gg:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=JUk5UFagcf4IlgTEQ/iWJeI/RTRWAeLpBdR/nh29wa0=; b=ZUSM4fyrNFH9WKm/m0lRoDdi1gKK1On+STpmSY7hn6Yyu7MW9hfD+pTBXllfgBQz78 gEiOTsiZgYLXOQ0DX7tqF/rBoO69FNAxUSZDoQZ/LV9CfDrnaRvZX6Psz0PRK56GCBta Nm8tp6gdU/J9iZDMM42kELWN8v/UXb0MS43bP2qhdaOaZm0QpolorJRSfBRV/N+ksWdU 0opkO5egwQ8d3zzcGzn0H0NbhehFFWBva2C7tV4kZca2UApmvj1vNbWjDN34v8oZVoDY 19ReOe4JhCg178+8vHSzx3LKasmQZg//BRNkmgqG46XGINkP0ejyjxkr4Wme26Vq0mGq mt0A== X-Forwarded-Encrypted: i=1; AJvYcCXW/NmAxf+cAlT5/P1nnL/z7SojM57esWpm9PPN59It4on18sua6ctiuzJDKy9CQ/PoQA+4AGpXIMauJiQ=@vger.kernel.org X-Gm-Message-State: AOJu0YyyamugKndgETPacnhdm8duMvcMxF6GqJAT9sXf49jviybyH13x SMurM/379iFnM7jEYWBn7yYphKZbb7V0rGkzZh+de6p0gTjoAMwJc1rl X-Gm-Gg: AY/fxX44dcoumGglnN5uHw6FTltI208ZlxaN+ZNt8u+KW034bNus8jq1u3O3cZmLawz C3wy5lvp49N6jiGH3Elz2h+4LE6Lh8oWmLuvydWLCfiqkqiOxp8dY0tJTp7YWBZ5B/DCqXtJ3hK rfpfXwaf7YQWU5pgGJd0x/TBJLRug5mx4djamAdK/NLjR7rwOcLVNARxoAWt4QbtTZGjT8kxDio tNb6IPqBIxhb3NswZqssfrmUI31bmFaFTCs4jeGtp9ivyZx9+v6E4hn5j1HrINI2ThGZDZi0rLa MHhPKO+VqQx2Uc4U5Yr9syNwaV+jij39fv7fPkhWh8sX2UtDcc9er2ekXe6pyyJQGWO8zRWBWGV ef2CzztZpz6uTgJnuU7m2PQQyDZ6fXnNcPfdGkPj5gJQBQygwhp5zlIVLpwrwNcrjj4gmhZ320k lDMArcPIH8zTYxTQ== X-Received: by 2002:a05:6000:2893:b0:430:f72e:c998 with SMTP id ffacd0b85a97d-43569bc59abmr9967735f8f.51.1768734032489; Sun, 18 Jan 2026 03:00:32 -0800 (PST) Received: from [192.168.1.243] ([143.58.192.3]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-4356992201csm16864635f8f.2.2026.01.18.03.00.31 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 18 Jan 2026 03:00:31 -0800 (PST) From: Andre Carvalho Date: Sun, 18 Jan 2026 11:00:25 +0000 Subject: [PATCH net-next v11 5/7] netconsole: introduce helpers for dynamic_netconsole_mutex lock/unlock Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260118-netcons-retrigger-v11-5-4de36aebcf48@gmail.com> References: <20260118-netcons-retrigger-v11-0-4de36aebcf48@gmail.com> In-Reply-To: <20260118-netcons-retrigger-v11-0-4de36aebcf48@gmail.com> To: Breno Leitao , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Shuah Khan , Simon Horman Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, Andre Carvalho X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=ed25519-sha256; t=1768734024; l=12692; i=asantostc@gmail.com; s=20250807; h=from:subject:message-id; bh=4PDwzlCI+jgQG2jAxOz1RlldCa/d+sKTSpqV/8xAtWQ=; b=n1tlGfJ0dqCeUafoiYG6d53iNV8/9mZo3d2HkLXyTSrR59v/rV15yYLeNpuN8h7pvpZl5SjU4 rjVFCtHsJ4yAdxfW/UQctV6IqTzCJi77U7G6Q9MF3GqUmsPz0MXBv6K X-Developer-Key: i=asantostc@gmail.com; a=ed25519; pk=eWre+RwFHCxkiaQrZLsjC67mZ/pZnzSM/f7/+yFXY4Q= This commit introduces two helper functions to perform lock/unlock on dynamic_netconsole_mutex providing no-op stub versions when compiled without CONFIG_NETCONSOLE_DYNAMIC and refactors existing call sites to use the new helpers. This is done following kernel coding style guidelines, in preparation for an upcoming change. It avoids the need for preprocessor conditionals in the call site and keeps the logic easier to follow. Signed-off-by: Andre Carvalho Reviewed-by: Breno Leitao --- drivers/net/netconsole.c | 97 +++++++++++++++++++++++++++++---------------= ---- 1 file changed, 58 insertions(+), 39 deletions(-) diff --git a/drivers/net/netconsole.c b/drivers/net/netconsole.c index 02a3463e8d24..46d990f3d904 100644 --- a/drivers/net/netconsole.c +++ b/drivers/net/netconsole.c @@ -213,6 +213,16 @@ static void netconsole_target_put(struct netconsole_ta= rget *nt) config_group_put(&nt->group); } =20 +static void dynamic_netconsole_mutex_lock(void) +{ + mutex_lock(&dynamic_netconsole_mutex); +} + +static void dynamic_netconsole_mutex_unlock(void) +{ + mutex_unlock(&dynamic_netconsole_mutex); +} + #else /* !CONFIG_NETCONSOLE_DYNAMIC */ =20 static int __init dynamic_netconsole_init(void) @@ -240,6 +250,15 @@ static void populate_configfs_item(struct netconsole_t= arget *nt, int cmdline_count) { } + +static void dynamic_netconsole_mutex_lock(void) +{ +} + +static void dynamic_netconsole_mutex_unlock(void) +{ +} + #endif /* CONFIG_NETCONSOLE_DYNAMIC */ =20 /* Check if the target was bound by mac address. */ @@ -495,9 +514,9 @@ static ssize_t sysdata_cpu_nr_enabled_show(struct confi= g_item *item, char *buf) struct netconsole_target *nt =3D to_target(item->ci_parent); bool cpu_nr_enabled; =20 - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); cpu_nr_enabled =3D !!(nt->sysdata_fields & SYSDATA_CPU_NR); - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); =20 return sysfs_emit(buf, "%d\n", cpu_nr_enabled); } @@ -509,9 +528,9 @@ static ssize_t sysdata_taskname_enabled_show(struct con= fig_item *item, struct netconsole_target *nt =3D to_target(item->ci_parent); bool taskname_enabled; =20 - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); taskname_enabled =3D !!(nt->sysdata_fields & SYSDATA_TASKNAME); - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); =20 return sysfs_emit(buf, "%d\n", taskname_enabled); } @@ -522,9 +541,9 @@ static ssize_t sysdata_release_enabled_show(struct conf= ig_item *item, struct netconsole_target *nt =3D to_target(item->ci_parent); bool release_enabled; =20 - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); release_enabled =3D !!(nt->sysdata_fields & SYSDATA_TASKNAME); - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); =20 return sysfs_emit(buf, "%d\n", release_enabled); } @@ -562,9 +581,9 @@ static ssize_t sysdata_msgid_enabled_show(struct config= _item *item, struct netconsole_target *nt =3D to_target(item->ci_parent); bool msgid_enabled; =20 - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); msgid_enabled =3D !!(nt->sysdata_fields & SYSDATA_MSGID); - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); =20 return sysfs_emit(buf, "%d\n", msgid_enabled); } @@ -584,7 +603,7 @@ static ssize_t enabled_store(struct config_item *item, unsigned long flags; ssize_t ret; =20 - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); ret =3D kstrtobool(buf, &enabled); if (ret) goto out_unlock; @@ -660,7 +679,7 @@ static ssize_t enabled_store(struct config_item *item, /* Deferred cleanup */ netconsole_process_cleanups(); out_unlock: - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); return ret; } =20 @@ -671,7 +690,7 @@ static ssize_t release_store(struct config_item *item, = const char *buf, bool release; ssize_t ret; =20 - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); if (nt->state =3D=3D STATE_ENABLED) { pr_err("target (%s) is enabled, disable to update parameters\n", config_item_name(&nt->group.cg_item)); @@ -687,7 +706,7 @@ static ssize_t release_store(struct config_item *item, = const char *buf, =20 ret =3D strnlen(buf, count); out_unlock: - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); return ret; } =20 @@ -698,7 +717,7 @@ static ssize_t extended_store(struct config_item *item,= const char *buf, bool extended; ssize_t ret; =20 - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); if (nt->state =3D=3D STATE_ENABLED) { pr_err("target (%s) is enabled, disable to update parameters\n", config_item_name(&nt->group.cg_item)); @@ -713,7 +732,7 @@ static ssize_t extended_store(struct config_item *item,= const char *buf, nt->extended =3D extended; ret =3D strnlen(buf, count); out_unlock: - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); return ret; } =20 @@ -722,18 +741,18 @@ static ssize_t dev_name_store(struct config_item *ite= m, const char *buf, { struct netconsole_target *nt =3D to_target(item); =20 - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); if (nt->state =3D=3D STATE_ENABLED) { pr_err("target (%s) is enabled, disable to update parameters\n", config_item_name(&nt->group.cg_item)); - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); return -EINVAL; } =20 strscpy(nt->np.dev_name, buf, IFNAMSIZ); trim_newline(nt->np.dev_name, IFNAMSIZ); =20 - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); return strnlen(buf, count); } =20 @@ -743,7 +762,7 @@ static ssize_t local_port_store(struct config_item *ite= m, const char *buf, struct netconsole_target *nt =3D to_target(item); ssize_t ret =3D -EINVAL; =20 - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); if (nt->state =3D=3D STATE_ENABLED) { pr_err("target (%s) is enabled, disable to update parameters\n", config_item_name(&nt->group.cg_item)); @@ -755,7 +774,7 @@ static ssize_t local_port_store(struct config_item *ite= m, const char *buf, goto out_unlock; ret =3D strnlen(buf, count); out_unlock: - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); return ret; } =20 @@ -765,7 +784,7 @@ static ssize_t remote_port_store(struct config_item *it= em, struct netconsole_target *nt =3D to_target(item); ssize_t ret =3D -EINVAL; =20 - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); if (nt->state =3D=3D STATE_ENABLED) { pr_err("target (%s) is enabled, disable to update parameters\n", config_item_name(&nt->group.cg_item)); @@ -777,7 +796,7 @@ static ssize_t remote_port_store(struct config_item *it= em, goto out_unlock; ret =3D strnlen(buf, count); out_unlock: - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); return ret; } =20 @@ -788,7 +807,7 @@ static ssize_t local_ip_store(struct config_item *item,= const char *buf, ssize_t ret =3D -EINVAL; int ipv6; =20 - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); if (nt->state =3D=3D STATE_ENABLED) { pr_err("target (%s) is enabled, disable to update parameters\n", config_item_name(&nt->group.cg_item)); @@ -802,7 +821,7 @@ static ssize_t local_ip_store(struct config_item *item,= const char *buf, =20 ret =3D strnlen(buf, count); out_unlock: - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); return ret; } =20 @@ -813,7 +832,7 @@ static ssize_t remote_ip_store(struct config_item *item= , const char *buf, ssize_t ret =3D -EINVAL; int ipv6; =20 - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); if (nt->state =3D=3D STATE_ENABLED) { pr_err("target (%s) is enabled, disable to update parameters\n", config_item_name(&nt->group.cg_item)); @@ -827,7 +846,7 @@ static ssize_t remote_ip_store(struct config_item *item= , const char *buf, =20 ret =3D strnlen(buf, count); out_unlock: - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); return ret; } =20 @@ -848,7 +867,7 @@ static ssize_t remote_mac_store(struct config_item *ite= m, const char *buf, u8 remote_mac[ETH_ALEN]; ssize_t ret =3D -EINVAL; =20 - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); if (nt->state =3D=3D STATE_ENABLED) { pr_err("target (%s) is enabled, disable to update parameters\n", config_item_name(&nt->group.cg_item)); @@ -863,7 +882,7 @@ static ssize_t remote_mac_store(struct config_item *ite= m, const char *buf, =20 ret =3D strnlen(buf, count); out_unlock: - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); return ret; } =20 @@ -984,7 +1003,7 @@ static ssize_t userdatum_value_store(struct config_ite= m *item, const char *buf, return -EMSGSIZE; =20 mutex_lock(&netconsole_subsys.su_mutex); - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); =20 ret =3D strscpy(udm->value, buf, sizeof(udm->value)); if (ret < 0) @@ -998,7 +1017,7 @@ static ssize_t userdatum_value_store(struct config_ite= m *item, const char *buf, goto out_unlock; ret =3D count; out_unlock: - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); mutex_unlock(&netconsole_subsys.su_mutex); return ret; } @@ -1026,7 +1045,7 @@ static ssize_t sysdata_msgid_enabled_store(struct con= fig_item *item, return ret; =20 mutex_lock(&netconsole_subsys.su_mutex); - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); curr =3D !!(nt->sysdata_fields & SYSDATA_MSGID); if (msgid_enabled =3D=3D curr) goto unlock_ok; @@ -1038,7 +1057,7 @@ static ssize_t sysdata_msgid_enabled_store(struct con= fig_item *item, =20 unlock_ok: ret =3D strnlen(buf, count); - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); mutex_unlock(&netconsole_subsys.su_mutex); return ret; } @@ -1055,7 +1074,7 @@ static ssize_t sysdata_release_enabled_store(struct c= onfig_item *item, return ret; =20 mutex_lock(&netconsole_subsys.su_mutex); - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); curr =3D !!(nt->sysdata_fields & SYSDATA_RELEASE); if (release_enabled =3D=3D curr) goto unlock_ok; @@ -1067,7 +1086,7 @@ static ssize_t sysdata_release_enabled_store(struct c= onfig_item *item, =20 unlock_ok: ret =3D strnlen(buf, count); - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); mutex_unlock(&netconsole_subsys.su_mutex); return ret; } @@ -1084,7 +1103,7 @@ static ssize_t sysdata_taskname_enabled_store(struct = config_item *item, return ret; =20 mutex_lock(&netconsole_subsys.su_mutex); - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); curr =3D !!(nt->sysdata_fields & SYSDATA_TASKNAME); if (taskname_enabled =3D=3D curr) goto unlock_ok; @@ -1096,7 +1115,7 @@ static ssize_t sysdata_taskname_enabled_store(struct = config_item *item, =20 unlock_ok: ret =3D strnlen(buf, count); - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); mutex_unlock(&netconsole_subsys.su_mutex); return ret; } @@ -1114,7 +1133,7 @@ static ssize_t sysdata_cpu_nr_enabled_store(struct co= nfig_item *item, return ret; =20 mutex_lock(&netconsole_subsys.su_mutex); - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); curr =3D !!(nt->sysdata_fields & SYSDATA_CPU_NR); if (cpu_nr_enabled =3D=3D curr) /* no change requested */ @@ -1130,7 +1149,7 @@ static ssize_t sysdata_cpu_nr_enabled_store(struct co= nfig_item *item, =20 unlock_ok: ret =3D strnlen(buf, count); - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); mutex_unlock(&netconsole_subsys.su_mutex); return ret; } @@ -1192,10 +1211,10 @@ static void userdatum_drop(struct config_group *gro= up, struct config_item *item) ud =3D to_userdata(&group->cg_item); nt =3D userdata_to_target(ud); =20 - mutex_lock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_lock(); update_userdata(nt); config_item_put(item); - mutex_unlock(&dynamic_netconsole_mutex); + dynamic_netconsole_mutex_unlock(); } =20 static struct configfs_attribute *userdata_attrs[] =3D { --=20 2.52.0