Skip to content
This repository was archived by the owner on Sep 30, 2022. It is now read-only.

btl/openib: queue pending fragments once only when running out of credit #369

Merged

Conversation

ggouaillardet
Copy link
Contributor

Fixes open-mpi/ompi#640

(cherry picked from commit open-mpi/ompi@9f171de)

@ggouaillardet
Copy link
Contributor Author

:bot:assign: @hjelmn

@ggouaillardet
Copy link
Contributor Author

:bot🏷️bug
:bot:milestone:v2.0.0

@ompiteam-bot
Copy link

OMPIBot error: Milestone "v2.0.0" does not exist.

@ggouaillardet
Copy link
Contributor Author

:bot🏷️bug
:bot:milestone:v2.0.0

@ompiteam-bot ompiteam-bot added this to the v2.0.0 milestone Jul 6, 2015
@ompiteam-bot
Copy link

OMPIBot error: Label "bug" is already set on issue 369.

@mike-dubman
Copy link
Member

bot:retest

@hppritcha
Copy link
Member

Jeff/Howard decided today (7/6/15) to let this cook in master for a week. If no problems seen, will merge at that time.

@hppritcha
Copy link
Member

time to merge

hppritcha added a commit that referenced this pull request Jul 14, 2015
…redit_fix

btl/openib: queue pending fragments once only when running out of credit
@hppritcha hppritcha merged commit 41dc998 into open-mpi:v2.x Jul 14, 2015
nrgraham23 pushed a commit to nrgraham23/ompi-release that referenced this pull request Aug 20, 2015
This pull request adds two Buffer references to the
Request class. Whenever a request object is created
that has associated buffers, the buffers should be added
to these references so the java garbage collector does
not dispose of the buffers prematurely.

This is a more robust expansion on the idea first proposed by
@ggouaillardet, and further discussed in open-mpi#369.

Fixes open-mpi#369

Manual cherry-pick from d363b5d

Signed-off-by: Nathaniel Graham <[email protected]>
@nrgraham23 nrgraham23 mentioned this pull request Aug 20, 2015
nrgraham23 pushed a commit to nrgraham23/ompi-release that referenced this pull request Aug 24, 2015
This pull request adds an arraylist of type Buffer to
the Request class.  Whenever a request object is created
that has associated buffers, the buffers should be added
to this array list so the java garbage collector does
not dispose of the buffers prematurely.

This is a more robust expansion on the idea first proposed by
@ggouaillardet

Fixes open-mpi#369

Signed-off-by: Nathaniel Graham <[email protected]>
tkordenbrock pushed a commit to tkordenbrock/ompi-release that referenced this pull request Aug 28, 2015
This pull request adds two Buffer references to the
Request class. Whenever a request object is created
that has associated buffers, the buffers should be added
to these references so the java garbage collector does
not dispose of the buffers prematurely.

This is a more robust expansion on the idea first proposed by
@ggouaillardet, and further discussed in open-mpi#369.

Fixes open-mpi#369

Manual cherry-pick from d363b5d

Signed-off-by: Nathaniel Graham <[email protected]>
tkordenbrock pushed a commit to tkordenbrock/ompi-release that referenced this pull request Aug 28, 2015
This pull request adds an arraylist of type Buffer to
the Request class.  Whenever a request object is created
that has associated buffers, the buffers should be added
to this array list so the java garbage collector does
not dispose of the buffers prematurely.

This is a more robust expansion on the idea first proposed by
@ggouaillardet

Fixes open-mpi#369

Signed-off-by: Nathaniel Graham <[email protected]>
alinask pushed a commit to alinask/ompi-release that referenced this pull request Dec 10, 2015
This pull request adds an arraylist of type Buffer to
the Request class.  Whenever a request object is created
that has associated buffers, the buffers should be added
to this array list so the java garbage collector does
not dispose of the buffers prematurely.

This is a more robust expansion on the idea first proposed by
@ggouaillardet

Fixes open-mpi#369

Signed-off-by: Nathaniel Graham <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants