Skip to content

Conversation

@thaystg
Copy link
Member

@thaystg thaystg commented Jan 18, 2024

Backport of these PR's:
#93039
#93363
#93987
#94358
#94066

Customer Impact

This backport is necessary to make possible to use mono iCorDebug to debug apps running on mono runtime in .net8.

Testing

Manually tested.

Risk

Medium. The changes are mainly located in debugger sources and are protected with is using icordebug check. The other changes are already in .net9 for some time and didn't cause any known issue.

thaystg and others added 5 commits January 18, 2024 12:32
…n from HotReload (dotnet#93039)

* Generating seq_points if we don't have pdb information from HotReload

* Apply suggestions from code review

Co-authored-by: Aleksey Kliger (λgeek) <[email protected]>

* Addressing @lambdageek comments

---------

Co-authored-by: Aleksey Kliger (λgeek) <[email protected]>
…#93987)

* Change implementation of OffsetToStringData as it's done on coreclr

* Fix comment

* Remove more comments

* completely remove the comments

* Update RuntimeHelpers.Mono.cs

* Addressing comments

* Addressing @lambdageek comment.
* Trying to fix performance regression.

* Fix performance issue
* Creating new messages, changing some olds, and fixing when jit_end is called while running a invoke_method

* Fix compilation error

* Fix compilation

* Applying suggestions from @lambdageek and adding more changes.

* Addressing @lambdageek suggestions.

* Addressing @lambdageek comment

* Addressing @lambdageek comments

* adding mono_component_api to function added.
@ghost ghost added the area-VM-meta-mono label Jan 18, 2024
@ghost ghost assigned thaystg Jan 18, 2024
@thaystg thaystg changed the title [mono][debugger] Debugger improvements on .net8 [release/8.0][mono][debugger] Debugger improvements on .net8 Jan 18, 2024
@thaystg thaystg marked this pull request as ready for review January 19, 2024 00:22
@thaystg thaystg added the Servicing-consider Issue for next servicing release review label Jan 19, 2024
@marek-safar marek-safar added this to the 8.0.x milestone Jan 19, 2024
@thaystg thaystg added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Jan 20, 2024
@thaystg
Copy link
Member Author

thaystg commented Jan 20, 2024

Approved by email.

@thaystg thaystg merged commit fe443a9 into dotnet:release/8.0-staging Jan 24, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Feb 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-VM-meta-mono Servicing-approved Approved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants