Skip to content

Let the debugger knows DATAS is on #5164

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 1 commit into from
Jan 24, 2025

Conversation

cshung
Copy link
Contributor

@cshung cshung commented Jan 9, 2025

This PR is meant to support #5090.

This PR is the diagnostics side of the work, this exposes a new field so that the debugger can display the information.
There are two more PRs on different repos to support this:

The code won't compile until the clrmd dependency is consumed.

On runtime: dotnet/runtime#107115
On clrmd: microsoft/clrmd#1301

@mikem8361
Copy link

The required CLRMD change should be referenced by main now. If you rebase/merge the latest main branch, it should work/build.

@cshung cshung force-pushed the public/datas-debugger branch from 9c997f8 to c3fba0d Compare January 21, 2025 18:13
@mikem8361
Copy link

@cshung you have Linux/Alpine build errors.

@cshung
Copy link
Contributor Author

cshung commented Jan 22, 2025

@cshung you have Linux/Alpine build errors.

I will take a look.

@cshung cshung force-pushed the public/datas-debugger branch from c3fba0d to 17d1f77 Compare January 24, 2025 18:39
@mikem8361 mikem8361 merged commit 18004e5 into dotnet:main Jan 24, 2025
20 checks passed
@cshung cshung deleted the public/datas-debugger branch January 24, 2025 21:59
@@ -448,6 +448,7 @@ private ulong PrintGCHeap(ClrRuntime clrRuntime)
{
Console.WriteLine();
ClrHeap heap = clrRuntime.Heap;
Console.WriteLine("DATAS = " + heap.DynamicAdaptationMode);
Copy link
Member

Choose a reason for hiding this comment

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

I thought it was called dynamic GC now not DATAS

@github-actions github-actions bot locked and limited conversation to collaborators Mar 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants