Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SMI Integration #83

Open
TizianoDeMatteis opened this issue Jan 9, 2020 · 0 comments
Open

SMI Integration #83

TizianoDeMatteis opened this issue Jan 9, 2020 · 0 comments
Assignees
Labels

Comments

@TizianoDeMatteis
Copy link
Contributor

TizianoDeMatteis commented Jan 9, 2020

Integration of a minimal set of SMI functionalities (p2p communications for the moment being).

Possible solution

Introduce the concept of remote streams

Technical details

If SMI should be used or not, is determined at codegeneration by looking at if remote streams are used

CMake Integration

The use of SMI is detected in the code-generation phase. In this case, proper Make targets are created for favoring compilation/emulation of SMI based programs.

This requires to define a topology file (that contains the mapping program <-> rank) for the sake of emulation. In this first implementation, this is not so meaningful but will be required for full SMI integration

Codegen object

Defined a target_name field, which can be set when returning a codegenobject and we want to have the field initialized

Generated code

For the sake of enabling an easy emulation toolchain, the host generated code will assume the presence of the following attributes:

  • smi_rank: current rank (int)
  • smi_num_ranks: total number of ranks (int)
  • smi_device: device used (int, useful for running on Noctua)

These must be defined by specializing the SDFG.

TODO: this must be cleaned

@TizianoDeMatteis TizianoDeMatteis self-assigned this Jan 9, 2020
@TizianoDeMatteis TizianoDeMatteis linked a pull request Mar 9, 2020 that will close this issue
@tbennun tbennun added the core label Sep 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants