[ovs-dev] [PATCHv5 1/2] ovs-thread: Add xpthread_barrier_*() wrappers.

Ben Pfaff blp at nicira.com
Tue Mar 4 20:00:36 UTC 2014


It's not really necessary, patch 1/2 makes sense on its own.

On Tue, Mar 04, 2014 at 10:06:28AM -0800, Joe Stringer wrote:
> Shall I wait for feedback on patch #2 before posting a fresh version of
> this?
> 
> 
> On 1 March 2014 10:43, Ben Pfaff <blp at nicira.com> wrote:
> 
> > Please.
> >
> > On Fri, Feb 28, 2014 at 02:43:27PM -0800, Joe Stringer wrote:
> > > I figured that if we're not using the retval currently, then it doesn't
> > > matter too much. But I'm happy to change it.
> > >
> > >
> > > On 28 February 2014 13:53, Ben Pfaff <blp at nicira.com> wrote:
> > >
> > > > On Fri, Feb 28, 2014 at 01:42:13PM -0800, Joe Stringer wrote:
> > > > > Signed-off-by: Joe Stringer <joestringer at nicira.com>
> > > >
> > > > I would have xpthread_barrier_wait() pass along the return value from
> > > > pthread_barrier_wait(), because some uses of barriers want that
> > > > information.
> > > >
> > > > Otherwise, looks good.
> > > >
> >



More information about the dev mailing list