Skip to content

Conversation

@janvorli
Copy link
Member

@janvorli janvorli commented Nov 7, 2025

This change adds check for thread abort to the INTOP_SAFEPOINT and also to the code resuming after catch. This makes all the libraries controlled execution tests pass.

This change adds check for thread abort to the INTOP_SAFEPOINT and also
to the code resuming after catch. This makes all the libraries
controlled execution tests pass.
@janvorli janvorli added this to the 11.0.0 milestone Nov 7, 2025
@janvorli janvorli self-assigned this Nov 7, 2025
Copilot AI review requested due to automatic review settings November 7, 2025 14:04
@janvorli janvorli requested review from BrzVlad and kg as code owners November 7, 2025 14:04
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @BrzVlad, @janvorli, @kg
See info in area-owners.md if you want to be subscribed.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds thread abort handling to the interpreter execution loop in the CoreCLR VM. The changes ensure that thread abort requests are properly detected and handled at safe points during interpreter execution.

  • Adds abort checking at INTOP_SAFEPOINT instruction to allow cooperative thread abort
  • Adds abort handling after exception catch resumption to prevent continuing execution when abort is requested

@janvorli
Copy link
Member Author

janvorli commented Nov 7, 2025

/ba-g the android build timeout is unrelated and happens frequently

@janvorli janvorli merged commit 767be2a into dotnet:main Nov 7, 2025
87 of 99 checks passed
@janvorli janvorli deleted the enable-thread-abort-in-interpreter branch November 7, 2025 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants