On Wed, Feb 22, 2012 at 7:58 PM, Pravin B Shelar <pshe...@nicira.com> wrote: > Namespace patch ported to upstream OVS. > > --8<--------------------------cut here-------------------------->8- > > Following patch adds support for network namespace to openvswitch. > > Signed-off-by: Pravin B Shelar <pshe...@nicira.com>
Applied to my upstream tree (at long last), thanks. I made a few changes and since I had to do them just to review it I figure it was easier to do it directly rather than bouncing it back and forth. They are: * Rebase to current net tree. The major source of conflicts was just copyright notices. * Updated commit message. The message that we have in the OVS tree isn't explicit enough about the side effects in regards to module unloading. We can't fix it there but we can do it here. * Apply the bug fix in 39435725ace48273700b55def25b2358f66ff398 (datapath: Fix use-after-free bug in dp_notify.) that was found after this initial version. * Apply the bug fix in 0ceaa66c393bb7e59126e66d578dd2b8431c2a49 (datapath: Fix namespace refcount leak on failed init.) that I found when reviewing this. * Roll together ovs_exit_net() and dp_destroy_all() since with genl_lock() we don't need the indirection anymore and dp_destroy_all()'s argument isn't type safe. Can you send out the patch to increase the number of ports on top on this? I think that is the last piece necessary to sync with upstream. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
