-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Flaky crashes with "error: pthread error: 22 (Invalid argument)" #34888
Comments
Here's the state of threads
|
It appears to me that the idle notify mechanism is racing with isolate shutdown, namely:
When @rmacnak-google Since this is related to the idle notifcation implementation, could you take a look at this? Instead of making a custom |
There was an another crash with this on the bots:
|
The issue here seems to be the monitor being allocated dynamically here
and being deleted here
This pattern has always resulted in flaky crashes in the VM due the order in which things run during cleanup and we made a decision to always create static versions of all monitors in the VM. |
This is also happening on Windows, just with slightly different symptoms. Now that we have windows crashdumps available, we can confirm that flaky crashes we have observed before are exactly the same underlying issue:
@a-siva I think you are looking at the wrong monitor - the one being deleted is There is a subtle lifetime issue -
|
…class table until the next safepoint. Compare the reload rollback path. Bug: #34888 Change-Id: I5b9e7cd8392f9c9d61df31481956a9fa67435b33 Reviewed-on: https://dart-review.googlesource.com/c/81940 Reviewed-by: Zach Anderson <zra@google.com> Commit-Queue: Ryan Macnak <rmacnak@google.com>
I have turned off IdleNotifier for now as the fix for this race is a bit involved, I am reworking the IdleNotifier code. |
The CL https://dart-review.googlesource.com/c/sdk/+/81381 should address this issue. |
…class table until the next safepoint. Compare the reload rollback path. Bug: #34888 Change-Id: I5b9e7cd8392f9c9d61df31481956a9fa67435b33 Reviewed-on: https://dart-review.googlesource.com/c/81940 Reviewed-by: Zach Anderson <zra@google.com> Commit-Queue: Ryan Macnak <rmacnak@google.com>
The idle notifier code has been rewritten and the windows monitor code implementation has been modified. We haven't seen these crashes after these fixes. |
From this log:
The text was updated successfully, but these errors were encountered: