Re: [ovs-dev] [PATCH] datapath: Refactor get_dp() function into multiple access APIs

2014-07-21 Thread Andy Zhou
O.K. I have send out a V2 that reduced the number of APIs from 4 to 2. get_dp() as before, and get_dp_rcu(). V2 is a simpler change. On Mon, Jul 21, 2014 at 11:04 AM, Pravin Shelar wrote: > On Wed, Jul 16, 2014 at 11:42 PM, Andy Zhou wrote: >> Remove get_dp() API, it becomes get_dp__() which is

Re: [ovs-dev] [PATCH] datapath: Refactor get_dp() function into multiple access APIs

2014-07-21 Thread Pravin Shelar
On Wed, Jul 16, 2014 at 11:42 PM, Andy Zhou wrote: > Remove get_dp() API, it becomes get_dp__() which is for internal > use only. Added the following APIs to make it clear of the locking > requirement. > > o get_dp_rcu() requires its caller to hold rcu lock. > o get_dp_ovsl() requires i

[ovs-dev] [PATCH] datapath: Refactor get_dp() function into multiple access APIs

2014-07-16 Thread Andy Zhou
Remove get_dp() API, it becomes get_dp__() which is for internal use only. Added the following APIs to make it clear of the locking requirement. o get_dp_rcu() requires its caller to hold rcu lock. o get_dp_ovsl() requires its caller to hold ovs lock. o get_dp_ovsl_rcu() requires its c