Ompi: Cherry pick ob1 fixes to release branches

Created on 3 May 2019  路  11Comments  路  Source: open-mpi/ompi

Need to cherry-pick the commits from the PRs listed below to v2.x, v3.0.x, v3.1.x, and v4.0.x:

  • #6556 (e630046a4b82bc01379fb055af4c0e414c2a8e8f): pml/ob1: fixed local handle sent during PUT control message
  • #6621 (a16cf0e4dd6df4dea820fecedd5920df632935b8): Fix the leak of fragments for persistent sends

Note that a16cf0e4dd6df4dea820fecedd5920df632935b8 contains use of OPAL_THREAD_SUB_FETCH_SIZE_T and OPAL_THREAD_ADD_FETCH_SIZE_T, neither of which exist before v4.0.x. Porting to the older branches is therefore a bit complex -- I took a whack at it and was not confident enough in what I finally managed to get to compile.

blocker v2.x v3.0.x v3.1.x bug 鈿狅笍 WIP-DNM!

All 11 comments

FYI: @EmmanuelBRELLE

I'm working on a fix for the PUT issue. If there is no rush to push this in, let's wait for few days until I'm done.

@bosilca Any updates? We're looking at the possibility of 4.0.2 in the "near" future.

Not enough hours in a day. I'm on the datatype issue right now, so I put this one aside. I might get to it before the end of the week.

@bosilca Any updates? We're looking at the possibility of 4.0.2 in the "near" future.

Same problem here, the only "solution" I have found is to do:

export OMPI_MCA_btl_vader_backing_directory=/tmp

before the _mpirun_ execution.

Can I do something if I do not want to type that export at every run?

Thank you!

mpirun --mca btl_vader_backing_directory /tmp ... should be equivalent. You can also set MCA params in a file -- see https://www.open-mpi.org/faq/?category=tuning#setting-mca-params.

PR #6634 cherry-picks these two fixes to v4.0.x branch.
Removing the Target: v4.0.x label to reflect that.

@jsquyres Have these fixes been backported to v3.1.x and v3.0.x? Can we close this Issue?

I think this can be closed?

Closing - this went into the 4.0 series and is in master. 3.0 is closed as far as I know. If someone disagrees, please reopen.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

pozdneev picture pozdneev  路  4Comments

PeterGottesman picture PeterGottesman  路  6Comments

AboorvaDevarajan picture AboorvaDevarajan  路  5Comments

oneZeroOneZeroOne picture oneZeroOneZeroOne  路  5Comments

ggouaillardet picture ggouaillardet  路  5Comments