Skip to content

DAOS-18882 vos: avoid heap_curr_allocated underflow#18103

Open
grom72 wants to merge 3 commits intomasterfrom
grom72/DAOS-18882
Open

DAOS-18882 vos: avoid heap_curr_allocated underflow#18103
grom72 wants to merge 3 commits intomasterfrom
grom72/DAOS-18882

Conversation

@grom72
Copy link
Copy Markdown
Contributor

@grom72 grom72 commented Apr 24, 2026

Update PMDK to incorporate the following fixes:

Steps for the author:

  • Commit message follows the guidelines.
  • Appropriate Features or Test-tag pragmas were used.
  • Appropriate Functional Test Stages were run.
  • At least two positive code reviews including at least one code owner from each category referenced in the PR.
  • Testing is complete. If necessary, forced-landing label added and a reason added in a comment.

After all prior steps are complete:

  • Gatekeeper requested (daos-gatekeeper added as a reviewer).

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 24, 2026

Ticket title is '0 size SCM on pool with no containers'
Status is 'In Review'
https://daosio.atlassian.net/browse/DAOS-18882

@grom72 grom72 changed the title DAOS-18882 pmdk: avoid heap_curr_allocated underflow DAOS-18882 vos: avoid heap_curr_allocated underflow Apr 24, 2026
@grom72 grom72 force-pushed the grom72/DAOS-18882 branch from 735b1f2 to b6c997b Compare April 24, 2026 10:08
Update PMDK to incorporate the following fixes:
- fix "The pool was not closed" message (no ADR failure) daos-stack/pmdk#36
- recalculate curr_allocated on underflow daos-stack/pmdk#37

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Priority: 2

Allow-unstable-test: true

Focus validation on PMem version

Skip-func-hw-test-medium: false
Skip-func-hw-test-medium-md-on-ssd: true
Skip-func-hw-test-medium-vmd: false
Skip-func-hw-test-medium-verbs-provider: false
Skip-func-hw-test-medium-verbs-provider-md-on-ssd: true
Skip-func-hw-test-large: false
Skip-func-hw-test-large-md-on-ssd: true
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
@grom72 grom72 force-pushed the grom72/DAOS-18882 branch from b6c997b to 91a8209 Compare April 24, 2026 10:15
grom72 added a commit that referenced this pull request Apr 24, 2026
Update PMDK to incorporate the following fixes:
- fix "The pool was not closed" message (no ADR failure) daos-stack/pmdk#36
- recalculate curr_allocated on underflow daos-stack/pmdk#37

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Priority: 2

Allow-unstable-test: true

Skip-func-hw-test-medium: false
Skip-func-hw-test-large: false
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
@osalyk osalyk marked this pull request as ready for review April 24, 2026 14:18
@osalyk osalyk requested a review from a team as a code owner April 24, 2026 14:18
@daosbuild3
Copy link
Copy Markdown
Collaborator

Comment thread utils/build.config Outdated
[patch_versions]
spdk=0001_3428322b812fe31cc3e1d0308a7f5bd4b06b9886.diff,0002_spdk_rwf_nowait.patch,0003_external_isal.patch
mercury=0001_dep_versions.patch,0002_ofi_counters.patch,0003_ofi_auth_key.patch
pmdk=https://github.com/daos-stack/pmdk/commit/69925cf455ef672c4cbdbdb13bef7ae581e67045.diff
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I made the change, but did not test locally.

Signed-off-by: Ryon Jensen <ryon.jensen@hpe.com>
@mchaarawi mchaarawi requested a review from soumagne April 24, 2026 19:41
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