On Tue, 23 Aug 2022 16:50:21 +0300 Andrey Zhadchenko wrote: > > Are you 100% sure that all user space making this call initializes > > dp_ifindex to 0? There is no validation in the kernel today that > > the field is not garbage as far as I can tell. > > > > If you are sure, please add the appropriate analysis to the commit msg. > > I am not sure that *all* users set dp_ifindex to zero. At least I do not > know about them. Primary ovs userspace ovs-vswitchd certainly set to > zero, others like WeaveNet do it too. But there can be more? > What is the policy regarding this? I can add a new attribute, it is not > hard.
IDK how many user space components driving OvS exist out there. We could risk it and reuse the field, but if we get it wrong and don't catch the regression before the final release is cut the result gets _really_ ugly. We will have _some_ user space out there expecting us to use ifindex and _some_ which expects it can pass garbage, so we can neither revert not leave it... If that makes sense. If using / adding attribute is not a hassle that'd be my vote. Using fixed structs like struct ovs_header is strongly discouraged for new families exactly because of situations like this :( _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
