Skip to content

Conversation

@Maoni0
Copy link
Member

@Maoni0 Maoni0 commented Apr 23, 2021

Port #51104 to Release/5.0

when we have an object that straddles the 2mb boundary for card mark stealing, we could get into a very specific situation where we process the same card twice right after the 2mb boundary and mistakenly erase it. we fix this by avoid processing the same card twice. #51104 has the detailed description of how this can happen.

Customer Impact

this causes an AV because GC would reclaim objects it shouldn't

Testing

tested locally and the customer who hit this has already tested a private 5.0 build and verified it fixed their problem.

Regression

no. this existed when we checked in the card mark stealing work in 5.0 (which means this only exists in 5.0).

@ghost ghost added the area-GC-coreclr label Apr 23, 2021
@ghost
Copy link

ghost commented Apr 23, 2021

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

Issue Details

Port #51104 to Release/5.0

when we have an object that straddles the 2mb boundary for card mark stealing, we could get into a very specific situation where we process the same card twice right after the 2mb boundary and mistakenly erase it. we fix this by avoid processing the same card twice. #51104 has the detailed description of how this can happen.

Customer Impact

this causes an AV because GC would reclaim objects it shouldn't

Testing

tested locally and the customer who hit this has already tested a private 5.0 build and verified it fixed their problem.

Regression

no. this existed when we checked in the card mark stealing work in 5.0 (which means this only exists in 5.0).

Author: Maoni0
Assignees: -
Labels:

area-GC-coreclr

Milestone: -

@jeffschwMSFT jeffschwMSFT added the Servicing-consider Issue for next servicing release review label Apr 23, 2021
@jeffschwMSFT jeffschwMSFT added this to the 5.0.x milestone Apr 23, 2021
Copy link
Member

@jeffschwMSFT jeffschwMSFT left a comment

Choose a reason for hiding this comment

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

Approved. Once you get a CR, I can take it for consideration in 5.0.x.

@Maoni0 Maoni0 requested a review from PeterSolMS April 23, 2021 05:16
@leecow leecow added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Apr 27, 2021
@leecow leecow modified the milestones: 5.0.x, 5.0.6 Apr 27, 2021
@Anipik Anipik merged commit 5b85ca0 into dotnet:release/5.0 May 5, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Jun 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-GC-coreclr Servicing-approved Approved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants