Or Gerlitz wrote:
Pavel Shamis (Pasha) wrote:
The patch is attached
Thanks, this patch basically replaces checks for the device transport
type to be IB to a check that makes sure either the former happens or
the port transport type is rdmaoe. As Jason, Tziporet and noted, the
port transport type seems to be bad and non-comapatible/operable idea,
so it should and probably could be avoided.
The only reason for this changes is the fact that for IB devices we
prefer to use our own open mpi connection managers. In case if we will
decide to use RDMA-CM for all devices the number of changes will be zero...
I see another patch @
http://www.open-mpi.org/community/lists/devel/2009/11/7063.php
can you send that one as well. The you sent patch isn't signed so I
can't address the author in further replies (unless you are the
author), also it wasn't generated with the -p option of diff which
would show for each change what is the effected function, doing so
would help in the review.
Ohh. It was our first patch. Some of iwarp devices have the "first
packet" issue. So the RDMACM code in Ompi have bunch of work-around code
that try to resolve the issue on MPI level. We don't have such problems
with IB, so we tried to disable the work-around flow for IB devices, but
it did not work well. So we decided to use the current ompi code as is,
in future maybe we will implement own ompi rdmacm code that will not
have all this work around flows.
Regards,
Pasha
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
