[ofw] RE: [PATCH 1/2] contain pthreads defs with ifdef	HAVE_LIBPTHREAD
    Smith, Stan 
    stan.smith at intel.com
       
    Thu Nov  5 09:07:45 PST 2009
    
    
  
Stan C. Smith wrote:
> Contain pthread definitions with ifdef HAVE_LIBPTHREAD
>
> Signed-off-by: stan smith <stan.smith at 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
>       int umad_port_id;
>       void *receiver;
>       int issmfd;
The use of ifdef HAVE_LIBPTHREAD was based on existing HAVE_LIBPTHREAD usage in opensm/include/opensm/osm_stats.h @ line 97.
If the community prefers the use of macros then these two patches can be skipped.
Please advise on path to travel.
Stan.
    
    
More information about the ofw
mailing list