Revert "Update `ThreadCounts` usage based on a change (#2324)" (#2452)
authorKoundinya Veluri <kouvel@users.noreply.github.com>
Tue, 27 Jul 2021 16:59:43 +0000 (09:59 -0700)
committerGitHub <noreply@github.com>
Tue, 27 Jul 2021 16:59:43 +0000 (09:59 -0700)
- Depends on https://github.com/dotnet/runtime/pull/56346
- Reverted commit 3d57bee719e6db6b19dce4026bdc7c47b1c3519b from PR https://github.com/dotnet/diagnostics/pull/2324 since the relevant change to `ThreadCounts` was reverted in https://github.com/dotnet/runtime/pull/56346

src/SOS/Strike/strike.cpp

index 3d74a7aadf6f43dbfb4abec3dcf1608635d8d08f..7a91246f424cdc2d7aeaac503588b48d06c6a79e 100644 (file)
@@ -8677,7 +8677,6 @@ DECLARE_API(ThreadPool)
     do // while (false)
     {
         UINT64 ui64Value = 0;
-        UINT32 ui32Value = 0;
 
         // Determine if the portable thread pool is enabled
         if (FAILED(
@@ -8774,12 +8773,12 @@ DECLARE_API(ThreadPool)
             accumulatedOffset += offset;
 
             offset = GetValueFieldOffset(vCountsField.MTOfType, W("_data"));
-            if (offset < 0 || FAILED(MOVE(ui32Value, cdaTpInstance + accumulatedOffset + offset)))
+            if (offset < 0 || FAILED(MOVE(ui64Value, cdaTpInstance + accumulatedOffset + offset)))
             {
                 ExtOut("    %s\n", "Failed to read PortableThreadPool._separated.counts._data");
                 break;
             }
-            UINT32 data = ui32Value;
+            UINT64 data = ui64Value;
 
             const UINT8 NumProcessingWorkShift = 0;
             const UINT8 NumExistingThreadsShift = 16;