Re: [ovs-dev] [PATCH 1/7] datapath-windows: fixes in OvsGetExtInfoIoctl()

2014-11-18 Thread Ankur Sharma
Acked-by: Ankur Sharma From: dev on behalf of Eitan Eliahu Sent: Wednesday, November 12, 2014 10:53 AM To: Nithin Raju Cc: dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH 1/7] datapath-windows: fixes in OvsGetExtInfoIoctl() Yes, we planned

Re: [ovs-dev] [PATCH 1/7] datapath-windows: fixes in OvsGetExtInfoIoctl()

2014-11-12 Thread Eitan Eliahu
DISPATCH level IRQL execution before holding the R/W Dispatch lock. Besides that It is good. Thanks, Eitan -Original Message- From: Nithin Raju Sent: Wednesday, November 12, 2014 10:49 AM To: Eitan Eliahu Cc: dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH 1/7] datapath-windows: fixes in

Re: [ovs-dev] [PATCH 1/7] datapath-windows: fixes in OvsGetExtInfoIoctl()

2014-11-12 Thread Nithin Raju
On Nov 12, 2014, at 9:48 AM, Eitan Eliahu wrote: > I don't see a reason why this function need to be called with gOvsCtrlLock > held. The contention is on the Hyper-V VSwitch port when a port could be > removed from a different thread context. But, this is taken care of by > dispatch lock. Su

Re: [ovs-dev] [PATCH 1/7] datapath-windows: fixes in OvsGetExtInfoIoctl()

2014-11-12 Thread Eitan Eliahu
I don't see a reason why this function need to be called with gOvsCtrlLock held. The contention is on the Hyper-V VSwitch port when a port could be removed from a different thread context. But, this is taken care of by dispatch lock. Thanks, Eitan -Original Message- From: dev [mailto:d