On Thu, Apr 23, 2015 at 1:27 PM, Sorin Vinturis <[email protected]> wrote: > There is no need to use gOvsCtrlLock spinlock to guard the switch > context, as there is now the switch context's reference count used > for this purpose. > > Now the gOvsCtrlLock spinlock guards only one shared resource, the > OVS_OPEN_INSTANCE global instance array. > > v2: Atomically test and set if the instance is in attach process, to > protect filter attach routine against multiple calls. > > v3: Changed gOvsInAttach global variable type into 'LONG volatile'. > Also removed an unrelated change from SubscribeIoctl handler. > > v4: Rebased the patch. When you 'git am' your patch, the commit message ends up with v2, v3, v4 information. The purpose of that information is only for the reviewer and should not end up in the commit message of ovs repo. So you should add it like below in between the two '---'. Once you do that, you can locally do a 'git am' to see that your versioning does not end up in the log. (I applied the patch, with the changes, thanks!)
> > Signed-off-by: Sorin Vinturis <[email protected]> > Acked-by: Nithin Raju <[email protected]> > --- v2: Atomically test and set if the instance is in attach process, to protect filter attach routine against multiple calls. v3: Changed gOvsInAttach global variable type into 'LONG volatile'. Also removed an unrelated change from SubscribeIoctl handler. v4: Rebased the patch. --- _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
