|
Florent GERMAIN
|
|
an atos business |
Florent,
All you need is to issue a Pull Request and make sure the commits are signed off (and you understand the legal implications of doing that)
Cheers,
Gilles
To unsubscribe from this group and stop receiving emails from it, send an email to devel+un...@lists.open-mpi.org.
Hello,Would you be opening a PR for both PML and OSC components?
The reason I ask is if you don’t have the OSC component ready to include in a PR how will you be supporting one sided ops?As pt2pt osc has been dropped, we need both pml and osc, thus our intent is to open a PR for each of these two components at the same time.
Is your recommendation to actually open a single PR for both?
That would be better.
Would your PRs touch code outside of the new components?I don't think we have any modification outside of than pml+osc+common components.
Very good!
Also it would be nice to add some documentation about the BXI network in for example the docs/installing-open-mpi/configure-cli-options/networking.rst file in the docs folder. Also double check if there may need to be updates to fork-system-popen.rst.We'll check these files.
If the BXI transport API is using some portals API there are additional things to consider. Actually I thought BXI used to talk portals4. Is there a reason you can't use the existing portals4 mtl and osc components?BXI hardware is indeed based on portals4 API. The main reason why we use an external component is to plug it in other communication frameworks like for example NCCL/RCCL. Is is also easier to optimize the communication protocols regarding BXI hardware. We also can benefit shared memory under some conditions, which seems to be harder using the portals4 mtl.