-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[release/8.0-staging] Get ThreadContext for AVX registers #104818
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/8.0-staging] Get ThreadContext for AVX registers #104818
Conversation
|
Friendly reminder that Monday July 15th is Code Complete day, that's the deadline to get this included in the August Release. |
jeffschwMSFT
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm. we will take for consideration in 8.0.x
|
Just realized there is an extra logging change in the backport. Is that expected? |
Ah, no, that came from the original branch, I've missed it - it should not be there, let me remove it. |
aeaa2a1 to
1928ba4
Compare
@carlossanlop yes, it is. |
|
when is this expected to be released? is there a way I can download the staging/beta version of the runtime in the mean time? |
|
Hi @danny-zegel-zocdoc, this fix should be in .NET 9 preview 6. You can download that here: https://dotnet.microsoft.com/en-us/download/dotnet/9.0 |
i'm working in .NET 8, would like a .NET 8 runtime |
|
The next .NET 8 update will be later this month. |
Backport of #104514 to release/8.0-staging
/cc @hoyosjs @mikelle-rogers
Customer Impact
Customers running on macOS x64 experience random crashes if the jitted code uses upper halves of the ymm registers. These halves were not being copied when a hardware exception occurred to the context used to handle the exception and were zeroed out when continuing after an exception.
Regression
Testing
Local directed testing
Risk
Low, the change adds previously missing data to the context