* Kyle Mestery (kmestery) (kmest...@cisco.com) wrote: > On Nov 27, 2012, at 9:24 PM, Chris Wright <chr...@sous-sol.org> wrote: > > * Kyle Mestery (kmest...@cisco.com) wrote: > >> + > >> + vxlan_port->port = dst_port; > >> + vxlan_port->count++; > >> + hlist_add_head(&vxlan_port->hash_node, > >> + vxlan_hash_bucket(net, dst_port)); > > > > A little unusual to have a hashtable for this. Is this expected to be > > temporary until IANA port is allocated? > > > The idea is to allow destination port configuration on a per-tunnel basis, > so I needed to track this. As Jesse said, maybe a hash table was overkill > here, but that's a design point I think (e.g. I could have done a linked > list instead).
Right, but destination port is meant to be a fixed port number. So I assume this is temporary, or are you saying this is a permanent feature? thanks, -chris _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
