Re: [PATCH 1/2] contain pthreads defs with ifdef HAVE_LIBPTHREAD

[Sasha Khapyorsky <sashak@voltaire.com>]

On 13:05 Wed 04 Nov     , Sean Hefty wrote:
> >Contain pthread definitions with ifdef HAVE_LIBPTHREAD
> >
> >Signed-off-by: stan smith <stan.smith@intel.com>
> >
> >diff --git a/opensm/include/vendor/osm_vendor_ibumad.h
> >b/opensm/include/vendor/osm_vendor_ibumad.h
> >index 0a4692d..d523412 100644
> >--- a/opensm/include/vendor/osm_vendor_ibumad.h
> >+++ b/opensm/include/vendor/osm_vendor_ibumad.h
> >@@ -160,8 +160,13 @@ typedef struct _osm_vendor {
> > 	char ca_names[OSM_UMAD_MAX_CAS][UMAD_CA_NAME_LEN];
> > 	vendor_match_tbl_t mtbl;
> > 	umad_port_t umad_port;
> >+#ifdef HAVE_LIBPTHREAD
> > 	pthread_mutex_t cb_mutex;
> > 	pthread_mutex_t match_tbl_mutex;
> >+#else
> >+	cl_mutex_t cb_mutex;
> >+	cl_mutex_t match_tbl_mutex;
> >+#endif
> 
> complib is available on both platforms, why not just use it?

We don't have cl_mutex_t in linux's implementation.

I didn't know that complib on windows already implements semantics which
are similar to most useful pthread things. That is good, and seems that
an easy porting could be done by using macros like:

#define pthread_* cl_thread_*

Sasha