Skip to content

[release/10.0] Revert forwarding activation signal#124879

Merged
steveisok merged 1 commit intorelease/10.0from
backport/pr-124877-to-release/10.0
Feb 26, 2026
Merged

[release/10.0] Revert forwarding activation signal#124879
steveisok merged 1 commit intorelease/10.0from
backport/pr-124877-to-release/10.0

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Feb 26, 2026

Backport of #124877 to release/10.0

/cc @janvorli

Customer Impact

  • Customer reported
  • Found internally

A recent change that was backported to .NET 10.0.4 (#124308) was found to cause problems to vsdbg. That problem is caused by part of the change that was actually additional to the fix that that change made and that is not that important. That part of the change started to forward activation signal to handlers registered before the coreclr. This change just reverts that part.

Regression

  • Yes
  • No

#124308

Testing

Local testing

Risk

Low. It just stops forwarding the signal which was the behavior until the recent change.

Recent change to fix lookup of current thread in the activation handler
also added forwarding of the activation signal to the previously
registered handler. That part of the change was unrelated to the actual
issue the change was fixing. It turned out that it breaks vsdbg
that doesn't expect to get the SIGUSR1 used for the activation on macOS.

This change reverts that part of the change to unblock vsdbg.
@janvorli janvorli requested a review from jkotas February 26, 2026 00:43
@janvorli janvorli added the Servicing-consider Issue for next servicing release review label Feb 26, 2026
@janvorli janvorli added this to the 10.0.x milestone Feb 26, 2026
@janvorli janvorli requested a review from steveisok February 26, 2026 00:49
@rbhanda rbhanda modified the milestones: 10.0.x, 10.0.4 Feb 26, 2026
@rbhanda rbhanda added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Feb 26, 2026
@steveisok steveisok enabled auto-merge (squash) February 26, 2026 02:36
@steveisok steveisok merged commit 19e7456 into release/10.0 Feb 26, 2026
106 of 109 checks passed
@steveisok steveisok deleted the backport/pr-124877-to-release/10.0 branch February 26, 2026 03:23
@mmitche
Copy link
Member

mmitche commented Feb 26, 2026

In order to make 10.0.4, this needs to be ported to the VMR internal/release/10.0.104 branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-PAL-coreclr Servicing-approved Approved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants