Skip to content

[release/8.0] Guard against -1 Returned from sysconf for the Cache Sizes Causing Large Gen0 Sizes and Budgets for Certain Linux Distributions. #100573

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

Closed
github-actions wants to merge 14 commits into from
Closed

[release/8.0] Guard against -1 Returned from sysconf for the Cache Sizes Causing Large Gen0 Sizes and Budgets for Certain Linux Distributions. #100573

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
48682af
Logging.
mrsharm Apr 1, 2024
d4e7abc
Fixed comparison check
mrsharm Apr 1, 2024
15beb12
Fix logical operations
mrsharm Apr 1, 2024
0a7a08d
Completely guard against the cacheSize as UINTMAX_MAX
mrsharm Apr 1, 2024
440ae97
Fix for right macro
mrsharm Apr 1, 2024
6266af0
Ensure we are guarded against all cases where cacheSize == SIZE_MAX
mrsharm Apr 1, 2024
3257f16
Added an extra guard and removed redundant case
mrsharm Apr 2, 2024
750632a
Comment clean
mrsharm Apr 2, 2024
5f787a5
Added some additional asserts
mrsharm Apr 2, 2024
13e973b
Removed unnecessary checks for cacheSize == SIZE_MAX
mrsharm Apr 2, 2024
d162afe
Cleaned up logic
mrsharm Apr 2, 2024
65d7638
Fix type casting comparison
mrsharm Apr 2, 2024
b3dec7e
Removed redundant comment
mrsharm Apr 3, 2024
6efb287
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 authored and github-actions committed Apr 3, 2024
commit 6266af0043f7480bbd94c572c0a7fee6718d35a5
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 @@ -854,7 +854,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 @@ -882,7 +882,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