[Mvapich-discuss] proposed patch to datatype option in OMB osu_util.c

[Raffenetti, Ken]

Hi,

I’d like to propose a patch to the -T/—type handling code in OMB.

The -T option currently accepts MPI datatype names by comparing only a
fixed-length prefix and then silently ignores unrecognized values. If a
user passes an unsupported datatype such as mpi_long_long, no datatype is
added to the benchmark option list and later default handling can run a
different datatype instead. This can make benchmark output misleading.

Use exact case-insensitive matches for the supported datatype names
and return PO_BAD_USAGE when the -T argument is not recognized.  This
keeps unsupported datatypes from silently falling back to a default
benchmark datatype and avoids unintended matches for datatypes such
as mpi_int64_t->mpi_int.

Thanks,
Ken
External recipient
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osu.edu/pipermail/mvapich-discuss/attachments/20260617/2924d95e/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: util.patch
Type: application/octet-stream
Size: 1796 bytes
Desc: util.patch
URL: <http://lists.osu.edu/pipermail/mvapich-discuss/attachments/20260617/2924d95e/attachment-0001.obj>