]> www.infradead.org Git - users/dwmw2/linux.git/commitdiff
rtnl: add helper to check if rtnl group has listeners
authorJamal Hadi Salim <jhs@mojatatu.com>
Fri, 8 Dec 2023 19:28:41 +0000 (16:28 -0300)
committerJakub Kicinski <kuba@kernel.org>
Tue, 12 Dec 2023 02:52:56 +0000 (18:52 -0800)
As of today, rtnl code creates a new skb and unconditionally fills and
broadcasts it to the relevant group. For most operations this is okay
and doesn't waste resources in general.

When operations are done without the rtnl_lock, as in tc-flower, such
skb allocation, message fill and no-op broadcasting can happen in all
cores of the system, which contributes to system pressure and wastes
precious cpu cycles when no one will receive the built message.

Introduce this helper so rtnetlink operations can simply check if someone
is listening and then proceed if necessary.

Reviewed-by: Jiri Pirko <jiri@nvidia.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Signed-off-by: Jamal Hadi Salim <jhs@mojatatu.com>
Signed-off-by: Victor Nogueira <victor@mojatatu.com>
Signed-off-by: Pedro Tammela <pctammela@mojatatu.com>
Link: https://lore.kernel.org/r/20231208192847.714940-2-pctammela@mojatatu.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
include/linux/rtnetlink.h

index 3d6cf306cd55e9184b3e56c7172c3e78a43cf42e..a7d757e96c55f7fe4605750ea0cd3947ec2c307d 100644 (file)
@@ -130,4 +130,11 @@ extern int ndo_dflt_bridge_getlink(struct sk_buff *skb, u32 pid, u32 seq,
 
 extern void rtnl_offload_xstats_notify(struct net_device *dev);
 
+static inline int rtnl_has_listeners(const struct net *net, u32 group)
+{
+       struct sock *rtnl = net->rtnl;
+
+       return netlink_has_listeners(rtnl, group);
+}
+
 #endif /* __LINUX_RTNETLINK_H */