On Wed, 25 Mar 2026 07:26:52 +0100 Przemek Kitszel wrote: > Current API makes it possible to access shared devlink instance's priv > data: > > void *devlink_shd_get_priv(struct devlink *devlink); > > but it is easy to forget (especially during rebase from "before shared > devlinks" era) and call: > > void *devlink_priv(struct devlink *devlink); > > which even has the same signature, so it's hard to catch the error.
The implicit conversion may make things hard to reason about. Are you sure you actually mean that it's "easy to forget" or it's easier for OOT transition? If we are worried about misuse we should instead add an accessor for "individual" (better name welcome) instance and WARN_ON() when devlink_priv() is used in the shared setup. Or add a third argument to devlink_priv() which will pass the size of the LHS ptr, and warn on attempts to access priv of the wrong size?
