ompi icon indicating copy to clipboard operation
ompi copied to clipboard

Correctly retain the datatypes for topological communicators.

Open bosilca opened this issue 3 years ago • 5 comments

I do not recall which issue this is related to, and I was not able to find it on github either, but this patch addresed the number of datatypes to release/retain for collective communications on topological communicators.

Signed-off-by: George Bosilca [email protected]

bosilca avatar Jun 24 '22 05:06 bosilca

@bosilca Is this just for main / v5.0.x? Or will we also need it for v4.0.x / v4.1.x?

jsquyres avatar Jun 24 '22 11:06 jsquyres

@ggouaillardet Could you review?

jsquyres avatar Jul 17 '22 11:07 jsquyres

I do not recall which issue this is related to, and I was not able to find it on github either, but this patch addresed the number of datatypes to release/retain for collective communications on topological communicators.

Signed-off-by: George Bosilca [email protected]

I think I helped report this issue by email. I was about to ask when this fix might be available. Happy to see this fix is being added.

marekdam avatar Aug 03 '22 18:08 marekdam

this is indeed the email that triggered my interest in the bug. thanks @marekdam for pointing it out.

@jsquyres we need this in main/5.0, and 4.1.

bosilca avatar Aug 04 '22 04:08 bosilca

@markalle and/or @devreal can you please review?

awlauria avatar Aug 19 '22 15:08 awlauria

Please bring back to v5.0.x and v4.1.x.

awlauria avatar Oct 12 '22 13:10 awlauria