On Wed, 09 Aug 2023 22:59:47 +0200 Johannes Berg wrote: > On Wed, 2023-08-09 at 11:26 -0700, Jakub Kicinski wrote: > > Only three families use info->userhdr and fixed headers > > are discouraged for new families. So remove the pointer > > from struct genl_info to save some space. Compute > > the header pointer at runtime. Saved space will be used > > for a family pointer in later patches. > > Seems fine to me, but I'm not sure I buy the rationale that it's for > saving space - it's a single pointer on the stack? I'd probably argue > the computation being pointless for basically everyone except for a > handful users?
Fair, I'll update all the commit messages. > Reviewed-by: Johannes Berg <johan...@sipsolutions.net> Thanks! _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
