[ofw] [PATCH] complib: add cl_nodenamemap

[Smith, Stan]

Sean Hefty wrote:
>> also, the implementation of cl_nodenamemap could be platform
>> dependent, with fopen, strdup, etc. replaced with non CRT calls that
>> are available.  I'll look into this option.
>
> These are the calls that need to be replaced, and some replacement
> calls:
>
> __imp_free                    HeapFree
> __imp__strtoui64
> __imp_malloc          HeapAlloc
> __imp_strerror                can remove
> __imp__errno          can remove
> __imp_fgets
> __imp_strtok
> __imp_fopen                   CreateFile
> __imp_fprintf         can remove
> __imp__strdup
> __imp__iob                    no idea where this comes from
> __imp_fclose          CloseHandle
>
> I don't know of easy replacement calls for the other functions.  We
> could obviously implement our own versions of these, but that doesn't
> seem worth it. The more painful calls to deal with are strtoui64 and
> fgets.  Thoughts anyone?

You're on a slippery slope....
Revert back to the previously working cl_nodenamemap.c as a standalone .obj file in infiniband_diags\src which is linked into all 2-or-3 diags apps which need it.
Make a Windows only version and link with MSVCRT.

My $0.02 worth.

>
> - Sean
>
> _______________________________________________
> ofw mailing list
> ofw at lists.openfabrics.org
> http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw