From 3d57bee719e6db6b19dce4026bdc7c47b1c3519b Mon Sep 17 00:00:00 2001 From: Koundinya Veluri Date: Thu, 10 Jun 2021 15:57:10 -0700 Subject: [PATCH] Update `ThreadCounts` usage based on a change (#2324) - Depends on https://github.com/dotnet/runtime/pull/53471 - In the above change `ThreadCounts._data` was changed from `ulong` to `uint`. Its usage in SOS was reading 8 bytes and only using the lower 4 bytes. Updated to read 4 bytes instead. No functional change, just updated to match. --- src/SOS/Strike/strike.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/SOS/Strike/strike.cpp b/src/SOS/Strike/strike.cpp index 203b32dc5..3ccf76e03 100644 --- a/src/SOS/Strike/strike.cpp +++ b/src/SOS/Strike/strike.cpp @@ -8626,6 +8626,7 @@ DECLARE_API(ThreadPool) do // while (false) { UINT64 ui64Value = 0; + UINT32 ui32Value = 0; // Determine if the portable thread pool is enabled if (FAILED( @@ -8722,12 +8723,12 @@ DECLARE_API(ThreadPool) accumulatedOffset += offset; offset = GetValueFieldOffset(vCountsField.MTOfType, W("_data")); - if (offset < 0 || FAILED(MOVE(ui64Value, cdaTpInstance + accumulatedOffset + offset))) + if (offset < 0 || FAILED(MOVE(ui32Value, cdaTpInstance + accumulatedOffset + offset))) { ExtOut(" %s\n", "Failed to read PortableThreadPool._separated.counts._data"); break; } - UINT64 data = ui64Value; + UINT32 data = ui32Value; const UINT8 NumProcessingWorkShift = 0; const UINT8 NumExistingThreadsShift = 16; -- 2.34.1