Skip to content

osc/ucx: optimizations  #11184

Open
Open
@janjust

Description

@janjust
  1. Store the rank with the base and tail to avoid false positives and allow us to flush to endpoint instead of the worker

  2. Instead of resetting the array after a flush, it might be more efficient to reset that particular entry in the completion callback. That probably wouldn't even need the mutex in the callback since we're only resetting two integer values to 0 (the window of opportunity for a race is small and even if there is one in the worst we don't recognize that entry as being empty). And there is a good chance that it would reduce the need for flushes since entries are continually reset once operations complete.

@devreal @MamziB for viz, relevant PR #11178

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions