summaryrefslogtreecommitdiffstats
path: root/net/devlink
diff options
context:
space:
mode:
authorJiri Pirko <jiri@nvidia.com>2023-01-18 16:21:10 +0100
committerJakub Kicinski <kuba@kernel.org>2023-01-19 19:08:37 -0800
commit9f167327efecc3977deff0c852760e3759b0c2a7 (patch)
treea250b1c68c34d5867056964010b08d12f24add15 /net/devlink
parent1dea3b4e4c52f4bed64d1c527d548e82ccaea15a (diff)
downloadlinux-stable-9f167327efecc3977deff0c852760e3759b0c2a7.tar.gz
linux-stable-9f167327efecc3977deff0c852760e3759b0c2a7.tar.bz2
linux-stable-9f167327efecc3977deff0c852760e3759b0c2a7.zip
devlink: remove devl*_port_health_reporter_destroy()
Remove port-specific health reporter destroy function as it is currently the same as the instance one so no longer needed. Inline __devlink_health_reporter_destroy() as it is no longer called from multiple places. Signed-off-by: Jiri Pirko <jiri@nvidia.com> Reviewed-by: Jacob Keller <jacob.e.keller@intel.com> Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'net/devlink')
-rw-r--r--net/devlink/leftover.c35
1 files changed, 2 insertions, 33 deletions
diff --git a/net/devlink/leftover.c b/net/devlink/leftover.c
index 548e4a379e48..d4e8c285b255 100644
--- a/net/devlink/leftover.c
+++ b/net/devlink/leftover.c
@@ -7442,13 +7442,6 @@ devlink_health_reporter_put(struct devlink_health_reporter *reporter)
devlink_health_reporter_free(reporter);
}
-static void
-__devlink_health_reporter_destroy(struct devlink_health_reporter *reporter)
-{
- list_del(&reporter->list);
- devlink_health_reporter_put(reporter);
-}
-
/**
* devl_health_reporter_destroy - destroy devlink health reporter
*
@@ -7459,7 +7452,8 @@ devl_health_reporter_destroy(struct devlink_health_reporter *reporter)
{
devl_assert_locked(reporter->devlink);
- __devlink_health_reporter_destroy(reporter);
+ list_del(&reporter->list);
+ devlink_health_reporter_put(reporter);
}
EXPORT_SYMBOL_GPL(devl_health_reporter_destroy);
@@ -7474,31 +7468,6 @@ devlink_health_reporter_destroy(struct devlink_health_reporter *reporter)
}
EXPORT_SYMBOL_GPL(devlink_health_reporter_destroy);
-/**
- * devl_port_health_reporter_destroy - destroy devlink port health reporter
- *
- * @reporter: devlink health reporter to destroy
- */
-void
-devl_port_health_reporter_destroy(struct devlink_health_reporter *reporter)
-{
- devl_assert_locked(reporter->devlink);
-
- __devlink_health_reporter_destroy(reporter);
-}
-EXPORT_SYMBOL_GPL(devl_port_health_reporter_destroy);
-
-void
-devlink_port_health_reporter_destroy(struct devlink_health_reporter *reporter)
-{
- struct devlink *devlink = reporter->devlink;
-
- devl_lock(devlink);
- devl_port_health_reporter_destroy(reporter);
- devl_unlock(devlink);
-}
-EXPORT_SYMBOL_GPL(devlink_port_health_reporter_destroy);
-
static int
devlink_nl_health_reporter_fill(struct sk_buff *msg,
struct devlink_health_reporter *reporter,