Re: [PATCH net-next v2 3/5] devlink: Allow set specific ops callbacks dynamically
From: Jakub Kicinski
Date: Wed Oct 06 2021 - 09:36:03 EST
On Wed, 6 Oct 2021 06:37:44 +0300 Leon Romanovsky wrote:
> Let's chose random kernel version (v5.11)
> https://elixir.bootlin.com/linux/v5.11/source/net/core/devlink.c#L10245
> as you can see, it doesn't hold ANY driver core locks,
Nope, that is not what I see.
> so it can be called in any time during driver .probe() or .remove().
Having a callback invoked after registering to a subsystem (which used
to be the case for devlink before the changes) is _normal_.
You keep talking about .probe() like it's some magic period of complete
quiescence.
> Drivers that have implemented ops.flash_update() have no idea about that.
I bet.
I don't think this discussion is going anywhere, count me out.