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] [NativeAOT] correctly initalize CONTEXT before failing fast #81837

Merged
merged 7 commits into from
Feb 9, 2023

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Feb 8, 2023

Backport of #81010 to release/7.0

/cc @jkotas @AustinWise

Customer Impact

Crash dumps of native AOT processes are undebuggable. The debugger does not show the stacktrace of the crash.

The problem is caused by uninitialized fields of CONTEXT structure, and thus it is non-deterministic. It seems to be more likely to happen on Windows 11 22H2.

Testing

Verified that the crash dump is diagnosable using deterministic repro (thank you @AustinWise for creating one!)

Risk

Low. The fix affects code path is only executed right before the process is about to crash.

@ghost
Copy link

ghost commented Feb 8, 2023

Tagging subscribers to this area: @agocke, @MichalStrehovsky, @jkotas
See info in area-owners.md if you want to be subscribed.

Issue Details

Backport of #81010 to release/7.0

/cc @jkotas @AustinWise

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-NativeAOT-coreclr

Milestone: -

@jkotas jkotas requested a review from janvorli February 8, 2023 18:44
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. we will take for consideration in 7.0.x

@carlossanlop
Copy link
Member

@janvorli @jkotas Don't forget to add the servicing-consider label and send the email to Tactics requesting approval. The branch closes on Monday for merging changes that should go into the March Servicing Release.

CI results are known and unrelated: #81544 #81391. We merged a fix for them a few hours after this PR was created: #81712.

@jkotas jkotas added the Servicing-consider Issue for next servicing release review label Feb 9, 2023
@jkotas
Copy link
Member

jkotas commented Feb 9, 2023

I have sent email yesterday and it has been approved already.

@carlossanlop carlossanlop added this to the 6.0.15 milestone Feb 9, 2023
@carlossanlop carlossanlop added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Feb 9, 2023
@carlossanlop
Copy link
Member

carlossanlop commented Feb 9, 2023

@jkotas Found it. Weird, Outlook wasn't finding the email by title, but I was able to find it by sender.

Approved by Tactics for 7.0.4.
Signed-off by area owners.
CI failures unrelated.
No OOB changes required for this PR.
Ready to merge. :shipit:

@carlossanlop carlossanlop modified the milestones: 6.0.15, 7.0.4 Feb 9, 2023
@carlossanlop carlossanlop merged commit b4f6d21 into release/7.0 Feb 9, 2023
@carlossanlop carlossanlop deleted the backport/pr-81010-to-release/7.0 branch February 9, 2023 18:38
@ghost ghost locked as resolved and limited conversation to collaborators Mar 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-NativeAOT-coreclr Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants