Skip to content

add margotree collective benchmark#466

Closed
adammoody wants to merge 1 commit intollnl:devfrom
adammoody:margotreebenchmark
Closed

add margotree collective benchmark#466
adammoody wants to merge 1 commit intollnl:devfrom
adammoody:margotreebenchmark

Conversation

@adammoody
Copy link
Copy Markdown
Collaborator

Description

Motivation and Context

How Has This Been Tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Performance enhancement (non-breaking change which improves efficiency)
  • Code cleanup (non-breaking change which makes code smaller or more readable)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation (a change to man pages or other documentation)

Checklist:

  • My code follows the UnifyFS code style requirements.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • All commit messages are properly formatted.

state_filesize_free(&glb_st);

double end = MPI_Wtime();
printf("procs=%d degree=%d reply=%d secs=%f\n", glb_ranks, degree, reply, (end - start));
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sandrain , this first call is a single invocation of the tree.

Comment on lines +812 to +813
int times = 100;
for (int i = 0; i < times; i++) {
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sandrain , after that I then executed 100 bcasts in a row here and reported the average cost on an iteration. In this case, all of them were started from root 0, and the next bcast didn't start until the previous one completed.

@adammoody adammoody closed this Nov 7, 2022
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 this pull request may close these issues.

1 participant