Skip to content

Add the ALC to DumpMT #3508

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.

Sign up for GitHub

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
hoyosjs merged 10 commits into from
Nov 22, 2022
Merged

Add the ALC to DumpMT #3508

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
0f7ad7f
Add the ALC to DumpMT
AaronRobinsonMSFT Nov 14, 2022
6dd921f
Update to use DML for ALC
AaronRobinsonMSFT Nov 14, 2022
bc2a954
Reduce change for testing
AaronRobinsonMSFT Nov 14, 2022
e88b0c3
Reduce change for testing
AaronRobinsonMSFT Nov 15, 2022
075a717
Bring back QI for ISOSDacInterface8
AaronRobinsonMSFT Nov 15, 2022
2ed6f09
Acquire ALC
AaronRobinsonMSFT Nov 15, 2022
93bc515
Revert back to original fix
AaronRobinsonMSFT Nov 15, 2022
219fc7e
Disable dumpmt for 7.0
hoyosjs Nov 21, 2022
805988e
Check for at least .NET 7
AaronRobinsonMSFT Nov 22, 2022
f0fa97f
Don't leave a failure status code.
AaronRobinsonMSFT Nov 22, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
29 changes: 24 additions & 5 deletions src/SOS/Strike/strike.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1276,7 +1276,7 @@ DECLARE_API(DumpMT)
}

EnableDMLHolder dmlHolder(dml);
TableOutput table(2, 16, AlignLeft, false);
TableOutput table(2, 20, AlignLeft, false);

if (nArg == 0)
{
Expand Down Expand Up @@ -1326,6 +1326,25 @@ DECLARE_API(DumpMT)
}
}

ReleaseHolder<ISOSDacInterface8> sos8;
if (SUCCEEDED(Status = g_sos->QueryInterface(__uuidof(ISOSDacInterface8), &sos8)))
{
CLRDATA_ADDRESS assemblyLoadContext = 0;
Status = sos8->GetAssemblyLoadContext(TO_CDADDR(dwStartAddr), &assemblyLoadContext);
if (SUCCEEDED(Status))
{
const char* title = "AssemblyLoadContext:";
if (assemblyLoadContext != 0)
{
table.WriteRow(title, PrefixHex(assemblyLoadContext));
}
else
{
table.WriteRow(title, "Default ALC - The managed instance of this context doesn't exist yet.");
}
}
}

table.WriteRow("BaseSize:", PrefixHex(vMethTable.BaseSize));
table.WriteRow("ComponentSize:", PrefixHex(vMethTable.ComponentSize));
table.WriteRow("DynamicStatics:", vMethTable.bIsDynamic ? "true" : "false");
Expand Down Expand Up @@ -10338,7 +10357,7 @@ DECLARE_API(EEVersion)
else
ExtOut("Workstation mode\n");

if (!GetGcStructuresValid())
if (!GetGcStructuresValid())
{
ExtOut("In plan phase of garbage collection\n");
}
Expand Down Expand Up @@ -15797,7 +15816,7 @@ class EnumMemoryCallback : public ICLRDataEnumMemoryRegionsCallback, ICLRDataLog
}
}
if (IsInterrupt())
{
{
return COR_E_OPERATIONCANCELED;
}
return S_OK;
Expand All @@ -15808,7 +15827,7 @@ class EnumMemoryCallback : public ICLRDataEnumMemoryRegionsCallback, ICLRDataLog
{
ExtOut("%s", message);
if (IsInterrupt())
{
{
return COR_E_OPERATIONCANCELED;
}
return S_OK;
Expand All @@ -15824,7 +15843,7 @@ DECLARE_API(enummem)
if (SUCCEEDED(Status))
{
ToRelease<ICLRDataEnumMemoryRegionsCallback> callback = new EnumMemoryCallback(false, true);
ULONG32 minidumpType =
ULONG32 minidumpType =
(MiniDumpWithPrivateReadWriteMemory |
MiniDumpWithDataSegs |
MiniDumpWithHandleData |
Expand Down