netdev - [PATCH net-next 0/2] net: netpoll: remove dead code and speed up rtnl-locked region

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-Id: <20250918-netpoll_jv-v1-0-67d50eeb2c26@debian.org>
Date: Thu, 18 Sep 2025 05:25:56 -0700
From: Breno Leitao <leitao@...ian.org>
To: "David S. Miller" <davem@...emloft.net>, 
 Eric Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>, 
 Paolo Abeni <pabeni@...hat.com>, Simon Horman <horms@...nel.org>
Cc: netdev@...r.kernel.org, linux-kernel@...r.kernel.org, 
 Breno Leitao <leitao@...ian.org>, efault@....de, jv@...sburgh.net, 
 kernel-team@...a.com, calvin@...nvd.org
Subject: [PATCH net-next 0/2] net: netpoll: remove dead code and speed up
 rtnl-locked region

This patchset introduces two minor modernizations to the netpoll
infrastructure:

The first patch removes the unused netpoll pointer from the netpoll_info
structure. This member is redundant and its presence does not benefit
multi-instance setups, as reported by Jay Vosburgh. Eliminating it cleans up
the structure and removes unnecessary code.

The second patch updates the netpoll resource cleanup routine to use
synchronize_net() instead of synchronize_rcu(). As __netpoll_free() is always
called under the RTNL lock, using synchronize_net() leverages the more
efficient synchronize_rcu_expedited() in these contexts, reducing time spent in
critical sections and improving performance.

Both changes simplify maintenance and enhance efficiency without altering
netpoll behavior.

This is not expected to conflict with the other netpoll fix into `net`.

Signed-off-by: Breno Leitao <leitao@...ian.org>
---
Breno Leitao (2):
      net: netpoll: remove unused netpoll pointer from netpoll_info
      net: netpoll: use synchronize_net() instead of synchronize_rcu()

 include/linux/netpoll.h | 1 -
 net/core/netpoll.c      | 3 +--
 2 files changed, 1 insertion(+), 3 deletions(-)
---
base-commit: 64d2616972b77506731fa0122d3c48cb04dbe21b
change-id: 20250918-netpoll_jv-9bf6f640f308

Best regards,
--  
Breno Leitao <leitao@...ian.org>