This is an automated email from the ASF dual-hosted git repository. maxyang pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudberry.git
commit 8ec795a53ad1ebc3025cdf21accbe58c5e19e1f8 Author: FairyFar <[email protected]> AuthorDate: Wed Apr 10 20:24:01 2024 +0800 Removed the permission check for `cpu.pressure` item when resource manager sets to 'group-v2'. `cpu.pressure` is one of API for PSI, but PSI(Pressure Stall Information) is an optional feature of the Linux kernel. PSI is not enabled by default on some Linux distributions. This causes GPDB to FAIL to start if `gp_resource_manager='group-v2'`. On the other hand, `cpu.pressure` is not used in GPDB resource management, so it can be removed. --- gpMgmt/bin/gpcheckresgroupv2impl | 1 - src/backend/utils/resgroup/cgroup-ops-linux-v2.c | 1 - 2 files changed, 2 deletions(-) diff --git a/gpMgmt/bin/gpcheckresgroupv2impl b/gpMgmt/bin/gpcheckresgroupv2impl index e08798c34b..4e15d048eb 100755 --- a/gpMgmt/bin/gpcheckresgroupv2impl +++ b/gpMgmt/bin/gpcheckresgroupv2impl @@ -47,7 +47,6 @@ class CgroupValidationVersionTwo(CgroupValidation): self.validate_permission("gpdb/cgroup.procs", "rw") self.validate_permission("gpdb/cpu.max", "rw") - self.validate_permission("gpdb/cpu.pressure", "rw") self.validate_permission("gpdb/cpu.weight", "rw") self.validate_permission("gpdb/cpu.weight.nice", "rw") self.validate_permission("gpdb/cpu.stat", "r") diff --git a/src/backend/utils/resgroup/cgroup-ops-linux-v2.c b/src/backend/utils/resgroup/cgroup-ops-linux-v2.c index 31fb345a5a..bdb4df6731 100644 --- a/src/backend/utils/resgroup/cgroup-ops-linux-v2.c +++ b/src/backend/utils/resgroup/cgroup-ops-linux-v2.c @@ -91,7 +91,6 @@ static int64 system_cfs_quota_us = -1LL; static const PermItem perm_items_cpu[] = { { CGROUP_COMPONENT_PLAIN, "cpu.max", R_OK | W_OK }, - { CGROUP_COMPONENT_PLAIN, "cpu.pressure", R_OK | W_OK }, { CGROUP_COMPONENT_PLAIN, "cpu.weight", R_OK | W_OK }, { CGROUP_COMPONENT_PLAIN, "cpu.weight.nice", R_OK | W_OK }, { CGROUP_COMPONENT_PLAIN, "cpu.stat", R_OK }, --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
