Skip to content

DAOS-19330 vos: Find highest minor_epc for rebuild overwrite#18682

Open
NiuYawei wants to merge 1 commit into
release/2.8from
niu/release-28/DAOS-19330
Open

DAOS-19330 vos: Find highest minor_epc for rebuild overwrite#18682
NiuYawei wants to merge 1 commit into
release/2.8from
niu/release-28/DAOS-19330

Conversation

@NiuYawei

Copy link
Copy Markdown
Contributor

For rebuild overwrites, the code must identify the overlapping extent with the highest minor epoch to ensure the minor epoch is bumped correctly. However, a bug in evt_ent_array_fill() causes the scan to terminate prematurely. This can result in two overlapping extents incorrectly sharing the exact same epoch and minor epoch (EVT_REBUILD_MINOR_MIN + 1).

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).

For rebuild overwrites, the code must identify the overlapping extent with the
highest minor epoch to ensure the minor epoch is bumped correctly. However, a
bug in evt_ent_array_fill() causes the scan to terminate prematurely. This can
result in two overlapping extents incorrectly sharing the exact same epoch and
minor epoch (EVT_REBUILD_MINOR_MIN + 1).

Signed-off-by: Niu Yawei <yawei.niu@hpe.com>
@NiuYawei
NiuYawei requested review from a team as code owners July 17, 2026 04:23
@github-actions

Copy link
Copy Markdown

Ticket title is 'DAOS 2.8.0RC2: Engines crash: vos EMRG src/vos/evtree.c:943 evt_find_visible() Assertion 'this_ent->en_epoch != next_ent->en_epoch || this_ent->en_minor_epc != next_ent->en_minor_epc''
Status is 'In Progress'
Labels: 'test_2.8.0rc'
https://daosio.atlassian.net/browse/DAOS-19330

@janekmi janekmi left a comment

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.

Do we have any test which can detect this issue?

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.

2 participants