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

Fixes for endpoint flush #3054

Merged
merged 3 commits into from
Nov 26, 2018
Merged

Fixes for endpoint flush #3054

merged 3 commits into from
Nov 26, 2018

Commits on Nov 24, 2018

  1. UCT/MM: Fix endpoint flush - don't update cached tail

    We must not update cached tail in uct_mm_ep_flush() if there are any
    pending elements. We may get new send resources but not use them, so
    flush could return UCS_OK while there pending requests.
    
    Fixes openucx#3052
    yosefe committed Nov 24, 2018
    Configuration menu
    Copy the full SHA
    1ac5745 View commit details
    Browse the repository at this point in the history
  2. UCP/EP: Fixes for endpoint flush for software rma/amo operations

    - Ignore remote completions in case of forced flush (CLOSE_MODE_CANCEL)
    
    - The UCP_EP_FLAG_FLUSH_STATE_VALID flag can't be used because it's not
      valid in release mode. Instead, use the UCP_EP_FLAG_DEST_EP flag as an
      indirect indication that we may have some operations which wait for
      software rma/amo completion.
    
    - Fix missing initializion of flush state in case of client/server
      connection establishment with p2p lanes and *without* pre-request.
    yosefe committed Nov 24, 2018
    Configuration menu
    Copy the full SHA
    3592e83 View commit details
    Browse the repository at this point in the history

Commits on Nov 25, 2018

  1. UCP/FLUSH: Fix typo in comment

    yosefe committed Nov 25, 2018
    Configuration menu
    Copy the full SHA
    7213814 View commit details
    Browse the repository at this point in the history