From nobody Fri Apr 3 16:08:33 2026 Received: from mail-dy1-f201.google.com (mail-dy1-f201.google.com [74.125.82.201]) (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 E3E65372EF6 for ; Fri, 3 Apr 2026 07:14:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.82.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775200473; cv=none; b=LUEY/X/HHAjrVeUWnFiFxGKnVXyMKN8bLyh6C9fkZu2Dg480lpSYZ1gLf1r4A0oaYSTSULPGaLbfDBeKNv31uOiri8Oz+iTkkSJtKQrB/OsFc+3c67apTSQB5LSss9mEFncXkMFX1zAfYlZWWq5wVi9MHwt3MxMVCOJdUVGtBW0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775200473; c=relaxed/simple; bh=6pngi6AbJj9IjokAnMcUuaAPhwS8Is6Gt3b4MrrqM2Y=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=UcYmJ9En+xbB1+cFD0r2xJ5vVYlc1ti/d5x9q82Ncryu8vWoYV3kenMHHtUR/6dK1xifzQvGZOb25D2snb0H2WXs4CT1VHhsS1BiWQ5tBt/zPhVlMVyQERyY5E4EztRLIG2wBrwOXVe4VoRXvNrv8FkTxtPJDbsj7wH0NtqcFIQ= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--marcharvey.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=GsprvlM6; arc=none smtp.client-ip=74.125.82.201 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--marcharvey.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="GsprvlM6" Received: by mail-dy1-f201.google.com with SMTP id 5a478bee46e88-2cc75e79b97so1172419eec.1 for ; Fri, 03 Apr 2026 00:14:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20251104; t=1775200471; x=1775805271; darn=vger.kernel.org; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=U9tx+KxgSxiX42Rwpk7xjo8R+fCKMdHqCETpVq4j3oU=; b=GsprvlM6/t/1Tc1evZyoVFsDaUfHPBPrpFLPEeb86kfa8e+UBp/yQnW8OySMeYYysr Ba0TjZNq5BR0+2joeRgfi3yKvg3oagM0oEvdafRya/z797RY8FW2sLVUfBXUTNqGGsdL g/ZL+AVoFeRTN6fT99Uq9xAiD84HfM+1XBLkKyBNqNJbUUBNSl9t8PDgryCklY3KBKjZ 2GC5Kc7tpJt/RHodOyZYw2QSXcf9vxB+6Lu+IgrOvuWTuF1sx5DuNyOp5VeB3/qkZ5Gd Y8RFQ6TL88xvV6fcqNITQHs0Wg/yxIaqfQt0Kk6Dwj4VktN8h9CgHQ2A5bh4MHmRk9mB yF0A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1775200471; x=1775805271; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=U9tx+KxgSxiX42Rwpk7xjo8R+fCKMdHqCETpVq4j3oU=; b=XApKJB8aYKLdJfgwHziHrJNE7c8zVNxFp0RHrsM6EQeXszRhC0tnDVVGgBXpqMUQOp y6W/w6/k74PJ/XfBqhUZ/7rGwMdN2pkxjtZYG8ic93bfj591QdHxBCwMzauGoQWWAzVY 8v6JUZqEb1y28MWSgkX3V1sIc3UaEpQ06V7DnXtlyMMD1ypSWkr24HuUbZJ4ZuTnWMLU lJgwB/X5TSQc/HkLLt2Et2QAOz6vBuprG6ZH8rvrRR2+8SgCpVItZn2x/VacpKMkXqep pPT+8rkBm7umXcsijwOhAqlSUQYsectNxkO4JmR0kHHD3mMbDgxLcfYxC2PKPpiKKU5T yVow== X-Forwarded-Encrypted: i=1; AJvYcCUw5C2S1J1sLPB2k0nGyFsVbUj3Hkllc3SJyu+XKJfTOHdH2xmeNKxBChfRHkycb5ETwi3FmqWsUVizFw4=@vger.kernel.org X-Gm-Message-State: AOJu0YwJ60bTlmcvGGl7q36aGeKCEAw0z32hYPsDU1Jekhz1QtD6kDb+ EFj+7HYzUr9gWVUt+LYTGbmcOXdjIKxh2nsVC//CAXqZaeWo4VttzLXi3o5yVXzReCStuYOw0GR ZOdgYXZz2v+xaZ+CRBrk7wA== X-Received: from dybpk7.prod.google.com ([2002:a05:7301:4287:b0:2c5:d9bf:436c]) (user=marcharvey job=prod-delivery.src-stubby-dispatcher) by 2002:a05:7301:1e87:b0:2c6:cdb3:bd8d with SMTP id 5a478bee46e88-2cbf9afd61emr1068538eec.7.1775200470904; Fri, 03 Apr 2026 00:14:30 -0700 (PDT) Date: Fri, 03 Apr 2026 07:14:22 +0000 In-Reply-To: <20260403-teaming-driver-internal-v4-0-d3032f33ca25@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20260403-teaming-driver-internal-v4-0-d3032f33ca25@google.com> X-Developer-Key: i=marcharvey@google.com; a=ed25519; pk=OzOeciadbfF5Bug/4/hyEAwfrruSY4tn0Q0LocyYUL0= X-Developer-Signature: v=1; a=ed25519-sha256; t=1775200467; l=3717; i=marcharvey@google.com; s=20260401; h=from:subject:message-id; bh=6pngi6AbJj9IjokAnMcUuaAPhwS8Is6Gt3b4MrrqM2Y=; b=4kA3H08ExpxSDoz7Uk31FsSlbiXlBRJodlvsA8hHkFHhJOG41Za0obB3F3DKo/OyB+4DFTaut JpxnL9txSO6BS8MXI0ukTtUr9fBqS4lbzm/Z5RQY9U43zJBlDb9uMW2 X-Mailer: b4 0.14.3 Message-ID: <20260403-teaming-driver-internal-v4-1-d3032f33ca25@google.com> Subject: [PATCH net-next v4 01/10] net: team: Annotate reads and writes for mixed lock accessed values From: Marc Harvey To: Jiri Pirko , 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, Marc Harvey Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable The team_port's "index" and the team's "en_port_count" are read in the hot transmit path, but are only written to when holding the rtnl lock. Use READ_ONCE() for all lockless reads of these values, and use WRITE_ONCE() for all writes. Signed-off-by: Marc Harvey --- Changes in v4: - None Changes in v3: - None Changes in v2: - None --- drivers/net/team/team_core.c | 11 ++++++----- drivers/net/team/team_mode_random.c | 2 +- include/linux/if_team.h | 4 ++-- 3 files changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/net/team/team_core.c b/drivers/net/team/team_core.c index 566a5d102c23..becd066279a6 100644 --- a/drivers/net/team/team_core.c +++ b/drivers/net/team/team_core.c @@ -938,7 +938,8 @@ static void team_port_enable(struct team *team, { if (team_port_enabled(port)) return; - port->index =3D team->en_port_count++; + WRITE_ONCE(port->index, team->en_port_count); + WRITE_ONCE(team->en_port_count, team->en_port_count + 1); hlist_add_head_rcu(&port->hlist, team_port_index_hash(team, port->index)); team_adjust_ops(team); @@ -958,7 +959,7 @@ static void __reconstruct_port_hlist(struct team *team,= int rm_index) for (i =3D rm_index + 1; i < team->en_port_count; i++) { port =3D team_get_port_by_index(team, i); hlist_del_rcu(&port->hlist); - port->index--; + WRITE_ONCE(port->index, port->index - 1); hlist_add_head_rcu(&port->hlist, team_port_index_hash(team, port->index)); } @@ -973,8 +974,8 @@ static void team_port_disable(struct team *team, team->ops.port_disabled(team, port); hlist_del_rcu(&port->hlist); __reconstruct_port_hlist(team, port->index); - port->index =3D -1; - team->en_port_count--; + WRITE_ONCE(port->index, -1); + WRITE_ONCE(team->en_port_count, team->en_port_count - 1); team_queue_override_port_del(team, port); team_adjust_ops(team); team_lower_state_changed(port); @@ -1245,7 +1246,7 @@ static int team_port_add(struct team *team, struct ne= t_device *port_dev, netif_addr_unlock_bh(dev); } =20 - port->index =3D -1; + WRITE_ONCE(port->index, -1); list_add_tail_rcu(&port->list, &team->port_list); team_port_enable(team, port); netdev_compute_master_upper_features(dev, true); diff --git a/drivers/net/team/team_mode_random.c b/drivers/net/team/team_mo= de_random.c index 53d0ce34b8ce..169a7bc865b2 100644 --- a/drivers/net/team/team_mode_random.c +++ b/drivers/net/team/team_mode_random.c @@ -16,7 +16,7 @@ static bool rnd_transmit(struct team *team, struct sk_buf= f *skb) struct team_port *port; int port_index; =20 - port_index =3D get_random_u32_below(team->en_port_count); + port_index =3D get_random_u32_below(READ_ONCE(team->en_port_count)); port =3D team_get_port_by_index_rcu(team, port_index); if (unlikely(!port)) goto drop; diff --git a/include/linux/if_team.h b/include/linux/if_team.h index ccb5327de26d..06f4d7400c1e 100644 --- a/include/linux/if_team.h +++ b/include/linux/if_team.h @@ -77,7 +77,7 @@ static inline struct team_port *team_port_get_rcu(const s= truct net_device *dev) =20 static inline bool team_port_enabled(struct team_port *port) { - return port->index !=3D -1; + return READ_ONCE(port->index) !=3D -1; } =20 static inline bool team_port_txable(struct team_port *port) @@ -272,7 +272,7 @@ static inline struct team_port *team_get_port_by_index_= rcu(struct team *team, struct hlist_head *head =3D team_port_index_hash(team, port_index); =20 hlist_for_each_entry_rcu(port, head, hlist) - if (port->index =3D=3D port_index) + if (READ_ONCE(port->index) =3D=3D port_index) return port; return NULL; } --=20 2.53.0.1185.g05d4b7b318-goog