Re: [ovs-dev] [PATCH 4/8] ofproto-dpif: Make vlan splinters thread safe.

2013-08-20 Thread Simon Horman
Thanks, I feel better about my understanding of the code now. I'll compose a patch in the not to distant future. On Tue, Aug 20, 2013 at 05:44:49PM -0700, Ethan Jackson wrote: > Just an oversight. Feel free to fix it. > > Ethan > > On Mon, Aug 19, 2013 at 6:29 PM, Simon Horman wrote: > > On Sa

Re: [ovs-dev] [PATCH 4/8] ofproto-dpif: Make vlan splinters thread safe.

2013-08-20 Thread Ethan Jackson
Just an oversight. Feel free to fix it. Ethan On Mon, Aug 19, 2013 at 6:29 PM, Simon Horman wrote: > On Sat, Aug 03, 2013 at 06:42:06PM -0700, Ethan Jackson wrote: >> Signed-off-by: Ethan Jackson >> --- >> ofproto/ofproto-dpif.c | 69 >> +++- >>

Re: [ovs-dev] [PATCH 4/8] ofproto-dpif: Make vlan splinters thread safe.

2013-08-19 Thread Simon Horman
On Sat, Aug 03, 2013 at 06:42:06PM -0700, Ethan Jackson wrote: > Signed-off-by: Ethan Jackson > --- > ofproto/ofproto-dpif.c | 69 > +++- > 1 file changed, 51 insertions(+), 18 deletions(-) > > diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-d

Re: [ovs-dev] [PATCH 4/8] ofproto-dpif: Make vlan splinters thread safe.

2013-08-05 Thread Ethan Jackson
> I wonder whether a rwlock is warranted here. Probably. Ill leave that for when we go back and optimize. Ethan X-CudaMail-Whitelist-To: dev@openvswitch.org ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 4/8] ofproto-dpif: Make vlan splinters thread safe.

2013-08-05 Thread Ben Pfaff
On Sat, Aug 03, 2013 at 06:42:06PM -0700, Ethan Jackson wrote: > Signed-off-by: Ethan Jackson I wonder whether a rwlock is warranted here. Acked-by: Ben Pfaff ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH 4/8] ofproto-dpif: Make vlan splinters thread safe.

2013-08-03 Thread Ethan Jackson
Signed-off-by: Ethan Jackson --- ofproto/ofproto-dpif.c | 69 +++- 1 file changed, 51 insertions(+), 18 deletions(-) diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c index 17bc12f..e2dcd3f 100644 --- a/ofproto/ofproto-dpif.c +++ b/ofproto