Skip to content

Fix GCEvent::Wait with timeout on OSX - #75298

Merged
VSadov merged 1 commit into
dotnet:mainfrom
VSadov:gcev
Sep 9, 2022
Merged

Fix GCEvent::Wait with timeout on OSX#75298
VSadov merged 1 commit into
dotnet:mainfrom
VSadov:gcev

Conversation

@VSadov

@VSadovVSadov commented Sep 8, 2022

Copy link
Copy Markdown
Member

Fixes:#75005

We are using uninitialized endTime when Wait with a timeout waits on a conditional variable on OSX.
This can result in unpredictable behavior. On NativeAOT it results in BGC starting without a signal and asserting/crashing.

@ghostghost added the area-GC-coreclr label Sep 8, 2022
@ghostghost assigned VSadovSep 8, 2022
@ghost

ghost commented Sep 8, 2022

Copy link
Copy Markdown

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

Issue Details

Fixes:#75005

We are using uninitialized endTime when waiting on a conditional variable on OSX.
This can result in unpredictable behavior. On NativeAOT it results in BGC starting without a signal and asserting/crashing.

Author:VSadov
Assignees:-
Labels:

area-GC-coreclr

Milestone:-

@VSadov

Copy link
Copy Markdown
MemberAuthor

I think this will need to be ported to 7.0

@jkotasjkotas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@Maoni0

Copy link
Copy Markdown
Member

agreed. please backport. thanks @VSadov!

@janvorlijanvorli left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thank you!

@VSadov
VSadov merged commit e8b1491 into dotnet:mainSep 9, 2022
@VSadov
VSadov deleted the gcev branch September 9, 2022 00:42
@VSadov

Copy link
Copy Markdown
MemberAuthor

Thanks!!

@VSadov

Copy link
Copy Markdown
MemberAuthor

/backport to release/7.0

@github-actions

Copy link
Copy Markdown
Contributor

Started backporting to release/7.0: https://github.com/dotnet/runtime/actions/runs/3019221522

@ghostghost locked as resolved and limited conversation to collaborators Oct 9, 2022
Sign up for freeto 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.

4 participants

@VSadov@Maoni0@jkotas@janvorli