Re: [patch net-next 7/8] devlink: convert reporters dump to devlink_nl_instance_iter_dump()

[Jiri Pirko <jiri@resnulli.us>]

Sat, Jan 07, 2023 at 10:49:08AM CET, jiri@resnulli.us wrote:
>From: Jiri Pirko <jiri@nvidia.com>
>
>Benefit from recently introduced instance iteration and convert
>reporters .dumpit generic netlink callback to use it.
>
>Signed-off-by: Jiri Pirko <jiri@nvidia.com>
>---
> net/devlink/devl_internal.h | 12 +----
> net/devlink/leftover.c      | 87 ++++++++++++++++---------------------
> net/devlink/netlink.c       |  6 ++-
> 3 files changed, 44 insertions(+), 61 deletions(-)
>
>diff --git a/net/devlink/devl_internal.h b/net/devlink/devl_internal.h
>index 52d958c1c977..2b0e119e7b84 100644
>--- a/net/devlink/devl_internal.h
>+++ b/net/devlink/devl_internal.h
>@@ -121,17 +121,6 @@ struct devlink_gen_cmd {
> 			struct netlink_callback *cb);
> };
> 
>-/* Iterate over registered devlink instances for devlink dump.
>- * devlink_put() needs to be called for each iterated devlink pointer
>- * in loop body in order to release the reference.
>- * Note: this is NOT a generic iterator, it makes assumptions about the use
>- *	 of @state and can only be used once per dumpit implementation.
>- */
>-#define devlink_dump_for_each_instance_get(msg, state, devlink)		\
>-	for (; (devlink = devlinks_xa_find_get(sock_net(msg->sk),	\
>-					       &state->instance));	\
>-	     state->instance++, state->idx = 0)
>-

I'm sorry, I managed to quash this during rebase by mistake. Sending v2.