mbox series

[net-next,0/2] rtnetlink: rtnl_stats_dump() changes

Message ID 20240502113748.1622637-1-edumazet@google.com (mailing list archive)
Headers show
Series rtnetlink: rtnl_stats_dump() changes | expand

Message

Eric Dumazet May 2, 2024, 11:37 a.m. UTC 
Getting rid of RTNL in rtnl_stats_dump() looks challenging.

In the meantime, we can:

1) Avoid RTNL acquisition for the final NLMSG_DONE marker.

2) Use for_each_netdev_dump() instead of the net->dev_index_head[]
   hash table.

Eric Dumazet (2):
  rtnetlink: change rtnl_stats_dump() return value
  rtnetlink: use for_each_netdev_dump() in rtnl_stats_dump()

 net/core/rtnetlink.c | 61 +++++++++++++++++---------------------------
 1 file changed, 23 insertions(+), 38 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org May 3, 2024, 10:20 p.m. UTC | #1 
Hello:

This series was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Thu,  2 May 2024 11:37:46 +0000 you wrote:
> Getting rid of RTNL in rtnl_stats_dump() looks challenging.
> 
> In the meantime, we can:
> 
> 1) Avoid RTNL acquisition for the final NLMSG_DONE marker.
> 
> 2) Use for_each_netdev_dump() instead of the net->dev_index_head[]
>    hash table.
> 
> [...]

Here is the summary with links:
  - [net-next,1/2] rtnetlink: change rtnl_stats_dump() return value
    https://git.kernel.org/netdev/net-next/c/136c2a9a2a87
  - [net-next,2/2] rtnetlink: use for_each_netdev_dump() in rtnl_stats_dump()
    https://git.kernel.org/netdev/net-next/c/0feb396f7428

You are awesome, thank you!