diff options
author | Sebastian Andrzej Siewior <bigeasy@linutronix.de> | 2024-02-02 17:37:46 +0100 |
---|---|---|
committer | Paolo Abeni <pabeni@redhat.com> | 2024-02-06 11:45:53 +0100 |
commit | 03ba6dc035c60991033529e630bd1552b2bca4d7 (patch) | |
tree | 1f46acef6a5baae748dc24232cdc8b93a6977fb9 /net/core/dst.c | |
parent | ead21932c336f47e2cebb3ef642dd9f485f18b29 (diff) |
net: dst: Make dst_destroy() static and return void.
Since commit 52df157f17e56 ("xfrm: take refcnt of dst when creating
struct xfrm_dst bundle") dst_destroy() returns only NULL and no caller
cares about the return value.
There are no in in-tree users of dst_destroy() outside of the file.
Make dst_destroy() static and return void.
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Reviewed-by: Eric Dumazet <edumazet@google.com>
Link: https://lore.kernel.org/r/20240202163746.2489150-1-bigeasy@linutronix.de
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
Diffstat (limited to 'net/core/dst.c')
-rw-r--r-- | net/core/dst.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/net/core/dst.c b/net/core/dst.c index 6838d3212c37..95f533844f17 100644 --- a/net/core/dst.c +++ b/net/core/dst.c @@ -96,7 +96,7 @@ void *dst_alloc(struct dst_ops *ops, struct net_device *dev, } EXPORT_SYMBOL(dst_alloc); -struct dst_entry *dst_destroy(struct dst_entry * dst) +static void dst_destroy(struct dst_entry *dst) { struct dst_entry *child = NULL; @@ -126,15 +126,13 @@ struct dst_entry *dst_destroy(struct dst_entry * dst) dst = child; if (dst) dst_release_immediate(dst); - return NULL; } -EXPORT_SYMBOL(dst_destroy); static void dst_destroy_rcu(struct rcu_head *head) { struct dst_entry *dst = container_of(head, struct dst_entry, rcu_head); - dst = dst_destroy(dst); + dst_destroy(dst); } /* Operations to mark dst as DEAD and clean up the net device referenced |