Re: [patch net-next] devlink: don't take instance lock for nested handle put

Jakub Kicinski
2023-10-09T15:15:33Z

On Sat, 7 Oct 2023 12:17:31 +0200 Jiri Pirko wrote:
>> Isn't the PF driver processing the "FW events"? A is PF here, and B
>> is SF, are you saying that the PF devlink instance can be completely
>> removed (not just unregistered, freed) before the SF instance is
>> unregistered?
>
> Kernel-wise, yes. The FW probably holds necessary resource until SF goes
> away.

I think kernel assuming that this should not happen and requiring
the PF driver to work around potentially stupid FW designs should
be entirely without our rights.