# Re: \[PATCH net-next 03/10\] devlink: Serialize access to rate domains

- From: Jakub Kicinski \<kuba@kernel.org\>
- Date: 2025-02-26 01:40:07 
- List: netdev
- Page: https://ratatoskr.run/netdev/2025/02/2944509
- Thread: https://ratatoskr.run/netdev/2025/02/2925900/t.md

```text
On Tue, 25 Feb 2025 14:36:07 +0100 Jiri Pirko wrote:
> >The problem comes from having a devlink instance per function /
> >port rather than for the ASIC. Spawn a single instance and the
> >problem will go away 🤷️  
> 
> Yeah, we currently have VF devlink ports created under PF devlink instance.
> That is aligned with PCI geometry. If we have a single per-ASIC parent
> devlink, this does not change and we still need to configure cross
> PF devlink instances.

Why would there still be PF instances? I'm not suggesting that you
create a hierarchy of instances.

> The only benefit I see is that we don't need rate domain, but
> we can use parent devlink instance lock instead. The locking ordering
> might be a bit tricky to fix though.
```