[LU-13502] Provide ability to fine-tune response tracking

• Status: Resolved
• Project: Lustre
• Fix Version/s: Lustre 2.14.0
• Type: Improvement
• Priority: Major
• Reporter: Chris Horn
• Assignee: Chris Horn
• Resolution: Fixed

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.

Comments

[Gerrit Updater]

Chris Horn ([e-mail]) uploaded a new 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: 1
Commit: 2301c9ec77a0202ffcfd5558b888810ec9088bd4

[Gerrit Updater]

Chris Horn ([e-mail]) uploaded a new 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: 1
Commit: 4083b86524594147416bca3cecdd228cda674555

[Gerrit Updater]

Chris Horn ([e-mail]) uploaded a new 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: 1
Commit: 6fadfc0a91dab093d58bb3dd8f469590d327f7ec

[Gerrit Updater]

Chris Horn ([e-mail]) 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 Updater]

Chris Horn ([e-mail]) 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 Updater]

Oleg Drokin ([e-mail]) 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 Updater]

Oleg Drokin ([e-mail]) 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 Updater]

Oleg Drokin ([e-mail]) 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

[Peter Jones]

Landed for 2.14

[Gerrit Updater]

Oleg Drokin ([e-mail]) 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

[Cory Spitz]

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

[Gerrit Updater]

Oleg Drokin ([e-mail]) 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

[Peter Jones]

Take two..