Skip to content

[release/6.0] [mono] Fix crash in common_call_trampoline due to inconsistent rgctx mode - #57676

Closed
github-actions[bot] wants to merge 1 commit into
release/6.0from
backport/pr-57665-to-release/6.0
Closed

[release/6.0] [mono] Fix crash in common_call_trampoline due to inconsistent rgctx mode#57676
github-actions[bot] wants to merge 1 commit into
release/6.0from
backport/pr-57665-to-release/6.0

Conversation

@github-actions

@github-actionsgithub-actionsBot commented Aug 18, 2021

Copy link
Copy Markdown
Contributor

Backport of #57665 to release/6.0

/cc @lambdageek@uweigand

Customer Impact

Code using generic default interface methods may crash when using the Mono (e.g. on Android or ios emulators or devices).

Testing

Manual testing.

Risk

Moderate - we can turn off generic sharing in the JIT, but not FullAOT.

…mode
* Assume MRGCTX mode if mini_method_is_default_method is true
* Fixes#57664
@ghost

Copy link
Copy Markdown

Tagging subscribers to this area:
See info in area-owners.md if you want to be subscribed.

Issue Details

Backport of #57665 to release/6.0

/cc @lambdageek@uweigand

Customer Impact

Testing

Risk

Author:github-actions[bot]
Assignees:-
Labels:

area-VM-meta-mono

Milestone:-

@jeffschwMSFTjeffschwMSFT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved. Please get a CR and we can merge for RC2.

@ViktorHofer

Copy link
Copy Markdown
Member

Closing as #57677 will automatically flow into release/6.0.

@ViktorHofer
ViktorHofer deleted the backport/pr-57665-to-release/6.0 branch August 19, 2021 16:53
@lewinglewing added Servicing-consider Issue for next servicing release review and removed Servicing-consider Issue for next servicing release review labels Aug 19, 2021
@ghostghost locked as resolved and limited conversation to collaborators Sep 18, 2021
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@ViktorHofer@jeffschwMSFT@lewing@uweigand