Commit b4536f0c authored by Tejun Heo's avatar Tejun Heo
Browse files

cgroup: implement cgroup_subsys->css_reset() 



cgroup is implementing support for subsystem dependency which would
require a way to enable a subsystem even when it's not directly
configured through "cgroup.subtree_control".

The previous patches added support for explicitly and implicitly
enabled subsystems and showing/hiding their interface files.  An
explicitly enabled subsystem may become implicitly enabled if it's
turned off through "cgroup.subtree_control" but there are subsystems
depending on it.  In such cases, the subsystem, as it's turned off
when seen from userland, shouldn't enforce any resource control.
Also, the subsystem may be explicitly turned on later again and its
interface files should be as close to the intial state as possible.

This patch adds cgroup_subsys->css_reset() which is invoked when a css
is hidden.  The callback should disable resource control and reset the
state to the vanilla state.

Signed-off-by: default avatarTejun Heo <tj@kernel.org>
Acked-by: default avatarLi Zefan <lizefan@huawei.com>
Acked-by: default avatarJohannes Weiner <hannes@cmpxchg.org>
parent f63070d3

Documentation/cgroups/cgroups.txt

+14 −0
Original line number Diff line number Diff line
@@ -599,6 +599,20 @@ fork. If this method returns 0 (success) then this should remain valid 
while the caller holds cgroup_mutex and it is ensured that either

attach() or cancel_attach() will be called in future.



void css_reset(struct cgroup_subsys_state *css)

(cgroup_mutex held by caller)



An optional operation which should restore @css's configuration to the

initial state.  This is currently only used on the unified hierarchy

when a subsystem is disabled on a cgroup through

"cgroup.subtree_control" but should remain enabled because other

subsystems depend on it.  cgroup core makes such a css invisible by

removing the associated interface files and invokes this callback so

that the hidden subsystem can return to the initial neutral state.

This prevents unexpected resource control from a hidden css and

ensures that the configuration is in the initial state when it is made

visible again later.



void cancel_attach(struct cgroup *cgrp, struct cgroup_taskset *tset)

(cgroup_mutex held by caller)

include/linux/cgroup.h

+1 −0
Original line number Diff line number Diff line
@@ -642,6 +642,7 @@ struct cgroup_subsys { 
	int (*css_online)(struct cgroup_subsys_state *css);

	void (*css_offline)(struct cgroup_subsys_state *css);

	void (*css_free)(struct cgroup_subsys_state *css);

	void (*css_reset)(struct cgroup_subsys_state *css);



	int (*can_attach)(struct cgroup_subsys_state *css,

			  struct cgroup_taskset *tset);

kernel/cgroup.c

+12 −4
Original line number Diff line number Diff line
@@ -2740,17 +2740,25 @@ static ssize_t cgroup_subtree_control_write(struct kernfs_open_file *of, 
	/*

	 * All tasks are migrated out of disabled csses.  Kill or hide

	 * them.  A css is hidden when the userland requests it to be

	 * disabled while other subsystems are still depending on it.

	 * disabled while other subsystems are still depending on it.  The

	 * css must not actively control resources and be in the vanilla

	 * state if it's made visible again later.  Controllers which may

	 * be depended upon should provide ->css_reset() for this purpose.

	 */

	for_each_subsys(ss, ssid) {

		if (!(disable & (1 << ssid)))

			continue;



		cgroup_for_each_live_child(child, cgrp) {

			if (css_disable & (1 << ssid))

				kill_css(cgroup_css(child, ss));

			else

			struct cgroup_subsys_state *css = cgroup_css(child, ss);



			if (css_disable & (1 << ssid)) {

				kill_css(css);

			} else {

				cgroup_clear_dir(child, 1 << ssid);

				if (ss->css_reset)

					ss->css_reset(css);

			}

		}

	}

CRA Git | Maintained and supported by SUSTech CRA and CCSE