diff options
| author | Eric Dumazet <edumazet@google.com> | 2025-01-07 17:47:01 +0300 |
|---|---|---|
| committer | Paolo Abeni <pabeni@redhat.com> | 2025-01-09 15:24:57 +0300 |
| commit | a3b3d2dc389568a77d0e25da17203e3616218e93 (patch) | |
| tree | 93aa3a5598c2339b4a3a1499c33917be93449daf | |
| parent | 2170a1f091486c2b5726a43c97a7d4b72073811c (diff) | |
| download | linux-a3b3d2dc389568a77d0e25da17203e3616218e93.tar.xz | |
net: hsr: remove synchronize_rcu() from hsr_add_port()
A synchronize_rcu() was added by mistake in commit
c5a759117210 ("net/hsr: Use list_head (and rcu) instead
of array for slave devices.")
RCU does not mandate to observe a grace period after
list_add_tail_rcu().
Signed-off-by: Eric Dumazet <edumazet@google.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Link: https://patch.msgid.link/20250107144701.503884-1-edumazet@google.com
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
| -rw-r--r-- | net/hsr/hsr_slave.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/net/hsr/hsr_slave.c b/net/hsr/hsr_slave.c index 006d6ef97e53..2a802a5de2ac 100644 --- a/net/hsr/hsr_slave.c +++ b/net/hsr/hsr_slave.c @@ -204,7 +204,6 @@ int hsr_add_port(struct hsr_priv *hsr, struct net_device *dev, } list_add_tail_rcu(&port->port_list, &hsr->ports); - synchronize_rcu(); master = hsr_port_get_hsr(hsr, HSR_PT_MASTER); netdev_update_features(master->dev); |