On 3/8/22 15:39, Roi Dayan wrote: > > > On 2022-03-08 4:12 PM, Ilya Maximets wrote: >> On 3/8/22 09:21, Johannes Berg wrote: >>> On Mon, 2022-03-07 at 21:45 -0800, Jakub Kicinski wrote: >>>> >>>> Let me add some people I associate with genetlink work in my head >>>> (fairly or not) to keep me fair here. >>> >>> :) >>> >>>> It's highly unacceptable for user space to straight up rewrite kernel >>>> uAPI types >>>> >>> >>> Agree. >> >> I 100% agree with that and will work on the userspace part to make sure >> we're not adding anything to the kernel uAPI types. >> >> FWIW, the quick grep over usespace code shows similar problem with a few >> other types, but they are less severe, because they are provided as part >> of OVS actions and kernel doesn't send anything that wasn't previously >> set by userspace in that case. There still might be a problem during the >> downgrade of the userspace while kernel configuration remains intact, >> but that is not a common scenario. Will work on fixing that in userspace. >> No need to change the kernel uAPI for these, IMO. >> > > since its rc7 we end up with kernel and ovs broken with each other. > can we revert the kernel patches anyway and introduce them again later > when ovs userspace is also updated?
I don't think this patch is part of 5-17-rc7. AFAICT, it's a candidate for 5.18, so we should still have a bit of time. Am I missing something? Best regards, Ilya Maximets. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
