Skip to content

Devlink cleanup for delay event series

Leon Romanovsky lkml netdev 1 version Applied

Status from this message: This series was applied to netdev/net-next.git (refs/heads/master):

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest6 patches 11 repliesApplied 6/6

Patches in v1 All 6 applied

#Subject and filesSizeRepliesApplied
1/6devlink: Simplify devlink_pernet_pre_exit callnet/core/devlink.c+10 -100 repliesApplied
2/6devlink: Remove check of always valid devlink pointernet/core/devlink.c+38 -560 repliesApplied
3/6devlink: Count struct devlink consumersinclude/net/devlink.h net/core/devlink.c+172 -359 repliesApplied
4/6devlink: Use xarray to store devlink instancesinclude/net/devlink.h net/core/devlink.c+50 -221 replyApplied
5/6devlink: Clear whole devlink_flash_notify structnet/core/devlink.c+2 -21 replyApplied
6/6net: hns3: remove always exist devlink pointer checkdrivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_devlink.c drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_devlink.c+2 -140 repliesApplied

Reviews trailers per patch

2 of 6 patches have a Reviewed-by or Acked-by.

PatchJacob Keller
1/6
2/6
3/6
4/6R-b
5/6R-b
6/6

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH net-next 0/6] Devlink cleanup for delay event series

From: Leon Romanovsky <[email protected]>

Hi,

Jakub's request to make sure that devlink events are delayed and not printed till they fully accessible [1] requires us to implement delayed event notification system in the devlink.

In order to do it, I moved some of my patches (xarray e.t.c) from the future series to be before "Move devlink_register to be near devlink_reload_enable" [2].

That allows us to rely on DEVLINK_REGISTERED xarray mark to decide if to print event or not.

Other patches are simple cleanup which is needed anyway.

[1] https://lore.kernel.org/lkml/20210811071817.4af5ab34@kicinski-fedora-pc1c0hjn.dhcp.thefacebook.com
[2] https://lore.kernel.org/lkml/[email protected]

Next in the queue:
* Delay event series
* Move devlink_register to be near devlink_reload_enable"
* Extension of devlink_ops to be set dynamically
* devlink_reload_* delete
* Devlink locks rework to user xarray and reference counting
* ????

Thanks

Leon Romanovsky (6):
devlink: Simplify devlink_pernet_pre_exit call
devlink: Remove check of always valid devlink pointer
devlink: Count struct devlink consumers
devlink: Use xarray to store devlink instances
devlink: Clear whole devlink_flash_notify struct
net: hns3: remove always exist devlink pointer check

 .../hisilicon/hns3/hns3pf/hclge_devlink.c     |   8 +-
 .../hisilicon/hns3/hns3vf/hclgevf_devlink.c   |   8 +-
 include/net/devlink.h                         |   4 +-
 net/core/devlink.c                            | 391 ++++++++++++------
 4 files changed, 273 insertions(+), 138 deletions(-)
Signature · 1 line
2.31.1