Skip to content

Add ICorDebugHeapValue4 -- CreatePinnedHandle #44471

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

Merged
merged 3 commits into from
Nov 11, 2020

Conversation

sdmaclea
Copy link
Contributor

No description provided.

@sdmaclea sdmaclea added this to the 6.0.0 milestone Nov 10, 2020
@sdmaclea sdmaclea self-assigned this Nov 10, 2020
Copy link
Contributor

@gregg-miskelly gregg-miskelly left a comment

Choose a reason for hiding this comment

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

Otherwise LGTM

@sdmaclea
Copy link
Contributor Author

/azp run runtime

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sdmaclea sdmaclea requested review from noahfalk and hoyosjs November 10, 2020 20:03
@kouvel
Copy link
Contributor

kouvel commented Nov 10, 2020

Thanks @sdmaclea! LGTM, though I'm not too familiar with adding APIs in this area.

@ghost
Copy link

ghost commented Nov 10, 2020

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


Issue meta data
Issue content:
Issue author: sdmaclea
Assignees: sdmaclea
Milestone: [object Object]

@sdmaclea sdmaclea merged commit 2c0f6fb into dotnet:master Nov 11, 2020
@sdmaclea
Copy link
Contributor Author

@kouvel I expect when you have the rest of the tiered JIT, data breakpoint deadlock fix finished. You will backport the whole fix to 5.0 servicing branch.

@kouvel
Copy link
Contributor

kouvel commented Nov 11, 2020

Yep about to put up a PR for my side shortly, will port both once that's in

sdmaclea added a commit to sdmaclea/runtime that referenced this pull request Nov 11, 2020
* Add ICorDebugHeapValue4
* Add EnableGCNotificationEvents deprecation comment
sdmaclea added a commit that referenced this pull request Nov 24, 2020
#44563)

* Add ICorDebugHeapValue4 -- CreatePinnedHandle (#44471)

* Add ICorDebugHeapValue4
* Add EnableGCNotificationEvents deprecation comment

* Drop support for IID_ICorDebugProcess10 and fix thread suspend logic (#44549)

* Stop providing IID_ICorDebugProcess10

Prevent older VS versions from setting managed data breakpoints.

* Simplify the thread collision logic to prevent deadlock

This is a simplification of #44539
as proposed by @kouvel
@ghost ghost locked as resolved and limited conversation to collaborators Dec 11, 2020
Sign up for free to 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.

6 participants