Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-13502

Provide ability to fine-tune response tracking

Details

    • Improvement
    • Resolution: Fixed
    • Major
    • Lustre 2.14.0
    • None
    • None
    • 9223372036854775807

    Description

      I think we need to provide a way to opt out of response trackers for DVS, and I was thinking it would be useful, at least for testing purposes, to have a parameter to control the behavior of response tracking. I was thinking:
      1. Create a parameter that allows them to be enabled:
      1.1 For all messages (default)
      1.2 Only PUTs
      1.3 Only GETs
      1.4 Internal LNet traffic only (discovery messages and pings)

      i.e. we can't turn it off for discovery messages and pings but we could for all other messages.
      To support 1.4 we could have a new MD option that says "attach an rspt regardless of the param".
      Then we could have another MD option that says "do not attach an rspt regardless of the param". We could use that for DVS.

      Attachments

        Activity

          [LU-13502] Provide ability to fine-tune response tracking
          pjones Peter Jones added a comment -

          Take two..

          pjones Peter Jones added a comment - Take two..

          Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38514/
          Subject: LU-13502 lnet: Add response tracking param to lnetctl
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: b67a8e76a50ec744af80db8bc5c4b73cb1a9ccf9

          gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38514/ Subject: LU-13502 lnet: Add response tracking param to lnetctl Project: fs/lustre-release Branch: master Current Patch Set: Commit: b67a8e76a50ec744af80db8bc5c4b73cb1a9ccf9
          spitzcor Cory Spitz added a comment -

          Re-opening since https://review.whamcloud.com/#/c/38514/ is yet to land. When that is merged then this issue can be re-resolved.

          spitzcor Cory Spitz added a comment - Re-opening since https://review.whamcloud.com/#/c/38514/ is yet to land. When that is merged then this issue can be re-resolved.

          Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38452/
          Subject: LU-13502 lnet: Conditionally attach rspt in LNetPut & LNetGet
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: 0722e7601f0ba5f33e9de6e353dffd11b961121c

          gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38452/ Subject: LU-13502 lnet: Conditionally attach rspt in LNetPut & LNetGet Project: fs/lustre-release Branch: master Current Patch Set: Commit: 0722e7601f0ba5f33e9de6e353dffd11b961121c
          pjones Peter Jones added a comment -

          Landed for 2.14

          pjones Peter Jones added a comment - Landed for 2.14

          Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38451/
          Subject: LU-13502 lnet: Ensure LNet pings and pushes are always tracked
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: 9ed12d2e8f3f2189d7794e55e7c0e8c0178ffde7

          gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38451/ Subject: LU-13502 lnet: Ensure LNet pings and pushes are always tracked Project: fs/lustre-release Branch: master Current Patch Set: Commit: 9ed12d2e8f3f2189d7794e55e7c0e8c0178ffde7

          Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38449/
          Subject: LU-13502 lnet: Add param to control response tracking
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: 5b3747294a4bf4a12ad494d2c72b72512794ba90

          gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38449/ Subject: LU-13502 lnet: Add param to control response tracking Project: fs/lustre-release Branch: master Current Patch Set: Commit: 5b3747294a4bf4a12ad494d2c72b72512794ba90

          Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38450/
          Subject: LU-13502 lnet: Add MD options for response tracking
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: 448834b8e575ff97b686691347de2e781c1d4b4b

          gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38450/ Subject: LU-13502 lnet: Add MD options for response tracking Project: fs/lustre-release Branch: master Current Patch Set: Commit: 448834b8e575ff97b686691347de2e781c1d4b4b

          Chris Horn (chris.horn@hpe.com) uploaded a new patch: https://review.whamcloud.com/38514
          Subject: LU-13502 lnet: Add response tracking param to lnetctl
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 7607346a43698ad94ff24d5ce988f1fdd4d4a48c

          gerrit Gerrit Updater added a comment - Chris Horn (chris.horn@hpe.com) uploaded a new patch: https://review.whamcloud.com/38514 Subject: LU-13502 lnet: Add response tracking param to lnetctl Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 7607346a43698ad94ff24d5ce988f1fdd4d4a48c

          Chris Horn (chris.horn@hpe.com) uploaded a new patch: https://review.whamcloud.com/38452
          Subject: LU-13502 lnet: Conditional attach rspt in LNetPut and LNetGet
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: a5a3bd646d4ce5f4cb74c78b32e49f89e812eddf

          gerrit Gerrit Updater added a comment - Chris Horn (chris.horn@hpe.com) uploaded a new patch: https://review.whamcloud.com/38452 Subject: LU-13502 lnet: Conditional attach rspt in LNetPut and LNetGet Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: a5a3bd646d4ce5f4cb74c78b32e49f89e812eddf

          People

            hornc Chris Horn
            hornc Chris Horn
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: