Skip to content

Guard against -1 Returned from sysconf for the Cache Sizes Causing Large Gen0 Sizes and Budgets for Certain Linux Distributions. #100502

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
mrsharm merged 14 commits into from
Apr 3, 2024
Merged

Guard against -1 Returned from sysconf for the Cache Sizes Causing Large Gen0 Sizes and Budgets for Certain Linux Distributions. #100502

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
198931a
Logging.
mrsharm Apr 1, 2024
b499a3a
Fixed comparison check
mrsharm Apr 1, 2024
3177f87
Fix logical operations
mrsharm Apr 1, 2024
a7bc64f
Completely guard against the cacheSize as UINTMAX_MAX
mrsharm Apr 1, 2024
a6677b1
Fix for right macro
mrsharm Apr 1, 2024
5868885
Ensure we are guarded against all cases where cacheSize == SIZE_MAX
mrsharm Apr 1, 2024
fadec3f
Added an extra guard and removed redundant case
mrsharm Apr 2, 2024
3f8edd8
Comment clean
mrsharm Apr 2, 2024
5e947b6
Added some additional asserts
mrsharm Apr 2, 2024
b442a1b
Removed unnecessary checks for cacheSize == SIZE_MAX
mrsharm Apr 2, 2024
692dd91
Cleaned up logic
mrsharm Apr 2, 2024
d765203
Fix type casting comparison
mrsharm Apr 2, 2024
c40ebd3
Removed redundant comment
mrsharm Apr 3, 2024
3469c01
Removed one more unneccesary guard
mrsharm Apr 3, 2024
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
Prev Previous commit
Next Next commit
Ensure we are guarded against all cases where cacheSize == SIZE_MAX
  • Loading branch information
@mrsharm
mrsharm committed Apr 1, 2024
commit 5868885759075c7843e55a5acb01db5c52440a4b
4 changes: 2 additions & 2 deletions src/coreclr/gc/unix/gcenv.unix.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -930,7 +930,7 @@ static size_t GetLogicalProcessorCacheSizeFromOS()
#endif

#if (defined(HOST_ARM64) || defined(HOST_LOONGARCH64)) && !defined(TARGET_APPLE)
if (cacheSize == 0)
if (cacheSize == 0 || cacheSize == SIZE_MAX)
{
// We expect to get the L3 cache size for Arm64 but currently expected to be missing that info
// from most of the machines.
Expand Down Expand Up @@ -958,7 +958,7 @@ static size_t GetLogicalProcessorCacheSizeFromOS()
#endif

#if HAVE_SYSCTLBYNAME
if (cacheSize == 0)
if (cacheSize == 0 || cacheSize == SIZE_MAX)
{
int64_t cacheSizeFromSysctl = 0;
size_t sz = sizeof(cacheSizeFromSysctl);
Expand Down