Re: [PATCH net-next 12/14] devlink: uniformly take the devlink instance lock in the dump loop

Jiri Pirko
2023-01-04T15:52:56Z

Wed, Jan 04, 2023 at 05:16:34AM CET, kuba@kernel.org wrote:
>Move the lock taking out of devlink_nl_cmd_region_get_devlink_dumpit().
>This way all dumps will take the instance lock in the main iteration
>loop directly, making refactoring and reading the code easier.
>
>Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>
>Signed-off-by: Jakub Kicinski <kuba@kernel.org>

Reviewed-by: Jiri Pirko <jiri@nvidia.com>