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

Only call current_thread_id once in try_select #804

Closed
wants to merge 1 commit into from

Conversation

glandium
Copy link

For some unclear reason as of writing, calling current_thread_id on each
iteration makes Firefox block before displaying a window on aarch64
Linux.

For some unclear reason as of writing, calling current_thread_id on each
iteration makes Firefox block before displaying a window on aarch64
Linux.
@cynecx
Copy link
Contributor

cynecx commented Mar 17, 2022

This is a duplicate of #802.

@glandium
Copy link
Author

@taiki-e Someone contacted you about https://bugzilla.mozilla.org/show_bug.cgi?id=1757571. This is where we are investigation-wise. Applying this patch fixes the issue. That would suggest some kind of race condition somewhere else, but I'm yet to find out the root cause of the problem. The change itself is good to apply anyways.

@glandium
Copy link
Author

Heh, I guess I should have checked pull requests.

@glandium glandium closed this Mar 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants