Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[release/7.0] [mono][sgen] Update memory load information after world is resumed #76534

Merged
merged 1 commit into from
Oct 4, 2022

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Oct 3, 2022

Backport of #76400 to release/7.0

/cc @akoeplinger @BrzVlad

Customer Impact

Following recent cgroup support (#74237), mono_determine_physical_ram_size is no longer safe to call during GC. This can lead to random hangs during GC when running on linux desktop platforms.

Testing

CI and local testing

Risk

Low. Small change to when memory threshold stat is being updated.

Following recent cgroup support, mono_determine_physical_ram_size is no longer safe to call during GC.
@ghost
Copy link

ghost commented Oct 3, 2022

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

Issue Details

Backport of #76400 to release/7.0

/cc @akoeplinger @BrzVlad

Customer Impact

Testing

Risk

IMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.

Author: github-actions[bot]
Assignees: -
Labels:

area-GC-mono

Milestone: -

@akoeplinger akoeplinger added this to the 7.0.0 milestone Oct 3, 2022
@akoeplinger
Copy link
Member

@steveisok @marek-safar please approve

@marek-safar
Copy link
Contributor

it needs to go via tactics approval

@steveisok steveisok added the Servicing-consider Issue for next servicing release review label Oct 3, 2022
@carlossanlop
Copy link
Member

@BrzVlad or @akoeplinger: please send an email to Tactics requesting approval.

@rbhanda rbhanda added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Oct 4, 2022
@akoeplinger
Copy link
Member

@carlossanlop this was approved by tactics, please merge :)

@carlossanlop
Copy link
Member

CI green, signed off, approved. Ready! :shipit:

@carlossanlop carlossanlop merged commit bba5f48 into release/7.0 Oct 4, 2022
@carlossanlop carlossanlop deleted the backport/pr-76400-to-release/7.0 branch October 4, 2022 19:55
@ghost ghost locked as resolved and limited conversation to collaborators Nov 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-GC-mono Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants