Skip to content

DAOS-8927 vos: Lock memory for VOS pools#7153

Merged
johannlombardi merged 4 commits intomasterfrom
jvolivie/unpin
Nov 3, 2021
Merged

DAOS-8927 vos: Lock memory for VOS pools#7153
johannlombardi merged 4 commits intomasterfrom
jvolivie/unpin

Conversation

@jolivier23
Copy link
Contributor

Change CART envirable to turn on memory pinning globally
with it off by default. Instead, lock each vos pool
individually.

Signed-off-by: Jeff Olivier jeffrey.v.olivier@intel.com

Change CART envirable to turn on memory pinning globally
with it off by default.   Instead, lock each vos pool
individually.

Signed-off-by: Jeff Olivier <jeffrey.v.olivier@intel.com>
Copy link
Collaborator

@daosbuild1 daosbuild1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. No errors found by checkpatch.

@daosbuild1
Copy link
Collaborator

Signed-off-by: Jeff Olivier <jeffrey.v.olivier@intel.com>
Copy link
Collaborator

@daosbuild1 daosbuild1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. No errors found by checkpatch.

@daosbuild1
Copy link
Collaborator

Copy link
Collaborator

@daosbuild1 daosbuild1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. No errors found by checkpatch.

@daosbuild1
Copy link
Collaborator

Allow-unstable-test: true

Signed-off-by: Jeff Olivier <jeffrey.v.olivier@intel.com>
Copy link
Collaborator

@daosbuild1 daosbuild1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. No errors found by checkpatch.

@jolivier23
Copy link
Contributor Author

The NLT warning is one that would need a force landing. This patch adds a warning when the system isn't configured for memory pinning (e.g. places a limit on locked pages).

@jolivier23 jolivier23 requested a review from a team November 2, 2021 13:59
@jolivier23
Copy link
Contributor Author

Failure is DAOS-8901

@johannlombardi johannlombardi merged commit 3eab023 into master Nov 3, 2021
@johannlombardi johannlombardi deleted the jvolivie/unpin branch November 3, 2021 11:21
@johannlombardi
Copy link
Contributor

Please backport to release/2.0

jolivier23 added a commit that referenced this pull request Nov 3, 2021
Change CART envirable to turn on memory pinning globally
with it off by default.   Instead, lock each vos pool
individually.

Signed-off-by: Jeff Olivier <jeffrey.v.olivier@intel.com>
johannlombardi pushed a commit that referenced this pull request Nov 8, 2021
Change CART envirable to turn on memory pinning globally
with it off by default.   Instead, lock each vos pool
individually.

mlock will fail if trying to lock memory to a page boundary and
PMDK doesn't even map the whole space if there is leftover bits
on a page.   Just round up the size to the next page boundary.

Signed-off-by: Jeff Olivier <jeffrey.v.olivier@intel.com>
janekmi added a commit that referenced this pull request Mar 5, 2026
Ref: #7153

Signed-off-by: Jan Michalski <jan-marian.michalski@hpe.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants