thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-21 15:12:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Incorporate review feedback

Browse Source
This commit is contained in:
Robert Schulze 2024-05-02 08:09:23 +00:00
parent bf7802025f
commit 5520f26f18
No known key found for this signature in database
GPG Key ID: 26703B55FB13728A
1 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
base/base/cgroupsv2.cpp
View File

@ -23,9 +23,9 @@ bool cgroupsV2MemoryControllerEnabled()
{
#if defined(OS_LINUX)
chassert(cgroupsV2Enabled());
/// According to https://docs.kernel.org/admin-guide/cgroup-v2.html, file "cgroup.controllers" defines
/// which controllers are enabled for the current cgroup. Check the bottom-most nested "cgroup.controllers"
/// file.
/// According to https://docs.kernel.org/admin-guide/cgroup-v2.html, file "cgroup.controllers" defines which controllers are available
/// for the current + child cgroups. The set of available controllers can be restricted from level to level using file
/// "cgroups.subtree_control". It is therefore sufficient to check the bottom-most nested "cgroup.controllers" file.
std::string cgroup = cgroupV2OfProcess();
auto cgroup_dir = cgroup.empty() ? default_cgroups_mount : (default_cgroups_mount / cgroup);
std::ifstream controllers_file(cgroup_dir / "cgroup.controllers");
@ -33,9 +33,7 @@ bool cgroupsV2MemoryControllerEnabled()
return false;
std::string controllers;
std::getline(controllers_file, controllers);
if (controllers.find("memory") != std::string::npos)
return true;
return false;
return controllers.find("memory") != std::string::npos;
#else
return false;
#endif