!60 [sync] PR-54: 解决libcgroup工具无法挂载多个controller到同一挂载点的问题

From: @openeuler-sync-bot Reviewed-by: @xuxuepeng Signed-off-by: @xuxuepeng
Add support for multiple controller mount to same point
2024-12-09 12:30:13 +00:00 · 2024-12-09 19:24:05 +08:00 · 2024-11-29 03:44:32 +00:00 · 2024-11-04 20:08:38 +08:00 · 2024-07-15 09:19:46 +00:00 · 2024-07-15 16:46:20 +08:00
6 changed files with 143 additions and 72 deletions
--- a/Add-support-for-multiple-controllers-on-same-mount-point.patch
+++ b/Add-support-for-multiple-controllers-on-same-mount-point.patch
@ -0,0 +1,73 @@
+From d9c7eee7cbae3de3b092fc317ff3e002e1a02bf9 Mon Sep 17 00:00:00 2001
+From: lipengyu <lipengyu@kylinos.cn>
+Date: Fri, 6 Dec 2024 14:38:39 +0800
+Subject: [PATCH] support-for-multiple-controller-nount-to-same-point
+
+---
+ src/config.c | 29 ++++++++++++++++++++++++++++-
+ 1 file changed, 28 insertions(+), 1 deletion(-)
+
+diff --git a/src/config.c b/src/config.c
+index 7678578..4d33b0d 100644
+--- a/src/config.c
+++ b/src/config.c
+@@ -629,6 +629,22 @@ void cgroup_config_cleanup_namespace_table(void)
+ 	memset(&config_namespace_table, 0, sizeof(struct cg_mount_table_s) * CG_CONTROLLER_MAX);
+ }
+ 
+static int is_valid_controller(char *ctrl)
+{
+	int i;
+
+	static const char *ctrl_list[] = { "blkio", "cpu", "cpuacct", "cpuset", "devices",
+					"freezer", "hugetlb", "memory", "misc", "net_cls",
+					"net_prio", "perf_event", "pids", "rdma", NULL };
+
+	for (i = 0; ctrl_list[i]; i++) {
+		if (strncmp(ctrl, ctrl_list[i], strlen(ctrl_list[i])) == 0)
+			return 1;
+	}
+
+	return 0;
+}
+
+ /**
+  * Add necessary options for mount. Currently only 'none' option is added
+  * for mounts with only 'name=xxx' and without real controller.
+@@ -657,6 +673,9 @@ static int cgroup_config_ajdust_mount_options(struct cg_mount_table_s *mount, un
+ 
+ 				strncpy(mount->name, controller, sizeof(mount->name));
+ 				mount->name[sizeof(mount->name)-1] = '\0';
+				free(controller);
+				token = strtok_r(NULL, ",", &save);
+				continue;
+ 			}
+ 
+ 			if (strncmp(token, "nodev", strlen("nodev")) == 0)
+@@ -668,6 +687,15 @@ static int cgroup_config_ajdust_mount_options(struct cg_mount_table_s *mount, un
+ 			if (strncmp(token, "nosuid", strlen("nosuid")) == 0)
+ 				*flags |= MS_NOSUID;
+ 
+			if (is_valid_controller(token)) {
+				controller = strdup(token);
+				if (controller == NULL)
+					break;
+				strncat(mount->name, ",", FILENAME_MAX - strlen(mount->name)-1);
+				strncat(mount->name, controller, FILENAME_MAX - strlen(mount->name) - 1);
+				free(controller);
+			}
+
+ 		} else if (!name_only) {
+ 			/*
+ 			 * We have controller + name=, do the right thing,
+@@ -679,7 +707,6 @@ static int cgroup_config_ajdust_mount_options(struct cg_mount_table_s *mount, un
+ 		token = strtok_r(NULL, ",", &save);
+ 	}
+ 
+-	free(controller);
+ 	free(opts);
+ 
+ 	if (name_only) {
+-- 
+2.25.1
+
--- a/cgconfig.service
+++ b/cgconfig.service
@ -1,18 +0,0 @@
-[Unit]
-Description=Control Group configuration service
-
-# The service should be able to start as soon as possible,
-# before any 'normal' services:
-DefaultDependencies=no
-Conflicts=shutdown.target
-Before=basic.target shutdown.target
-
-[Service]
-Type=oneshot
-RemainAfterExit=yes
-Delegate=yes
-ExecStart=/usr/sbin/cgconfigparser -l /etc/cgconfig.conf -s 1664
-ExecStop=/usr/sbin/cgclear -l /etc/cgconfig.conf -e
-
-[Install]
-WantedBy=sysinit.target
--- a/config.patch
+++ b/config.patch
@ -0,0 +1,13 @@
+--- libcgroup-3.1.0/samples/config/cgconfig.sysconfig.orig	2019-12-19 11:25:25.547000000 +0800
+++ libcgroup-3.1.0/samples/config/cgconfig.sysconfig	2019-12-19 11:26:01.142000000 +0800
+@@ -5,8 +5,5 @@
+ # controller to limit cpu.shares of this default group and allowing some more
+ # important group take most of the CPU.
+ #
+-# By default, create these groups:
+-CREATE_DEFAULT=yes
+-
+-# Uncomment following line to disable creation of the default group on startup:
+-# CREATE_DEFAULT=no
+# By default, do not create these groups:
+CREATE_DEFAULT=no
--- a/libcgroup-3.1.0.tar.gz
+++ b/libcgroup-3.1.0.tar.gz
--- a/libcgroup.spec
+++ b/libcgroup.spec
@ -5,13 +5,17 @@
 Summary: Libcgroup is a library that abstracts the control group file system in Linux
 Name: libcgroup
 Version: 3.1.0
-Release: 1
-License: LGPLv2+
-URL: http://libcg.sourceforge.net/
-Source0: https://github.com/%{name}/%{name}/archive/v%{version}/%{name}-%{version}.tar.gz
-Source1: cgconfig.service
-Provides: libcgroup-pam libcgroup-tools
-Obsoletes: libcgroup-pam libcgroup-tools
+Release: 6
+License: LGPL-2.1-or-later
+URL: https://libcg.sourceforge.net/
+Source: https://github.com/%{name}/%{name}/archive/v%{version}/%{name}-%{version}.tar.gz
+Provides: libcgroup-pam = %{version}-%{release}
+Provides: libcgroup-tools < %{version}-%{release}
+Obsoletes: libcgroup-pam = %{version}-%{release}
+Obsoletes: libcgroup-tools < %{version}-%{release}
+
+Patch0: config.patch
+Patch1: Add-support-for-multiple-controllers-on-same-mount-point.patch

 BuildRequires: autoconf, automake, libtool
 BuildRequires: gcc,gcc-c++,byacc
@ -32,37 +36,32 @@ Requires: %{name}%{?_isa} = %{version}-%{release}
 Devel provides API for creating,deleting and modifying cgroup nodes.It allows
 the creation of cgroups' configuration and provides scripts for managing it.

-%package help
-Summary: It provides helpful information for libcgroup
-%description help
-It provides helpful information for libcgroup-pam,libcgroup-devel,libcgroup-tools and libcgroup.
+%package_help

 %prep
-%setup  -q  -n %{name}-%{version}
+%autosetup -p1 -n %{name}-%{version}

 %build
 autoreconf -vif
-%configure --enable-pam-module-dir=%{_libdir}/security --enable-opaque-hierarchy="name=systemd" --disable-daemon
+%configure --enable-pam-module-dir=%{_libdir}/security --enable-opaque-hierarchy="name=systemd" --disable-daemon --disable-static
 %ifarch sw_64
 make
 %else
-make %{?_smp_mflags}
+%make_build
 %endif

 %install
-make DESTDIR=$RPM_BUILD_ROOT install
+%make_install
+%delete_la

 # config
 install -d ${RPM_BUILD_ROOT}%{_sysconfdir}
 install -d ${RPM_BUILD_ROOT}%{_sysconfdir}/sysconfig
-install -m 644 samples/config/cgconfig.conf $RPM_BUILD_ROOT%{_sysconfdir}/cgconfig.conf
 install -m 644 samples/config/cgsnapshot_denylist.conf $RPM_BUILD_ROOT%{_sysconfdir}/cgsnapshot_denylist.conf

 # Only one pam_cgroup.so is needed
 #mv -f $RPM_BUILD_ROOT%{_libdir}/security/pam_cgroup.so $RPM_BUILD_ROOT%{_libdir}/security/pam_cgroup.so
-rm -f $RPM_BUILD_ROOT%{_libdir}/security/pam_cgroup.{,l}a $RPM_BUILD_ROOT%{_libdir}/security/pam_cgroup.so.*
-
-rm -f $RPM_BUILD_ROOT%{_libdir}/*.{,l}a
+rm -f $RPM_BUILD_ROOT%{_libdir}/security/pam_cgroup.so.*

 rm -f $RPM_BUILD_ROOT%{_libdir}/libcgroupfortesting.*

@ -70,51 +69,29 @@ rm -f $RPM_BUILD_ROOT%{_mandir}/man5/cgred.conf.5*
 rm -f $RPM_BUILD_ROOT%{_mandir}/man5/cgrules.conf.5*
 rm -f $RPM_BUILD_ROOT%{_mandir}/man8/cgrulesengd.8*

-# unit and sysconfig
-install -d ${RPM_BUILD_ROOT}%{_unitdir}
-install -m 644 %SOURCE1 ${RPM_BUILD_ROOT}%{_unitdir}/
-
 %pre
 getent group cgred >/dev/null || groupadd -r cgred

-%post
-%systemd_post cgconfig.service
-
-%preun
-%systemd_preun cgconfig.service
-
-%postun
-%systemd_postun_with_restart cgconfig.service
-
-%triggerun -- libcgroup < 0.38
-/usr/bin/systemd-sysv-convert --save cgconfig >/dev/null 2>&1 ||:
-/sbin/chkconfig --del cgconfig >/dev/null 2>&1 || :
-/bin/systemctl try-restart cgconfig.service >/dev/null 2>&1 || :
-
 %files
-%{!?_licensedir:%global license %%doc}
 %license COPYING
 %{_libdir}/libcgroup.so.*
 %config(noreplace) %{_sysconfdir}/cgsnapshot_denylist.conf
-%config(noreplace) %{_sysconfdir}/cgconfig.conf
-/usr/bin/cgget
-/usr/bin/cgset
-/usr/bin/cgcreate
-/usr/bin/cgdelete
-/usr/bin/cgsnapshot
-/usr/bin/lscgroup
-/usr/bin/lssubsys
-/usr/sbin/cgconfigparser
-%attr(2755, root, cgred) /usr/bin/cgexec
-%attr(2755, root, cgred) /usr/bin/cgclassify
+%{_bindir}/cgget
+%{_bindir}/cgset
+%{_bindir}/cgcreate
+%{_bindir}/cgdelete
+%{_bindir}/cgsnapshot
+%{_bindir}/lscgroup
+%{_bindir}/lssubsys
+%{_sbindir}/cgconfigparser
+%attr(2755, root, cgred) %{_bindir}/cgexec
+%attr(2755, root, cgred) %{_bindir}/cgclassify
 %attr(0755, root, root) %{_libdir}/security/pam_cgroup.so
-%{_unitdir}/cgconfig.service
 %{_bindir}/cgxget
 %{_bindir}/cgxset
 %{_bindir}/libcgroup_systemd_idle_thread

 %files devel
-%{!?_licensedir:%global license %%doc}
 %license COPYING
 %{_libdir}/libcgroup.so
 %{_libdir}/pkgconfig/libcgroup.pc
@ -122,13 +99,39 @@ getent group cgred >/dev/null || groupadd -r cgred
 %{_includedir}/libcgroup/*.h

 %files help
-%license COPYING
 %doc README README_systemd
 %attr(0644, root, root) %{_mandir}/man1/*
 %attr(0644, root, root) %{_mandir}/man5/*
 %attr(0644, root, root) %{_mandir}/man8/*

 %changelog
+* Tue Nov 26 2024 lipengyu <lipengyu@kylinos.cn> - 3.1.0-6
+- Type:update
+- Id:NA
+- SUG:NA
+- DESC:Add support for multiple controller mount to same point
+
+* Fri Oct 11 2024 Funda Wang <fundawang@yeah.net> - 3.1.0-5
+- apply patch
+
+* Fri Jul 12 2024 liuxu <liuxu156@huawei.com> - 3.1.0-4
+- Type: bugfix
+- Id:NA
+- SUG:NA
+- DESC:fix config.patch
+
+* Fri Apr 12 2024 liuxu <liuxu156@huawei.com> - 3.1.0-3
+- Type: bugfix
+- Id:NA
+- SUG:NA
+- DESC:fix corrupted libcgroup-3.1.0.tar.gz
+
+* Mon Aug 28 2023 luoyujie <luoyujie5@huawei.com> - 3.1.0-2
+- Type: enhancement
+- Id:NA
+- SUG:NA
+- DESC:delete unnecessary cgconfig service support
+
 * Wed Aug 16 2023 luoyujie <luoyujie5@huawei.com> - 3.1.0-1
 - Type: upgrade
 - Id:NA
--- a/libcgroup.yaml
+++ b/libcgroup.yaml
@ -1,4 +1,4 @@
 version_control: github
 src_repo: libcgroup/libcgroup
-tag_prefix:
-seperator: "."
+tag_prefix: ^v
+separator: "."
Author	SHA1	Message	Date
openeuler-ci-bot	4a2176c1ec	!60 [sync] PR-54: 解决libcgroup工具无法挂载多个controller到同一挂载点的问题 From: @openeuler-sync-bot Reviewed-by: @xuxuepeng Signed-off-by: @xuxuepeng	2024-12-09 12:30:13 +00:00
lipengyu	c236534ee1	Add support for multiple controller mount to same point (cherry picked from commit 3874189ca454292a222194033c3802978f0f9642)	2024-12-09 19:24:05 +08:00
openeuler-ci-bot	2b37537bfa	!50 [sync] PR-49: apply patch From: @openeuler-sync-bot Reviewed-by: @xuxuepeng Signed-off-by: @xuxuepeng	2024-11-29 03:44:32 +00:00
Funda Wang	93bfdf16af	apply patch (cherry picked from commit 4f8a8d6ea6bbb5f90497e51770fd643817198379)	2024-11-04 20:08:38 +08:00
openeuler-ci-bot	25a8455a9a	!43 [sync] PR-42: fix config.patch From: @openeuler-sync-bot Reviewed-by: @xuxuepeng Signed-off-by: @xuxuepeng	2024-07-15 09:19:46 +00:00
liuxu	d10ce759cd	fix config.patch Signed-off-by: liuxu <liuxu156@huawei.com> (cherry picked from commit 9a85b38ffe056971ab85cba631deb45c2d6fdd63)	2024-07-15 16:46:20 +08:00
openeuler-ci-bot	cbf002fb4a	!38 [sync] PR-37: fix corrupted libcgroup-3.1.0.tar.gz From: @openeuler-sync-bot Reviewed-by: @flyflyflypeng Signed-off-by: @flyflyflypeng	2024-04-23 03:29:34 +00:00
liuxu	e993696761	fix corrupted libcgroup-3.1.0.tar.gz (cherry picked from commit fb8373098b2292176de6c1cb67bce77800c911f5)	2024-04-12 11:43:57 +08:00
openeuler-ci-bot	9c158533ca	!32 delete cgconfig service support From: @Rose-yujie Reviewed-by: @hcnbxx, @duguhaotian Signed-off-by: @duguhaotian	2023-08-29 03:17:34 +00:00
luoyujie	30a4787758	delete cgconfig service	2023-08-28 19:18:28 +08:00