Skip to content

DAOS-18919 rebuild: fix a bug of csum process in migrate_fetch_update…#18184

Open
liuxuezhao wants to merge 1 commit intorelease/2.8from
lxz/rb_csum_fix_28
Open

DAOS-18919 rebuild: fix a bug of csum process in migrate_fetch_update…#18184
liuxuezhao wants to merge 1 commit intorelease/2.8from
lxz/rb_csum_fix_28

Conversation

@liuxuezhao
Copy link
Copy Markdown
Contributor

…_inline

in migrate_fetch_update_inline() it possibly with multiple IODs inline, if there is zero iod_size in the middle, the original code possibly cause the packed csums cannot be correctly unpacked, for replica object. This patch fixes it by refine related code.
Renames a few fields to make it easier to grep, and a few other small refines.

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

…_inline

in migrate_fetch_update_inline() it possibly with multiple IODs inline,
if there is zero iod_size in the middle, the original code possibly
cause the packed csums cannot be correctly unpacked, for replica object.
This patch fixes it by refine related code.
Renames a few fields to make it easier to grep, and a few other small
refines.

Signed-off-by: Xuezhao Liu <xuezhao.liu@hpe.com>
@liuxuezhao liuxuezhao requested review from a team as code owners May 6, 2026 09:17
@liuxuezhao liuxuezhao removed request for a team May 6, 2026 09:17
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

Ticket title is 'bug in rebuild migrate_fetch_update_inline() csum processing with multiple IODs'
Status is 'Open'
Labels: 'scrubbed_2.8'
https://daosio.atlassian.net/browse/DAOS-18919

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.

1 participant