[ovs-dev] [PATCH v2] sparse: Avoid sparse warnings for additional pthread initializers.

Ben Pfaff blp at nicira.com
Mon Jul 22 21:03:13 UTC 2013


Thanks, I applied this version.

On Mon, Jul 22, 2013 at 01:55:34PM -0700, Andy Zhou wrote:
> Looks good. It works now in my local build. Thanks.
> 
> 
> On Mon, Jul 22, 2013 at 1:31 PM, Ben Pfaff <blp at nicira.com> wrote:
> 
> > Reported-by: Andy Zhou <azhou at nicira.com>
> > Signed-off-by: Ben Pfaff <blp at nicira.com>
> > ---
> > v1->v2: Avoid conflict with ovs-thread.h definitions.
> >
> >  include/sparse/pthread.h |   11 ++++++++++-
> >  1 files changed, 10 insertions(+), 1 deletions(-)
> >
> > diff --git a/include/sparse/pthread.h b/include/sparse/pthread.h
> > index 723c351..6cdf5c8 100644
> > --- a/include/sparse/pthread.h
> > +++ b/include/sparse/pthread.h
> > @@ -33,12 +33,21 @@ int pthread_rwlock_unlock(pthread_rwlock_t *rwlock)
> > OVS_RELEASES(rwlock);
> >  int pthread_cond_wait(pthread_cond_t *, pthread_mutex_t *mutex)
> >      OVS_MUST_HOLD(mutex);
> >
> > -/* Sparse complains about the proper PTHREAD_MUTEX_INITIALIZER definition.
> > +/* Sparse complains about the proper PTHREAD_*_INITIALIZER definitions.
> >   * Luckily, it's not a real compiler so we can overwrite it with something
> >   * simple. */
> >  #undef PTHREAD_MUTEX_INITIALIZER
> >  #define PTHREAD_MUTEX_INITIALIZER {}
> >
> > +#undef PTHREAD_RWLOCK_INITIALIZER
> > +#define PTHREAD_RWLOCK_INITIALIZER {}
> > +
> > +#undef PTHREAD_ADAPTIVE_MUTEX_INITIALIZER_NP
> > +#define PTHREAD_ADAPTIVE_MUTEX_INITIALIZER_NP {}
> > +
> > +#undef PTHREAD_ERRORCHECK_MUTEX_INITIALIZER_NP
> > +#define PTHREAD_ERRORCHECK_MUTEX_INITIALIZER_NP {}
> > +
> >  #define pthread_mutex_trylock(MUTEX)                    \
> >      ({                                                  \
> >          int retval = pthread_mutex_trylock(mutex);      \
> > --
> > 1.7.2.5
> >
> >



More information about the dev mailing list