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

Use monitor ref for all iterations #9443

Open
wants to merge 2 commits into
base: maint-26
Choose a base branch
from

Conversation

garazdawi
Copy link
Contributor

No description provided.

@garazdawi garazdawi added team:VM Assigned to OTP team VM fix labels Feb 14, 2025
@garazdawi garazdawi self-assigned this Feb 14, 2025
Copy link
Contributor

github-actions bot commented Feb 14, 2025

CT Test Results

    3 files     69 suites   1h 2m 8s ⏱️
1 536 tests 1 288 ✅ 244 💤 4 ❌
1 736 runs  1 439 ✅ 293 💤 4 ❌

For more details on these failures, see this check.

Results for commit 969431e.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@garazdawi garazdawi requested a review from bjorng February 20, 2025 12:49
@garazdawi
Copy link
Contributor Author

@bjorng I added a testcase that checks that the receive optimization present, can you check if my approach is reasonable?

@bjorng
Copy link
Contributor

bjorng commented Feb 21, 2025

@garazdawi Yes, it looks reasonable. I've used a similar approach in some test cases.

RaimoNiskanen
RaimoNiskanen previously approved these changes Feb 21, 2025
Copy link
Contributor

@RaimoNiskanen RaimoNiskanen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@garazdawi garazdawi force-pushed the lukas/erts/fix-prim_inet-selective-receive/OTP-19493 branch from cdf48f2 to 969431e Compare February 21, 2025 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix team:VM Assigned to OTP team VM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants