Skip to content

Conversation

@PeterSolMS
Copy link
Contributor

The case where we have high memory load and huge main memory can cause the return value to be larger than total_new_allocation.

As it turns out, this is contributing to issue #52592 - gen 0 budget shoots upruptly to 5.1 GB and drops back.

…ain memory - the return value should never be larger than total_new_allocation.
@ghost ghost added the area-GC-coreclr label Aug 26, 2021
@ghost
Copy link

ghost commented Aug 26, 2021

Tagging subscribers to this area: @dotnet/gc
See info in area-owners.md if you want to be subscribed.

Issue Details

The case where we have high memory load and huge main memory can cause the return value to be larger than total_new_allocation.

As it turns out, this is contributing to issue #52592 - gen 0 budget shoots upruptly to 5.1 GB and drops back.

Author: PeterSolMS
Assignees: -
Labels:

area-GC-coreclr

Milestone: -

{
return max (mem_one_percent, total_min_allocation);
size_t total_max_allocation = max (mem_one_percent, total_min_allocation);
return min (total_new_allocation, total_max_allocation);
Copy link
Member

Choose a reason for hiding this comment

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

have we figured out a way to validate such changes? Would the tuning simulator help in such cases?

Copy link
Member

Choose a reason for hiding this comment

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

yes, I think we should totally add budget calculation to the tuning simulator.

@PeterSolMS PeterSolMS merged commit 327b9c2 into dotnet:main Aug 27, 2021
@PeterSolMS
Copy link
Contributor Author

/backport to release/6.0

@github-actions
Copy link
Contributor

github-actions bot commented Sep 2, 2021

@ghost ghost locked as resolved and limited conversation to collaborators Oct 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants