Skip to content

Conversation

@hppritcha
Copy link
Member

@yburette

Signed-off-by: Howard Pritchard [email protected]

@hppritcha hppritcha added this to the v2.1.0 milestone Aug 5, 2016
@yburette
Copy link
Member

yburette commented Aug 5, 2016

@hppritcha I don't think this is what you want.

av_type is the MCA parameter -- e.g. MTL_OFI_AV_MAP.
Instead, the av_attr that's passed to fi_av_open() requires either FI_AV_MAP or FI_AV_TABLE.
Maybe you can reuse hints->domain_attr->av_type...

Or am I missing something?

@artpol84
Copy link
Contributor

artpol84 commented Aug 6, 2016

bot:mellanox:retest

@hppritcha
Copy link
Member Author

@yburette you're correct. I'll fix.

@artpol84
Copy link
Contributor

artpol84 commented Aug 7, 2016

bot:mellanox:retest

@hppritcha hppritcha force-pushed the topic/minor_ofi_fix branch from c248803 to e46eee3 Compare August 10, 2016 22:10
@hppritcha
Copy link
Member Author

bot:mellanox:retest

@lanl-ompi
Copy link
Contributor

looks like the mlnx bug is due to the dynamic procs problem.

@lanl-ompi lanl-ompi merged commit 96c7762 into open-mpi:master Aug 16, 2016
@hppritcha hppritcha deleted the topic/minor_ofi_fix branch May 2, 2018 02:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants