[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