Re: [PATCH] opensm/osm_vendor_ibumad: Use OSM_UMAD_MAX_AGENTS rather than UMAD_CA_MAX_AGENTS

Sasha Khapyorsky
2009-10-22T13:35:43Z

On 08:49 Thu 22 Oct , Hal Rosenstock wrote:
> >>  /****s* OpenSM: Vendor UMAD/osm_ca_info_t
> >>  * NAME
> >>  *   osm_ca_info_t
> >> @@ -154,7 +156,7 @@ typedef struct _osm_vendor {
> >>       osm_ca_info_t *p_ca_info;
> >>       uint32_t timeout;
> >>       int max_retries;
> >> -     osm_bind_handle_t agents[UMAD_CA_MAX_AGENTS];
> >> +     osm_bind_handle_t agents[OSM_UMAD_MAX_AGENTS];
> >
> > Any reason?
>
> To make the umad vendor layer "stand on its own". This was discussed
> in the Windows porting issues. See thread which ends
> http://lists.openfabrics.org/pipermail/ofw/2009-October/005819.html

I (and guess that many other OpenSM source and change log readers) am
not subscribed on ofw list and unaware about this discussion. Could you
summarize (in the patch change log) what is going on there and what is
motivation for this change?