On Sat, Apr 9, 2016 at 10:17 AM, Manish Chopra <manish.cho...@qlogic.com> wrote: > Rationale behind this change is that with some OVS configuration > UDP ports doesn't get notified to the driver using > .ndo_[add|del]_vxlan_port. So for the driver to work with > these specific ports in that environment we need to have them configured > on adapter by default for the required hardware offload support.
I think you are referring to old out of tree code - no version of upstream OVS does this. In addition, any old code won't work against the new kernels that would include this driver update anyways so there won't be a benefit in any case. Please just use the normal registration mechanism that is already exposed. I also noticed that in the Geneve case you aren't currently registering for port notifications and just using the assigned port number in all cases, which isn't right.
