Re: [PATCH net-next 05/14] devlink: Decouple rate storage from associated devlink object

[Jiri Pirko]

Thu, Nov 20, 2025 at 02:09:17PM +0100, tariqt@xxxxxxxxxx wrote:
>From: Cosmin Ratiu <cratiu@xxxxxxxxxx>
>
>Devlink rate leafs and nodes were stored in their respective devlink
>objects pointed to by devlink_rate->devlink.
>
>This patch removes that association by introducing the concept of
>'rate node devlink', which is where all rates that could link to each
>other are stored. For now this is the same as devlink_rate->devlink.
>
>After this patch, the devlink rates stored in this devlink instance
>could potentially be from multiple other devlink instances. So all rate
>node manipulation code was updated to:
>- correctly compare the actual devlink object during iteration.
>- maybe acquire additional locks (noop for now).
>
>Signed-off-by: Cosmin Ratiu <cratiu@xxxxxxxxxx>
>Reviewed-by: Carolina Jubran <cjubran@xxxxxxxxxx>
>Signed-off-by: Tariq Toukan <tariqt@xxxxxxxxxx>

Reviewed-by: Jiri Pirko <jiri@xxxxxxxxxx>