On Wed, Aug 22, 2012 at 2:54 PM, Jesse Gross <[email protected]> wrote: > On Wed, Feb 22, 2012 at 7:58 PM, Pravin B Shelar <[email protected]> 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 <[email protected]> > > 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.
I will send it out shortly. btw, I did not see "datapath: Relax set header validation." patch in upstream repo. Thanks. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
