Skip to content

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

Merged
jeffschwMSFT merged 1 commit into
release/6.0-rc1from
backport/pr-57665-to-release/6.0-rc1
Aug 19, 2021
Merged

[release/6.0-rc1] [mono] Fix crash in common_call_trampoline due to inconsistent rgctx mode#57677
jeffschwMSFT merged 1 commit into
release/6.0-rc1from
backport/pr-57665-to-release/6.0-rc1

Conversation

@github-actions

@github-actionsgithub-actionsBot commented Aug 18, 2021

Copy link
Copy Markdown
Contributor

Backport of #57665 to release/6.0-rc1

/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-rc1

/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 take for consideration in RC1.

@lewinglewing added Servicing-consider Issue for next servicing release review Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Aug 19, 2021
@lewing

Copy link
Copy Markdown
Member

Approved in email.

@jeffschwMSFT

Copy link
Copy Markdown
Member

@lewing can you request a cr and then we can merge.

@jeffschwMSFT
jeffschwMSFT merged commit fe6bd2d into release/6.0-rc1Aug 19, 2021
@lewing
lewing deleted the backport/pr-57665-to-release/6.0-rc1 branch August 19, 2021 18:51
@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.

Labels

area-VM-meta-monoServicing-approvedApproved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@lewing@jeffschwMSFT@uweigand