-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Test failure Interop\\MarshalAPI\\IUnknown\\IUnknownTest\\IUnknownTest.cmd #85234
Comments
Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch Issue DetailsFailed in: runtime-coreclr gcstress-extra 20230423.1 Failed tests:
Error message:
Stack trace:
|
@markples, PTAL GC stress test failure. |
Failed again in: runtime-coreclr gcstress-extra 20230430.1 Failed tests:
Error message:
Stack trace:
|
This test continues to fail in GCStress where @dotnet/gc Issue with @dotnet/interop-contrib Failure is in an interop test. |
Let me also rewrite this test. It is very odd and I'm not ever sure it is worth having. I will check it out today, but I signed off on disabling it too. |
This looks like a GC issue. I have a very simple repro and I'm not sure how other COM tests are passing with GCStress at this point. I will update this issue as I uncover more details. |
The following is where the logic is failing. runtime/src/coreclr/gc/handletablescan.cpp Lines 962 to 970 in a442971
The object being verified is valid according to all tools I know of. The returned runtime/src/coreclr/gc/handletablescan.cpp Lines 927 to 931 in a442971
The runtime/src/coreclr/gc/handletablescan.cpp Line 173 in a442971
There is a slew of debug code in /cc @dotnet/gc |
Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch Issue DetailsFailed in: runtime-coreclr gcstress-extra 20230423.1 Failed tests:
Error message:
Stack trace:
|
Tagging subscribers to this area: @dotnet/gc Issue DetailsFailed in: runtime-coreclr gcstress-extra 20230423.1 Failed tests:
Error message:
Stack trace:
|
After we root cause this test, I'm going to need some help from @dotnet/gc, we can remove this test entirely. All the APIs here are being tested in libraries already. |
It should be related to frozen objects for which we report generation (age) as 0x7fffffff (int.MaxValue) so presumably it's just the assert needs to be improved 🤔 |
I don't think so. If you look above after the assert the runtime is told to record an EE exception (that is, |
/cc @cshung |
This issue reproduces reliably on Windows x64 under a Checked runtime.
It looks like the predicate in runtime/src/coreclr/gc/handletablescan.cpp Lines 902 to 932 in 143a9bf
|
Multiple tests will need to be reenabled when this is fixed. See #96364 |
Failed in: runtime-coreclr gcstress-extra 20230423.1
Failed tests:
Error message:
Stack trace:
The text was updated successfully, but these errors were encountered: