Skip to content

Commit 7666dbe

Browse files
Jiri PirkoPaolo Abeni
authored andcommitted
net: devlink: add WARN_ON_ONCE to check return value of unregister_netdevice_notifier_net() call
As the return value is not 0 only in case there is no such notifier block registered, add a WARN_ON_ONCE() to yell about it. Suggested-by: Ido Schimmel <idosch@idosch.org> Signed-off-by: Jiri Pirko <jiri@nvidia.com> Reviewed-by: Ido Schimmel <idosch@nvidia.com> Link: https://lore.kernel.org/r/20221125100255.1786741-1-jiri@resnulli.us Signed-off-by: Paolo Abeni <pabeni@redhat.com>
1 parent cb55ff7 commit 7666dbe

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

net/core/devlink.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9907,8 +9907,8 @@ void devlink_free(struct devlink *devlink)
99079907

99089908
xa_destroy(&devlink->snapshot_ids);
99099909

9910-
unregister_netdevice_notifier_net(devlink_net(devlink),
9911-
&devlink->netdevice_nb);
9910+
WARN_ON_ONCE(unregister_netdevice_notifier_net(devlink_net(devlink),
9911+
&devlink->netdevice_nb));
99129912

99139913
xa_erase(&devlinks, devlink->index);
99149914

0 commit comments

Comments
 (0)