Skip to content

Conversation

@gnailzenh
Copy link
Contributor

A degraded EC read will allocate and register an extra buffer to recover data, which may cause ENOMEM in some cases.

this workaround does not prevent dynamic buffer allocation and registration, it does provide relatively precise control over the resources consumed by degraded EC reads.

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

A degraded EC read will allocate and register an extra buffer
to recover data, which may cause ENOMEM in some cases.

this workaround does not prevent dynamic buffer allocation and
registration, it does provide relatively precise control over the
resources consumed by degraded EC reads.

Signed-off-by: Liang Zhen <gnailzenh@gmail.com>
@gnailzenh gnailzenh requested review from a team as code owners January 24, 2026 03:18
@github-actions
Copy link

Errors are Unable to load ticket data
https://daosio.atlassian.net/browse/DAOS-18487

@gnailzenh
Copy link
Contributor Author

This is another patch to decrease the chance of running into ENOMEM, because the aurora branch still uses the old resource management which only has 256MB DMA buffer for rebuild, so I decrease the amplification factor to 6, which is 8 for master & 2.6

@daosbuild3
Copy link
Collaborator

* registration, it does provide relatively precise control over the
* resources consumed by degraded EC reads.
*/
data_size *= MIN(6, obj_ec_data_tgt_nr(&mrone->mo_oca));
Copy link
Contributor

Choose a reason for hiding this comment

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

please consider to define a new variable to avoid affect migrate_dkey(), as commented on master PR.

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.

6 participants