Re: [ovs-dev] [PATCH 18/18] ofproto/ofproto: Use relaxed atomics.

2014-08-29 Thread Jarno Rajahalme
Thanks for the reviews! Series apart from the liv/ovs-thread changes pushed to master. Jarno On Aug 28, 2014, at 12:22 PM, Ben Pfaff wrote: > On Fri, Aug 22, 2014 at 01:58:29PM -0700, Jarno Rajahalme wrote: >> Neither 'miss_config', 'n_missed', nor 'n_matched' is used to >> synchronize the s

Re: [ovs-dev] [PATCH 18/18] ofproto/ofproto: Use relaxed atomics.

2014-08-28 Thread Ben Pfaff
On Fri, Aug 22, 2014 at 01:58:29PM -0700, Jarno Rajahalme wrote: > Neither 'miss_config', 'n_missed', nor 'n_matched' is used to > synchronize the state of any other variable, so we can use relaxed > atomic operations on them. > > Signed-off-by: Jarno Rajahalme Acked-by: Ben Pfaff _

[ovs-dev] [PATCH 18/18] ofproto/ofproto: Use relaxed atomics.

2014-08-22 Thread Jarno Rajahalme
Neither 'miss_config', 'n_missed', nor 'n_matched' is used to synchronize the state of any other variable, so we can use relaxed atomic operations on them. Signed-off-by: Jarno Rajahalme --- ofproto/ofproto-dpif.c | 10 ++ ofproto/ofproto.c | 12 +++- 2 files changed, 13