Skip to content

GetTotalPauseDuration implementation #68835

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.

Sign up for GitHub

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

Jump to bottom
Merged
cshung merged 7 commits into from
May 20, 2022
Merged

GetTotalPauseDuration implementation #68835

cshung merged 7 commits into from
May 20, 2022

Conversation

@cshung
Copy link
Contributor

@cshung cshung commented May 3, 2022 •
edited
Loading

Fixes #66036

  • Mono implementation - we will probably merge before that is done.
  • Organize the EE -> GC interfaces such that it is version resilience.
  • Testing
  • API Documentation

@ghost
Copy link

ghost commented May 3, 2022

Note regarding the new-api-needs-documentation label:

This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, to please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change.

@ghost
Copy link

ghost commented May 3, 2022

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@ghost ghost assigned cshung May 3, 2022
@cshung cshung force-pushed the public/public-pause-time branch from b98898f to a08fe86 Compare May 3, 2022 20:45
@ghost
Copy link

ghost commented May 3, 2022

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

Issue Details

Fixes #66036

  • Mono implementation - we will probably merge before that is done.
  • Organize the EE -> GC interfaces such that it is version resilience.
  • Testing
  • API Documentation
Author: cshung
Assignees: cshung
Labels:

area-GC-coreclr, new-api-needs-documentation
Milestone: -

@marek-safar marek-safar requested a review from BrzVlad May 4, 2022 08:40
@cshung cshung force-pushed the public/public-pause-time branch from 73222f9 to 2783560 Compare May 4, 2022 16:41
Maoni0
Maoni0 reviewed May 5, 2022
View reviewed changes
Maoni0
Maoni0 reviewed May 5, 2022
View reviewed changes
Maoni0
Maoni0 reviewed May 5, 2022
View reviewed changes
@cshung cshung force-pushed the public/public-pause-time branch from 2783560 to 7233dff Compare May 9, 2022 19:36
@cshung cshung changed the title [WIP] GetTotalPauseDuration implementation GetTotalPauseDuration implementation May 12, 2022
@Maoni0 Maoni0 mentioned this pull request May 13, 2022
Closed
Maoni0
Maoni0 reviewed May 14, 2022
View reviewed changes
@cshung cshung merged commit f1efc5c into dotnet:main May 20, 2022
@cshung cshung deleted the public/public-pause-time branch May 20, 2022 21:09
@ghost ghost locked as resolved and limited conversation to collaborators Jun 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@MichalStrehovsky MichalStrehovsky MichalStrehovsky left review comments

@Maoni0 Maoni0 Maoni0 approved these changes

@marek-safar marek-safar Awaiting requested review from marek-safar

@BrzVlad BrzVlad Awaiting requested review from BrzVlad

Assignees

@cshung cshung

Labels

area-GC-coreclr new-api-needs-documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[API Proposal]: Exposing the total paused duration in GC.

3 participants

@cshung @Maoni0 @MichalStrehovsky