On Thu, Sep 10, 2026 at 08:09:01PM +0200, Björn Töpel wrote: > From: Jakub Kicinski <[email protected]> > > netdev_config manipulation will become slightly more complicated > soon and will be used by both ethtool and the queue API. > Encapsulate the logic in helper functions. > > Signed-off-by: Jakub Kicinski <[email protected]> > Signed-off-by: Björn Töpel <[email protected]>
Reviewed-by: Breno Leitao <[email protected]> > +void __netdev_free_config(struct netdev_config *cfg) > +{ > + kfree(cfg); > +} When I read this one, I was expecting __netdev_free_config() to "become more complicated", but, it continues to alway be a single kfree(). How is this helper useful? --breno
