Skip to content

Exclude cgroup implementation from non-Linux Unix builds #126738

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
jkotas merged 7 commits into from
Apr 20, 2026
+65 −40
Merged

Exclude cgroup implementation from non-Linux Unix builds #126738

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
a1c1573
Initial plan
Copilot Apr 10, 2026
689c080
Exclude cgroup code from non-Linux Unix builds using TARGET_LINUX/TAR…
Copilot Apr 10, 2026
bdf2f36
Move PAL_GetCpuLimit into both branches of the TARGET_LINUX/TARGET_AN…
Copilot Apr 10, 2026
da6f367
Extend single TARGET_LINUX guard to cover GetRestrictedPhysicalMemory…
Copilot Apr 10, 2026
2445c5b
Move TARGET_LINUX guard before includes; remove redundant platform-co…
Copilot Apr 10, 2026
1ab725d
Restore original variable declaration order in GetPhysicalMemoryUsed …
Copilot Apr 10, 2026
06109a0
Merge branch 'main' into copilot/exclude-cgroup-variants-non-linux
jkotas Apr 18, 2026
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
30 changes: 27 additions & 3 deletions src/coreclr/gc/unix/cgroup.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,8 @@ Module Name:
#define SIZE_T_MAX (~(size_t)0)
#endif

#if defined(TARGET_LINUX) || defined(TARGET_ANDROID)
Comment thread
jkotas marked this conversation as resolved.
Outdated

#define CGROUP2_SUPER_MAGIC 0x63677270

#define PROC_MOUNTINFO_FILENAME "/proc/self/mountinfo"
Expand Down Expand Up @@ -619,13 +621,13 @@ size_t GetRestrictedPhysicalMemoryLimit()

bool GetPhysicalMemoryUsed(size_t* val)
{
if (val == nullptr)
return false;

bool result = false;
size_t linelen;
char* line = nullptr;

if (val == nullptr)
Comment thread
jkotas marked this conversation as resolved.
return false;

// Linux uses cgroup usage to trigger oom kills.
if (CGroup::GetPhysicalMemoryUsage(val))
return true;
Expand Down Expand Up @@ -656,3 +658,25 @@ bool GetPhysicalMemoryUsed(size_t* val)
free(line);
return result;
}

#else // !(TARGET_LINUX || TARGET_ANDROID)

void InitializeCGroup()
{
}

void CleanupCGroup()
{
}

size_t GetRestrictedPhysicalMemoryLimit()
{
return 0;
}

bool GetPhysicalMemoryUsed(size_t* val)
{
Comment thread
jkotas marked this conversation as resolved.
return false;
}

#endif // TARGET_LINUX || TARGET_ANDROID
15 changes: 15 additions & 0 deletions src/coreclr/nativeaot/Runtime/unix/cgroupcpu.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,8 @@ Module Name:

#include "cgroupcpu.h"

#if defined(TARGET_LINUX) || defined(TARGET_ANDROID)

#define CGROUP2_SUPER_MAGIC 0x63677270

#define BASE_TEN 10
Expand Down Expand Up @@ -510,3 +512,16 @@ bool GetCpuLimit(uint32_t* val)

return CGroup::GetCpuLimit(val);
}

#else // !(TARGET_LINUX || TARGET_ANDROID)

void InitializeCpuCGroup()
{
}

bool GetCpuLimit(uint32_t* val)
{
Comment thread
jkotas marked this conversation as resolved.
return false;
}

#endif // TARGET_LINUX || TARGET_ANDROID
21 changes: 21 additions & 0 deletions src/coreclr/pal/src/misc/cgroup.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,8 @@ SET_DEFAULT_DEBUG_CHANNEL(MISC);
#include <sys/vfs.h>
#endif

#if defined(TARGET_LINUX) || defined(TARGET_ANDROID)

#define CGROUP2_SUPER_MAGIC 0x63677270

#define BASE_TEN 10
Expand Down Expand Up @@ -525,3 +527,22 @@ PAL_GetCpuLimit(UINT* val)

return CGroup::GetCpuLimit(val);
}

#else // !(TARGET_LINUX || TARGET_ANDROID)

void InitializeCGroup()
{
}

void CleanupCGroup()
{
}

BOOL
PALAPI
Comment thread
jkotas marked this conversation as resolved.
PAL_GetCpuLimit(UINT* val)
{
Comment thread
jkotas marked this conversation as resolved.
return FALSE;
}

#endif // TARGET_LINUX || TARGET_ANDROID
Loading