On Mon, 7 Mar 2022 10:49:31 +0200 Roi Dayan wrote: > >> I think there is a missing userspace fix. didnt verify yet. > >> but in ovs userspace odp-netlink.h created from > >> datapath/linux/compat/include/linux/openvswitch.h > >> and that file is not synced the change here. > >> So the new enum OVS_KEY_ATTR_IPV6_EXTHDRS is missing and also struct > >> ovs_key_ipv6_exthdrs which is needed in lib/udp-util.c > >> in struct ovs_flow_key_attr_lens to add expected len for > >> OVS_KEY_ATTR_IPV6_EXTHDR. > > > > I guess if this is creating backward compatibility issues, this > > patch should be reverted/fixed. As a kmod upgrade should not break > > existing deployments. > > it looks like it does. we can't work with ovs without reverting this. > can we continue with reverting this commit please?
Sure, can someone ELI5 what the problem is? What's "kmod upgrade" in this context a kernel upgrade or loading a newer module in older kernel? How can adding a new nl attr break user space? Does the user space actually care about the OVS_KEY_ATTR_TUNNEL_INFO wart? _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev