[ofw] Re: [PATCH] osmtest - Add OSM_CDECL to main() declaration

Sean Hefty sean.hefty at intel.com
Wed Oct 14 21:57:36 PDT 2009


>> Sean maintains a separate set of patches applied to IB diags in order to
>address issues like the x86 requirement for __cdecl on main().
>> Since OSM_CDECL was already defined in OFED opensm, it did not seem to be a
>major concession to prefix main() with it.
>> Options:
>> 1) utilize OSM_CDECL
>> 2) throw an ugly ifdef __WIN__ or ifdef __linux__ around main()
>> 3) Create more Windows-only patches required for the Windows implementation
>of OFED opensm & tools.
>
>Any options for windows compiler about call conventions?

To clarify, I maintain a single patch to handle this for the signal() callback
handler, not the ib-diags main() functions.

Here's what I use in the sources file for building the diags:

MSC_WARNING_LEVEL = /W3 /WX /wd4007

- Sean




More information about the ofw mailing list