-
Notifications
You must be signed in to change notification settings - Fork 11
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
when to use flagcxBarrier? #29
Comments
Yep, you may use devHandle->streamSynchronize(stream) to achieve synchronization. Here flagcxBarrier is added to testify barrier op in all test cases. Since flagcxBarrier introduces an extra allreduce operation, it will cause performance degradation for sure. Thank you for your feedback. We will fix this issue as soon as possible. |
Hi, may I submit a PR to fix this issue? |
Hi, this issue has already been fixed in the latest version. |
ok, thanks |
hi,
I have seen some code in this file
https://github.com/FlagOpen/FlagCX/blob/main/test/perf/test_sendrecv.cpp#70
Is the flagcxBarrier here works the same as devHandle->streamSynchronize(steam) ?
why here use flagcxBarrier, It add more time when timing the performance of send/recv.
and what does flagcxBarrier used for ?
The text was updated successfully, but these errors were encountered: