Skip to content

K
Kernel

openeuler / Kernel
大约 1 年 前同步成功

通知 5
Star 0
Fork 0
K
Kernel

体验新版 GitCode，发现更多精彩内容 >>

提交 eb95419b 编写于 作者: T Tejun Heo
浏览文件
操作

cgroup: pass around cgroup_subsys_state instead of cgroup in subsystem methods 

cgroup is currently in the process of transitioning to using struct
cgroup_subsys_state * as the primary handle instead of struct cgroup *
in subsystem implementations for the following reasons.

* With unified hierarchy, subsystems will be dynamically bound and
  unbound from cgroups and thus css's (cgroup_subsys_state) may be
  created and destroyed dynamically over the lifetime of a cgroup,
  which is different from the current state where all css's are
  allocated and destroyed together with the associated cgroup.  This
  in turn means that cgroup_css() should be synchronized and may
  return NULL, making it more cumbersome to use.

* Differing levels of per-subsystem granularity in the unified
  hierarchy means that the task and descendant iterators should behave
  differently depending on the specific subsystem the iteration is
  being performed for.

* In majority of the cases, subsystems only care about its part in the
  cgroup hierarchy - ie. the hierarchy of css's.  Subsystem methods
  often obtain the matching css pointer from the cgroup and don't
  bother with the cgroup pointer itself.  Passing around css fits
  much better.

This patch converts all cgroup_subsys methods to take @css instead of
@cgroup.  The conversions are mostly straight-forward.  A few
noteworthy changes are

* ->css_alloc() now takes css of the parent cgroup rather than the
  pointer to the new cgroup as the css for the new cgroup doesn't
  exist yet.  Knowing the parent css is enough for all the existing
  subsystems.

* In kernel/cgroup.c::offline_css(), unnecessary open coded css
  dereference is replaced with local variable access.

This patch shouldn't cause any behavior differences.

v2: Unnecessary explicit cgrp->subsys[] deref in css_online() replaced
    with local variable @css as suggested by Li Zefan.

    Rebased on top of new for-3.12 which includes for-3.11-fixes so
    that ->css_free() invocation added by da0a12ca ("cgroup: fix a
    leak when percpu_ref_init() fails") is converted too.  Suggested
    by Li Zefan.
Signed-off-by: NTejun Heo <tj@kernel.org>
Acked-by: NLi Zefan <lizefan@huawei.com>
Acked-by: NMichal Hocko <mhocko@suse.cz>
Acked-by: NVivek Goyal <vgoyal@redhat.com>
Acked-by: NAristeu Rozanski <aris@redhat.com>
Acked-by: NDaniel Wagner <daniel.wagner@bmw-carit.de>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Johannes Weiner <hannes@cmpxchg.org>
Cc: Balbir Singh <bsingharora@gmail.com>
Cc: Matt Helsley <matthltc@us.ibm.com>
Cc: Jens Axboe <axboe@kernel.dk>
Cc: Steven Rostedt <rostedt@goodmis.org>
上级 63876986
隐藏空白更改
内联 并排
Showing with 197 addition and 171 deletion
block/blk-cgroup.c
浏览文件 @ eb95419b
...@@ -765,18 +765,18 @@ struct cftype blkcg_files[] = { ...@@ -765,18 +765,18 @@ struct cftype blkcg_files[] = {
/** /**
* blkcg_css_offline - cgroup css_offline callback * blkcg_css_offline - cgroup css_offline callback
* @cgroup: cgroup of interest * @css: css of interest
* *
* This function is called when @cgroup is about to go away and responsible * This function is called when @css is about to go away and responsible
* for shooting down all blkgs associated with @cgroup. blkgs should be * for shooting down all blkgs associated with @css. blkgs should be
* removed while holding both q and blkcg locks. As blkcg lock is nested * removed while holding both q and blkcg locks. As blkcg lock is nested
* inside q lock, this function performs reverse double lock dancing. * inside q lock, this function performs reverse double lock dancing.
* *
* This is the blkcg counterpart of ioc_release_fn(). * This is the blkcg counterpart of ioc_release_fn().
*/ */
static void blkcg_css_offline(struct cgroup *cgroup) static void blkcg_css_offline(struct cgroup_subsys_state *css)
{ {
struct blkcg *blkcg = cgroup_to_blkcg(cgroup); struct blkcg *blkcg = css_to_blkcg(css);
spin_lock_irq(&blkcg->lock); spin_lock_irq(&blkcg->lock);
...@@ -798,21 +798,21 @@ static void blkcg_css_offline(struct cgroup *cgroup) ...@@ -798,21 +798,21 @@ static void blkcg_css_offline(struct cgroup *cgroup)
spin_unlock_irq(&blkcg->lock); spin_unlock_irq(&blkcg->lock);
} }
static void blkcg_css_free(struct cgroup *cgroup) static void blkcg_css_free(struct cgroup_subsys_state *css)
{ {
struct blkcg *blkcg = cgroup_to_blkcg(cgroup); struct blkcg *blkcg = css_to_blkcg(css);
if (blkcg != &blkcg_root) if (blkcg != &blkcg_root)
kfree(blkcg); kfree(blkcg);
} }
static struct cgroup_subsys_state *blkcg_css_alloc(struct cgroup *cgroup) static struct cgroup_subsys_state *
blkcg_css_alloc(struct cgroup_subsys_state *parent_css)
{ {
static atomic64_t id_seq = ATOMIC64_INIT(0); static atomic64_t id_seq = ATOMIC64_INIT(0);
struct blkcg *blkcg; struct blkcg *blkcg;
struct cgroup *parent = cgroup->parent;
if (!parent) { if (!parent_css) {
blkcg = &blkcg_root; blkcg = &blkcg_root;
goto done; goto done;
} }
...@@ -883,14 +883,15 @@ void blkcg_exit_queue(struct request_queue *q) ...@@ -883,14 +883,15 @@ void blkcg_exit_queue(struct request_queue *q)
* of the main cic data structures. For now we allow a task to change * of the main cic data structures. For now we allow a task to change
* its cgroup only if it's the only owner of its ioc. * its cgroup only if it's the only owner of its ioc.
*/ */
static int blkcg_can_attach(struct cgroup *cgrp, struct cgroup_taskset *tset) static int blkcg_can_attach(struct cgroup_subsys_state *css,
struct cgroup_taskset *tset)
{ {
struct task_struct *task; struct task_struct *task;
struct io_context *ioc; struct io_context *ioc;
int ret = 0; int ret = 0;
/* task_lock() is needed to avoid races with exit_io_context() */ /* task_lock() is needed to avoid races with exit_io_context() */
cgroup_taskset_for_each(task, cgrp, tset) { cgroup_taskset_for_each(task, css->cgroup, tset) {
task_lock(task); task_lock(task);
ioc = task->io_context; ioc = task->io_context;
if (ioc && atomic_read(&ioc->nr_tasks) > 1) if (ioc && atomic_read(&ioc->nr_tasks) > 1)
......
include/linux/cgroup.h
浏览文件 @ eb95419b
...@@ -579,18 +579,22 @@ int cgroup_taskset_size(struct cgroup_taskset *tset); ...@@ -579,18 +579,22 @@ int cgroup_taskset_size(struct cgroup_taskset *tset);
*/ */
struct cgroup_subsys { struct cgroup_subsys {
struct cgroup_subsys_state *(*css_alloc)(struct cgroup *cgrp); struct cgroup_subsys_state *(*css_alloc)(struct cgroup_subsys_state *parent_css);
int (*css_online)(struct cgroup *cgrp); int (*css_online)(struct cgroup_subsys_state *css);
void (*css_offline)(struct cgroup *cgrp); void (*css_offline)(struct cgroup_subsys_state *css);
void (*css_free)(struct cgroup *cgrp); void (*css_free)(struct cgroup_subsys_state *css);
int (*can_attach)(struct cgroup *cgrp, struct cgroup_taskset *tset); int (*can_attach)(struct cgroup_subsys_state *css,
void (*cancel_attach)(struct cgroup *cgrp, struct cgroup_taskset *tset); struct cgroup_taskset *tset);
void (*attach)(struct cgroup *cgrp, struct cgroup_taskset *tset); void (*cancel_attach)(struct cgroup_subsys_state *css,
struct cgroup_taskset *tset);
void (*attach)(struct cgroup_subsys_state *css,
struct cgroup_taskset *tset);
void (*fork)(struct task_struct *task); void (*fork)(struct task_struct *task);
void (*exit)(struct cgroup *cgrp, struct cgroup *old_cgrp, void (*exit)(struct cgroup_subsys_state *css,
struct cgroup_subsys_state *old_css,
struct task_struct *task); struct task_struct *task);
void (*bind)(struct cgroup *root); void (*bind)(struct cgroup_subsys_state *root_css);
int subsys_id; int subsys_id;
int disabled; int disabled;
......
kernel/cgroup.c
浏览文件 @ eb95419b
...@@ -853,8 +853,11 @@ static void cgroup_free_fn(struct work_struct *work) ...@@ -853,8 +853,11 @@ static void cgroup_free_fn(struct work_struct *work)
/* /*
* Release the subsystem state objects. * Release the subsystem state objects.
*/ */
for_each_root_subsys(cgrp->root, ss) for_each_root_subsys(cgrp->root, ss) {
ss->css_free(cgrp); struct cgroup_subsys_state *css = cgrp->subsys[ss->subsys_id];
ss->css_free(css);
}
cgrp->root->number_of_cgroups--; cgrp->root->number_of_cgroups--;
mutex_unlock(&cgroup_mutex); mutex_unlock(&cgroup_mutex);
...@@ -1056,7 +1059,7 @@ static int rebind_subsystems(struct cgroupfs_root *root, ...@@ -1056,7 +1059,7 @@ static int rebind_subsystems(struct cgroupfs_root *root,
list_move(&ss->sibling, &root->subsys_list); list_move(&ss->sibling, &root->subsys_list);
ss->root = root; ss->root = root;
if (ss->bind) if (ss->bind)
ss->bind(cgrp); ss->bind(cgrp->subsys[i]);
/* refcount was already taken, and we're keeping it */ /* refcount was already taken, and we're keeping it */
root->subsys_mask |= bit; root->subsys_mask |= bit;
...@@ -1066,7 +1069,7 @@ static int rebind_subsystems(struct cgroupfs_root *root, ...@@ -1066,7 +1069,7 @@ static int rebind_subsystems(struct cgroupfs_root *root,
BUG_ON(cgrp->subsys[i]->cgroup != cgrp); BUG_ON(cgrp->subsys[i]->cgroup != cgrp);
if (ss->bind) if (ss->bind)
ss->bind(cgroup_dummy_top); ss->bind(cgroup_dummy_top->subsys[i]);
cgroup_dummy_top->subsys[i]->cgroup = cgroup_dummy_top; cgroup_dummy_top->subsys[i]->cgroup = cgroup_dummy_top;
cgrp->subsys[i] = NULL; cgrp->subsys[i] = NULL;
cgroup_subsys[i]->root = &cgroup_dummy_root; cgroup_subsys[i]->root = &cgroup_dummy_root;
...@@ -2049,8 +2052,10 @@ static int cgroup_attach_task(struct cgroup *cgrp, struct task_struct *tsk, ...@@ -2049,8 +2052,10 @@ static int cgroup_attach_task(struct cgroup *cgrp, struct task_struct *tsk,
* step 1: check that we can legitimately attach to the cgroup. * step 1: check that we can legitimately attach to the cgroup.
*/ */
for_each_root_subsys(root, ss) { for_each_root_subsys(root, ss) {
struct cgroup_subsys_state *css = cgrp->subsys[ss->subsys_id];
if (ss->can_attach) { if (ss->can_attach) {
retval = ss->can_attach(cgrp, &tset); retval = ss->can_attach(css, &tset);
if (retval) { if (retval) {
failed_ss = ss; failed_ss = ss;
goto out_cancel_attach; goto out_cancel_attach;
...@@ -2089,8 +2094,10 @@ static int cgroup_attach_task(struct cgroup *cgrp, struct task_struct *tsk, ...@@ -2089,8 +2094,10 @@ static int cgroup_attach_task(struct cgroup *cgrp, struct task_struct *tsk,
* step 4: do subsystem attach callbacks. * step 4: do subsystem attach callbacks.
*/ */
for_each_root_subsys(root, ss) { for_each_root_subsys(root, ss) {
struct cgroup_subsys_state *css = cgrp->subsys[ss->subsys_id];
if (ss->attach) if (ss->attach)
ss->attach(cgrp, &tset); ss->attach(css, &tset);
} }
/* /*
...@@ -2109,10 +2116,12 @@ static int cgroup_attach_task(struct cgroup *cgrp, struct task_struct *tsk, ...@@ -2109,10 +2116,12 @@ static int cgroup_attach_task(struct cgroup *cgrp, struct task_struct *tsk,
out_cancel_attach: out_cancel_attach:
if (retval) { if (retval) {
for_each_root_subsys(root, ss) { for_each_root_subsys(root, ss) {
struct cgroup_subsys_state *css = cgrp->subsys[ss->subsys_id];
if (ss == failed_ss) if (ss == failed_ss)
break; break;
if (ss->cancel_attach) if (ss->cancel_attach)
ss->cancel_attach(cgrp, &tset); ss->cancel_attach(css, &tset);
} }
} }
out_free_group_list: out_free_group_list:
...@@ -4206,14 +4215,15 @@ static void init_cgroup_css(struct cgroup_subsys_state *css, ...@@ -4206,14 +4215,15 @@ static void init_cgroup_css(struct cgroup_subsys_state *css,
/* invoke ->css_online() on a new CSS and mark it online if successful */ /* invoke ->css_online() on a new CSS and mark it online if successful */
static int online_css(struct cgroup_subsys *ss, struct cgroup *cgrp) static int online_css(struct cgroup_subsys *ss, struct cgroup *cgrp)
{ {
struct cgroup_subsys_state *css = cgrp->subsys[ss->subsys_id];
int ret = 0; int ret = 0;
lockdep_assert_held(&cgroup_mutex); lockdep_assert_held(&cgroup_mutex);
if (ss->css_online) if (ss->css_online)
ret = ss->css_online(cgrp); ret = ss->css_online(css);
if (!ret) if (!ret)
cgrp->subsys[ss->subsys_id]->flags |= CSS_ONLINE; css->flags |= CSS_ONLINE;
return ret; return ret;
} }
...@@ -4228,9 +4238,9 @@ static void offline_css(struct cgroup_subsys *ss, struct cgroup *cgrp) ...@@ -4228,9 +4238,9 @@ static void offline_css(struct cgroup_subsys *ss, struct cgroup *cgrp)
return; return;
if (ss->css_offline) if (ss->css_offline)
ss->css_offline(cgrp); ss->css_offline(css);
cgrp->subsys[ss->subsys_id]->flags &= ~CSS_ONLINE; css->flags &= ~CSS_ONLINE;
} }
/* /*
...@@ -4305,7 +4315,7 @@ static long cgroup_create(struct cgroup *parent, struct dentry *dentry, ...@@ -4305,7 +4315,7 @@ static long cgroup_create(struct cgroup *parent, struct dentry *dentry,
for_each_root_subsys(root, ss) { for_each_root_subsys(root, ss) {
struct cgroup_subsys_state *css; struct cgroup_subsys_state *css;
css = ss->css_alloc(cgrp); css = ss->css_alloc(parent->subsys[ss->subsys_id]);
if (IS_ERR(css)) { if (IS_ERR(css)) {
err = PTR_ERR(css); err = PTR_ERR(css);
goto err_free_all; goto err_free_all;
...@@ -4313,7 +4323,7 @@ static long cgroup_create(struct cgroup *parent, struct dentry *dentry, ...@@ -4313,7 +4323,7 @@ static long cgroup_create(struct cgroup *parent, struct dentry *dentry,
err = percpu_ref_init(&css->refcnt, css_release); err = percpu_ref_init(&css->refcnt, css_release);
if (err) { if (err) {
ss->css_free(cgrp); ss->css_free(css);
goto err_free_all; goto err_free_all;
} }
...@@ -4386,7 +4396,7 @@ static long cgroup_create(struct cgroup *parent, struct dentry *dentry, ...@@ -4386,7 +4396,7 @@ static long cgroup_create(struct cgroup *parent, struct dentry *dentry,
if (css) { if (css) {
percpu_ref_cancel_init(&css->refcnt); percpu_ref_cancel_init(&css->refcnt);
ss->css_free(cgrp); ss->css_free(css);
} }
} }
mutex_unlock(&cgroup_mutex); mutex_unlock(&cgroup_mutex);
...@@ -4641,7 +4651,7 @@ static void __init cgroup_init_subsys(struct cgroup_subsys *ss) ...@@ -4641,7 +4651,7 @@ static void __init cgroup_init_subsys(struct cgroup_subsys *ss)
/* Create the top cgroup state for this subsystem */ /* Create the top cgroup state for this subsystem */
list_add(&ss->sibling, &cgroup_dummy_root.subsys_list); list_add(&ss->sibling, &cgroup_dummy_root.subsys_list);
ss->root = &cgroup_dummy_root; ss->root = &cgroup_dummy_root;
css = ss->css_alloc(cgroup_dummy_top); css = ss->css_alloc(cgroup_dummy_top->subsys[ss->subsys_id]);
/* We don't handle early failures gracefully */ /* We don't handle early failures gracefully */
BUG_ON(IS_ERR(css)); BUG_ON(IS_ERR(css));
init_cgroup_css(css, ss, cgroup_dummy_top); init_cgroup_css(css, ss, cgroup_dummy_top);
...@@ -4720,7 +4730,7 @@ int __init_or_module cgroup_load_subsys(struct cgroup_subsys *ss) ...@@ -4720,7 +4730,7 @@ int __init_or_module cgroup_load_subsys(struct cgroup_subsys *ss)
* struct, so this can happen first (i.e. before the dummy root * struct, so this can happen first (i.e. before the dummy root
* attachment). * attachment).
*/ */
css = ss->css_alloc(cgroup_dummy_top); css = ss->css_alloc(cgroup_dummy_top->subsys[ss->subsys_id]);
if (IS_ERR(css)) { if (IS_ERR(css)) {
/* failure case - need to deassign the cgroup_subsys[] slot. */ /* failure case - need to deassign the cgroup_subsys[] slot. */
cgroup_subsys[ss->subsys_id] = NULL; cgroup_subsys[ss->subsys_id] = NULL;
...@@ -4836,7 +4846,7 @@ void cgroup_unload_subsys(struct cgroup_subsys *ss) ...@@ -4836,7 +4846,7 @@ void cgroup_unload_subsys(struct cgroup_subsys *ss)
* the cgrp->subsys pointer to find their state. note that this * the cgrp->subsys pointer to find their state. note that this
* also takes care of freeing the css_id. * also takes care of freeing the css_id.
*/ */
ss->css_free(cgroup_dummy_top); ss->css_free(cgroup_dummy_top->subsys[ss->subsys_id]);
cgroup_dummy_top->subsys[ss->subsys_id] = NULL; cgroup_dummy_top->subsys[ss->subsys_id] = NULL;
mutex_unlock(&cgroup_mutex); mutex_unlock(&cgroup_mutex);
...@@ -5192,10 +5202,10 @@ void cgroup_exit(struct task_struct *tsk, int run_callbacks) ...@@ -5192,10 +5202,10 @@ void cgroup_exit(struct task_struct *tsk, int run_callbacks)
*/ */
for_each_builtin_subsys(ss, i) { for_each_builtin_subsys(ss, i) {
if (ss->exit) { if (ss->exit) {
struct cgroup *old_cgrp = cset->subsys[i]->cgroup; struct cgroup_subsys_state *old_css = cset->subsys[i];
struct cgroup *cgrp = task_cgroup(tsk, i); struct cgroup_subsys_state *css = task_css(tsk, i);
ss->exit(cgrp, old_cgrp, tsk); ss->exit(css, old_css, tsk);
} }
} }
} }
...@@ -5529,7 +5539,8 @@ struct cgroup_subsys_state *cgroup_css_from_dir(struct file *f, int id) ...@@ -5529,7 +5539,8 @@ struct cgroup_subsys_state *cgroup_css_from_dir(struct file *f, int id)
} }
#ifdef CONFIG_CGROUP_DEBUG #ifdef CONFIG_CGROUP_DEBUG
static struct cgroup_subsys_state *debug_css_alloc(struct cgroup *cgrp) static struct cgroup_subsys_state *
debug_css_alloc(struct cgroup_subsys_state *parent_css)
{ {
struct cgroup_subsys_state *css = kzalloc(sizeof(*css), GFP_KERNEL); struct cgroup_subsys_state *css = kzalloc(sizeof(*css), GFP_KERNEL);
...@@ -5539,9 +5550,9 @@ static struct cgroup_subsys_state *debug_css_alloc(struct cgroup *cgrp) ...@@ -5539,9 +5550,9 @@ static struct cgroup_subsys_state *debug_css_alloc(struct cgroup *cgrp)
return css; return css;
} }
static void debug_css_free(struct cgroup *cgrp) static void debug_css_free(struct cgroup_subsys_state *css)
{ {
kfree(cgrp->subsys[debug_subsys_id]); kfree(css);
} }
static u64 debug_taskcount_read(struct cgroup *cgrp, struct cftype *cft) static u64 debug_taskcount_read(struct cgroup *cgrp, struct cftype *cft)
......
kernel/cgroup_freezer.c
浏览文件 @ eb95419b
...@@ -91,7 +91,8 @@ static const char *freezer_state_strs(unsigned int state) ...@@ -91,7 +91,8 @@ static const char *freezer_state_strs(unsigned int state)
struct cgroup_subsys freezer_subsys; struct cgroup_subsys freezer_subsys;
static struct cgroup_subsys_state *freezer_css_alloc(struct cgroup *cgroup) static struct cgroup_subsys_state *
freezer_css_alloc(struct cgroup_subsys_state *parent_css)
{ {
struct freezer *freezer; struct freezer *freezer;
...@@ -104,16 +105,16 @@ static struct cgroup_subsys_state *freezer_css_alloc(struct cgroup *cgroup) ...@@ -104,16 +105,16 @@ static struct cgroup_subsys_state *freezer_css_alloc(struct cgroup *cgroup)
} }
/** /**
* freezer_css_online - commit creation of a freezer cgroup * freezer_css_online - commit creation of a freezer css
* @cgroup: cgroup being created * @css: css being created
* *
* We're committing to creation of @cgroup. Mark it online and inherit * We're committing to creation of @css. Mark it online and inherit
* parent's freezing state while holding both parent's and our * parent's freezing state while holding both parent's and our
* freezer->lock. * freezer->lock.
*/ */
static int freezer_css_online(struct cgroup *cgroup) static int freezer_css_online(struct cgroup_subsys_state *css)
{ {
struct freezer *freezer = cgroup_freezer(cgroup); struct freezer *freezer = css_freezer(css);
struct freezer *parent = parent_freezer(freezer); struct freezer *parent = parent_freezer(freezer);
/* /*
...@@ -140,15 +141,15 @@ static int freezer_css_online(struct cgroup *cgroup) ...@@ -140,15 +141,15 @@ static int freezer_css_online(struct cgroup *cgroup)
} }
/** /**
* freezer_css_offline - initiate destruction of @cgroup * freezer_css_offline - initiate destruction of a freezer css
* @cgroup: cgroup being destroyed * @css: css being destroyed
* *
* @cgroup is going away. Mark it dead and decrement system_freezing_count * @css is going away. Mark it dead and decrement system_freezing_count if
* if it was holding one. * it was holding one.
*/ */
static void freezer_css_offline(struct cgroup *cgroup) static void freezer_css_offline(struct cgroup_subsys_state *css)
{ {
struct freezer *freezer = cgroup_freezer(cgroup); struct freezer *freezer = css_freezer(css);
spin_lock_irq(&freezer->lock); spin_lock_irq(&freezer->lock);
...@@ -160,9 +161,9 @@ static void freezer_css_offline(struct cgroup *cgroup) ...@@ -160,9 +161,9 @@ static void freezer_css_offline(struct cgroup *cgroup)
spin_unlock_irq(&freezer->lock); spin_unlock_irq(&freezer->lock);
} }
static void freezer_css_free(struct cgroup *cgroup) static void freezer_css_free(struct cgroup_subsys_state *css)
{ {
kfree(cgroup_freezer(cgroup)); kfree(css_freezer(css));
} }
/* /*
...@@ -174,25 +175,26 @@ static void freezer_css_free(struct cgroup *cgroup) ...@@ -174,25 +175,26 @@ static void freezer_css_free(struct cgroup *cgroup)
* @freezer->lock. freezer_attach() makes the new tasks conform to the * @freezer->lock. freezer_attach() makes the new tasks conform to the
* current state and all following state changes can see the new tasks. * current state and all following state changes can see the new tasks.
*/ */
static void freezer_attach(struct cgroup *new_cgrp, struct cgroup_taskset *tset) static void freezer_attach(struct cgroup_subsys_state *new_css,
struct cgroup_taskset *tset)
{ {
struct freezer *freezer = cgroup_freezer(new_cgrp); struct freezer *freezer = css_freezer(new_css);
struct task_struct *task; struct task_struct *task;
bool clear_frozen = false; bool clear_frozen = false;
spin_lock_irq(&freezer->lock); spin_lock_irq(&freezer->lock);
/* /*
* Make the new tasks conform to the current state of @new_cgrp. * Make the new tasks conform to the current state of @new_css.
* For simplicity, when migrating any task to a FROZEN cgroup, we * For simplicity, when migrating any task to a FROZEN cgroup, we
* revert it to FREEZING and let update_if_frozen() determine the * revert it to FREEZING and let update_if_frozen() determine the
* correct state later. * correct state later.
* *
* Tasks in @tset are on @new_cgrp but may not conform to its * Tasks in @tset are on @new_css but may not conform to its
* current state before executing the following - !frozen tasks may * current state before executing the following - !frozen tasks may
* be visible in a FROZEN cgroup and frozen tasks in a THAWED one. * be visible in a FROZEN cgroup and frozen tasks in a THAWED one.
*/ */
cgroup_taskset_for_each(task, new_cgrp, tset) { cgroup_taskset_for_each(task, new_css->cgroup, tset) {
if (!(freezer->state & CGROUP_FREEZING)) { if (!(freezer->state & CGROUP_FREEZING)) {
__thaw_task(task); __thaw_task(task);
} else { } else {
......
kernel/cpuset.c
浏览文件 @ eb95419b
...@@ -1455,9 +1455,10 @@ static int fmeter_getrate(struct fmeter *fmp) ...@@ -1455,9 +1455,10 @@ static int fmeter_getrate(struct fmeter *fmp)
} }
/* Called by cgroups to determine if a cpuset is usable; cpuset_mutex held */ /* Called by cgroups to determine if a cpuset is usable; cpuset_mutex held */
static int cpuset_can_attach(struct cgroup *cgrp, struct cgroup_taskset *tset) static int cpuset_can_attach(struct cgroup_subsys_state *css,
struct cgroup_taskset *tset)
{ {
struct cpuset *cs = cgroup_cs(cgrp); struct cpuset *cs = css_cs(css);
struct task_struct *task; struct task_struct *task;
int ret; int ret;
...@@ -1468,11 +1469,11 @@ static int cpuset_can_attach(struct cgroup *cgrp, struct cgroup_taskset *tset) ...@@ -1468,11 +1469,11 @@ static int cpuset_can_attach(struct cgroup *cgrp, struct cgroup_taskset *tset)
* flag is set. * flag is set.
*/ */
ret = -ENOSPC; ret = -ENOSPC;
if (!cgroup_sane_behavior(cgrp) && if (!cgroup_sane_behavior(css->cgroup) &&
(cpumask_empty(cs->cpus_allowed) || nodes_empty(cs->mems_allowed))) (cpumask_empty(cs->cpus_allowed) || nodes_empty(cs->mems_allowed)))
goto out_unlock; goto out_unlock;
cgroup_taskset_for_each(task, cgrp, tset) { cgroup_taskset_for_each(task, css->cgroup, tset) {
/* /*
* Kthreads which disallow setaffinity shouldn't be moved * Kthreads which disallow setaffinity shouldn't be moved
* to a new cpuset; we don't want to change their cpu * to a new cpuset; we don't want to change their cpu
...@@ -1501,11 +1502,11 @@ static int cpuset_can_attach(struct cgroup *cgrp, struct cgroup_taskset *tset) ...@@ -1501,11 +1502,11 @@ static int cpuset_can_attach(struct cgroup *cgrp, struct cgroup_taskset *tset)
return ret; return ret;
} }
static void cpuset_cancel_attach(struct cgroup *cgrp, static void cpuset_cancel_attach(struct cgroup_subsys_state *css,
struct cgroup_taskset *tset) struct cgroup_taskset *tset)
{ {
mutex_lock(&cpuset_mutex); mutex_lock(&cpuset_mutex);
cgroup_cs(cgrp)->attach_in_progress--; css_cs(css)->attach_in_progress--;
mutex_unlock(&cpuset_mutex); mutex_unlock(&cpuset_mutex);
} }
...@@ -1516,7 +1517,8 @@ static void cpuset_cancel_attach(struct cgroup *cgrp, ...@@ -1516,7 +1517,8 @@ static void cpuset_cancel_attach(struct cgroup *cgrp,
*/ */
static cpumask_var_t cpus_attach; static cpumask_var_t cpus_attach;
static void cpuset_attach(struct cgroup *cgrp, struct cgroup_taskset *tset) static void cpuset_attach(struct cgroup_subsys_state *css,
struct cgroup_taskset *tset)
{ {
/* static buf protected by cpuset_mutex */ /* static buf protected by cpuset_mutex */
static nodemask_t cpuset_attach_nodemask_to; static nodemask_t cpuset_attach_nodemask_to;
...@@ -1524,7 +1526,7 @@ static void cpuset_attach(struct cgroup *cgrp, struct cgroup_taskset *tset) ...@@ -1524,7 +1526,7 @@ static void cpuset_attach(struct cgroup *cgrp, struct cgroup_taskset *tset)
struct task_struct *task; struct task_struct *task;
struct task_struct *leader = cgroup_taskset_first(tset); struct task_struct *leader = cgroup_taskset_first(tset);
struct cgroup *oldcgrp = cgroup_taskset_cur_cgroup(tset); struct cgroup *oldcgrp = cgroup_taskset_cur_cgroup(tset);
struct cpuset *cs = cgroup_cs(cgrp); struct cpuset *cs = css_cs(css);
struct cpuset *oldcs = cgroup_cs(oldcgrp); struct cpuset *oldcs = cgroup_cs(oldcgrp);
struct cpuset *cpus_cs = effective_cpumask_cpuset(cs); struct cpuset *cpus_cs = effective_cpumask_cpuset(cs);
struct cpuset *mems_cs = effective_nodemask_cpuset(cs); struct cpuset *mems_cs = effective_nodemask_cpuset(cs);
...@@ -1539,7 +1541,7 @@ static void cpuset_attach(struct cgroup *cgrp, struct cgroup_taskset *tset) ...@@ -1539,7 +1541,7 @@ static void cpuset_attach(struct cgroup *cgrp, struct cgroup_taskset *tset)
guarantee_online_mems(mems_cs, &cpuset_attach_nodemask_to); guarantee_online_mems(mems_cs, &cpuset_attach_nodemask_to);
cgroup_taskset_for_each(task, cgrp, tset) { cgroup_taskset_for_each(task, css->cgroup, tset) {
/* /*
* can_attach beforehand should guarantee that this doesn't * can_attach beforehand should guarantee that this doesn't
* fail. TODO: have a better way to handle failure here * fail. TODO: have a better way to handle failure here
...@@ -1940,11 +1942,12 @@ static struct cftype files[] = { ...@@ -1940,11 +1942,12 @@ static struct cftype files[] = {
* cgrp: control group that the new cpuset will be part of * cgrp: control group that the new cpuset will be part of
*/ */
static struct cgroup_subsys_state *cpuset_css_alloc(struct cgroup *cgrp) static struct cgroup_subsys_state *
cpuset_css_alloc(struct cgroup_subsys_state *parent_css)
{ {
struct cpuset *cs; struct cpuset *cs;
if (!cgrp->parent) if (!parent_css)
return &top_cpuset.css; return &top_cpuset.css;
cs = kzalloc(sizeof(*cs), GFP_KERNEL); cs = kzalloc(sizeof(*cs), GFP_KERNEL);
...@@ -1964,9 +1967,9 @@ static struct cgroup_subsys_state *cpuset_css_alloc(struct cgroup *cgrp) ...@@ -1964,9 +1967,9 @@ static struct cgroup_subsys_state *cpuset_css_alloc(struct cgroup *cgrp)
return &cs->css; return &cs->css;
} }
static int cpuset_css_online(struct cgroup *cgrp) static int cpuset_css_online(struct cgroup_subsys_state *css)
{ {
struct cpuset *cs = cgroup_cs(cgrp); struct cpuset *cs = css_cs(css);
struct cpuset *parent = parent_cs(cs); struct cpuset *parent = parent_cs(cs);
struct cpuset *tmp_cs; struct cpuset *tmp_cs;
struct cgroup *pos_cgrp; struct cgroup *pos_cgrp;
...@@ -1984,7 +1987,7 @@ static int cpuset_css_online(struct cgroup *cgrp) ...@@ -1984,7 +1987,7 @@ static int cpuset_css_online(struct cgroup *cgrp)
number_of_cpusets++; number_of_cpusets++;
if (!test_bit(CGRP_CPUSET_CLONE_CHILDREN, &cgrp->flags)) if (!test_bit(CGRP_CPUSET_CLONE_CHILDREN, &css->cgroup->flags))
goto out_unlock; goto out_unlock;
/* /*
...@@ -2024,9 +2027,9 @@ static int cpuset_css_online(struct cgroup *cgrp) ...@@ -2024,9 +2027,9 @@ static int cpuset_css_online(struct cgroup *cgrp)
* will call rebuild_sched_domains_locked(). * will call rebuild_sched_domains_locked().
*/ */
static void cpuset_css_offline(struct cgroup *cgrp) static void cpuset_css_offline(struct cgroup_subsys_state *css)
{ {
struct cpuset *cs = cgroup_cs(cgrp); struct cpuset *cs = css_cs(css);
mutex_lock(&cpuset_mutex); mutex_lock(&cpuset_mutex);
...@@ -2039,9 +2042,9 @@ static void cpuset_css_offline(struct cgroup *cgrp) ...@@ -2039,9 +2042,9 @@ static void cpuset_css_offline(struct cgroup *cgrp)
mutex_unlock(&cpuset_mutex); mutex_unlock(&cpuset_mutex);
} }
static void cpuset_css_free(struct cgroup *cgrp) static void cpuset_css_free(struct cgroup_subsys_state *css)
{ {
struct cpuset *cs = cgroup_cs(cgrp); struct cpuset *cs = css_cs(css);
free_cpumask_var(cs->cpus_allowed); free_cpumask_var(cs->cpus_allowed);
kfree(cs); kfree(cs);
......
kernel/events/core.c
浏览文件 @ eb95419b
...@@ -7778,7 +7778,8 @@ static int __init perf_event_sysfs_init(void) ...@@ -7778,7 +7778,8 @@ static int __init perf_event_sysfs_init(void)
device_initcall(perf_event_sysfs_init); device_initcall(perf_event_sysfs_init);
#ifdef CONFIG_CGROUP_PERF #ifdef CONFIG_CGROUP_PERF
static struct cgroup_subsys_state *perf_cgroup_css_alloc(struct cgroup *cont) static struct cgroup_subsys_state *
perf_cgroup_css_alloc(struct cgroup_subsys_state *parent_css)
{ {
struct perf_cgroup *jc; struct perf_cgroup *jc;
...@@ -7795,11 +7796,10 @@ static struct cgroup_subsys_state *perf_cgroup_css_alloc(struct cgroup *cont) ...@@ -7795,11 +7796,10 @@ static struct cgroup_subsys_state *perf_cgroup_css_alloc(struct cgroup *cont)
return &jc->css; return &jc->css;
} }
static void perf_cgroup_css_free(struct cgroup *cont) static void perf_cgroup_css_free(struct cgroup_subsys_state *css)
{ {
struct perf_cgroup *jc; struct perf_cgroup *jc = container_of(css, struct perf_cgroup, css);
jc = container_of(cgroup_css(cont, perf_subsys_id),
struct perf_cgroup, css);
free_percpu(jc->info); free_percpu(jc->info);
kfree(jc); kfree(jc);
} }
...@@ -7811,15 +7811,17 @@ static int __perf_cgroup_move(void *info) ...@@ -7811,15 +7811,17 @@ static int __perf_cgroup_move(void *info)
return 0; return 0;
} }
static void perf_cgroup_attach(struct cgroup *cgrp, struct cgroup_taskset *tset) static void perf_cgroup_attach(struct cgroup_subsys_state *css,
struct cgroup_taskset *tset)
{ {
struct task_struct *task; struct task_struct *task;
cgroup_taskset_for_each(task, cgrp, tset) cgroup_taskset_for_each(task, css->cgroup, tset)
task_function_call(task, __perf_cgroup_move, task); task_function_call(task, __perf_cgroup_move, task);
} }
static void perf_cgroup_exit(struct cgroup *cgrp, struct cgroup *old_cgrp, static void perf_cgroup_exit(struct cgroup_subsys_state *css,
struct cgroup_subsys_state *old_css,
struct task_struct *task) struct task_struct *task)
{ {
/* /*
......
kernel/sched/core.c
浏览文件 @ eb95419b
...@@ -7094,16 +7094,17 @@ static inline struct task_group *cgroup_tg(struct cgroup *cgrp) ...@@ -7094,16 +7094,17 @@ static inline struct task_group *cgroup_tg(struct cgroup *cgrp)
return css_tg(cgroup_css(cgrp, cpu_cgroup_subsys_id)); return css_tg(cgroup_css(cgrp, cpu_cgroup_subsys_id));
} }
static struct cgroup_subsys_state *cpu_cgroup_css_alloc(struct cgroup *cgrp) static struct cgroup_subsys_state *
cpu_cgroup_css_alloc(struct cgroup_subsys_state *parent_css)
{ {
struct task_group *tg, *parent; struct task_group *parent = css_tg(parent_css);
struct task_group *tg;
if (!cgrp->parent) { if (!parent) {
/* This is early initialization for the top cgroup */ /* This is early initialization for the top cgroup */
return &root_task_group.css; return &root_task_group.css;
} }
parent = cgroup_tg(cgrp->parent);
tg = sched_create_group(parent); tg = sched_create_group(parent);
if (IS_ERR(tg)) if (IS_ERR(tg))
return ERR_PTR(-ENOMEM); return ERR_PTR(-ENOMEM);
...@@ -7111,38 +7112,38 @@ static struct cgroup_subsys_state *cpu_cgroup_css_alloc(struct cgroup *cgrp) ...@@ -7111,38 +7112,38 @@ static struct cgroup_subsys_state *cpu_cgroup_css_alloc(struct cgroup *cgrp)
return &tg->css; return &tg->css;
} }
static int cpu_cgroup_css_online(struct cgroup *cgrp) static int cpu_cgroup_css_online(struct cgroup_subsys_state *css)
{ {
struct task_group *tg = cgroup_tg(cgrp); struct task_group *tg = css_tg(css);
struct task_group *parent = css_tg(css_parent(&tg->css)); struct task_group *parent = css_tg(css_parent(css));
if (parent) if (parent)
sched_online_group(tg, parent); sched_online_group(tg, parent);
return 0; return 0;
} }
static void cpu_cgroup_css_free(struct cgroup *cgrp) static void cpu_cgroup_css_free(struct cgroup_subsys_state *css)
{ {
struct task_group *tg = cgroup_tg(cgrp); struct task_group *tg = css_tg(css);
sched_destroy_group(tg); sched_destroy_group(tg);
} }
static void cpu_cgroup_css_offline(struct cgroup *cgrp) static void cpu_cgroup_css_offline(struct cgroup_subsys_state *css)
{ {
struct task_group *tg = cgroup_tg(cgrp); struct task_group *tg = css_tg(css);
sched_offline_group(tg); sched_offline_group(tg);
} }
static int cpu_cgroup_can_attach(struct cgroup *cgrp, static int cpu_cgroup_can_attach(struct cgroup_subsys_state *css,
struct cgroup_taskset *tset) struct cgroup_taskset *tset)
{ {
struct task_struct *task; struct task_struct *task;
cgroup_taskset_for_each(task, cgrp, tset) { cgroup_taskset_for_each(task, css->cgroup, tset) {
#ifdef CONFIG_RT_GROUP_SCHED #ifdef CONFIG_RT_GROUP_SCHED
if (!sched_rt_can_attach(cgroup_tg(cgrp), task)) if (!sched_rt_can_attach(css_tg(css), task))
return -EINVAL; return -EINVAL;
#else #else
/* We don't support RT-tasks being in separate groups */ /* We don't support RT-tasks being in separate groups */
...@@ -7153,18 +7154,18 @@ static int cpu_cgroup_can_attach(struct cgroup *cgrp, ...@@ -7153,18 +7154,18 @@ static int cpu_cgroup_can_attach(struct cgroup *cgrp,
return 0; return 0;
} }
static void cpu_cgroup_attach(struct cgroup *cgrp, static void cpu_cgroup_attach(struct cgroup_subsys_state *css,
struct cgroup_taskset *tset) struct cgroup_taskset *tset)
{ {
struct task_struct *task; struct task_struct *task;
cgroup_taskset_for_each(task, cgrp, tset) cgroup_taskset_for_each(task, css->cgroup, tset)
sched_move_task(task); sched_move_task(task);
} }
static void static void cpu_cgroup_exit(struct cgroup_subsys_state *css,
cpu_cgroup_exit(struct cgroup *cgrp, struct cgroup *old_cgrp, struct cgroup_subsys_state *old_css,
struct task_struct *task) struct task_struct *task)
{ {
/* /*
* cgroup_exit() is called in the copy_process() failure path. * cgroup_exit() is called in the copy_process() failure path.
......
kernel/sched/cpuacct.c
浏览文件 @ eb95419b
...@@ -62,11 +62,12 @@ static struct cpuacct root_cpuacct = { ...@@ -62,11 +62,12 @@ static struct cpuacct root_cpuacct = {
}; };
/* create a new cpu accounting group */ /* create a new cpu accounting group */
static struct cgroup_subsys_state *cpuacct_css_alloc(struct cgroup *cgrp) static struct cgroup_subsys_state *
cpuacct_css_alloc(struct cgroup_subsys_state *parent_css)
{ {
struct cpuacct *ca; struct cpuacct *ca;
if (!cgrp->parent) if (!parent_css)
return &root_cpuacct.css; return &root_cpuacct.css;
ca = kzalloc(sizeof(*ca), GFP_KERNEL); ca = kzalloc(sizeof(*ca), GFP_KERNEL);
...@@ -92,9 +93,9 @@ static struct cgroup_subsys_state *cpuacct_css_alloc(struct cgroup *cgrp) ...@@ -92,9 +93,9 @@ static struct cgroup_subsys_state *cpuacct_css_alloc(struct cgroup *cgrp)
} }
/* destroy an existing cpu accounting group */ /* destroy an existing cpu accounting group */
static void cpuacct_css_free(struct cgroup *cgrp) static void cpuacct_css_free(struct cgroup_subsys_state *css)
{ {
struct cpuacct *ca = cgroup_ca(cgrp); struct cpuacct *ca = css_ca(css);
free_percpu(ca->cpustat); free_percpu(ca->cpustat);
free_percpu(ca->cpuusage); free_percpu(ca->cpuusage);
......
mm/hugetlb_cgroup.c
浏览文件 @ eb95419b
...@@ -73,19 +73,18 @@ static inline bool hugetlb_cgroup_have_usage(struct hugetlb_cgroup *h_cg) ...@@ -73,19 +73,18 @@ static inline bool hugetlb_cgroup_have_usage(struct hugetlb_cgroup *h_cg)
return false; return false;
} }
static struct cgroup_subsys_state *hugetlb_cgroup_css_alloc(struct cgroup *cgroup) static struct cgroup_subsys_state *
hugetlb_cgroup_css_alloc(struct cgroup_subsys_state *parent_css)
{ {
struct hugetlb_cgroup *parent_h_cgroup = hugetlb_cgroup_from_css(parent_css);
struct hugetlb_cgroup *h_cgroup;
int idx; int idx;
struct cgroup *parent_cgroup;
struct hugetlb_cgroup *h_cgroup, *parent_h_cgroup;
h_cgroup = kzalloc(sizeof(*h_cgroup), GFP_KERNEL); h_cgroup = kzalloc(sizeof(*h_cgroup), GFP_KERNEL);
if (!h_cgroup) if (!h_cgroup)
return ERR_PTR(-ENOMEM); return ERR_PTR(-ENOMEM);
parent_cgroup = cgroup->parent; if (parent_h_cgroup) {
if (parent_cgroup) {
parent_h_cgroup = hugetlb_cgroup_from_cgroup(parent_cgroup);
for (idx = 0; idx < HUGE_MAX_HSTATE; idx++) for (idx = 0; idx < HUGE_MAX_HSTATE; idx++)
res_counter_init(&h_cgroup->hugepage[idx], res_counter_init(&h_cgroup->hugepage[idx],
&parent_h_cgroup->hugepage[idx]); &parent_h_cgroup->hugepage[idx]);
...@@ -97,11 +96,11 @@ static struct cgroup_subsys_state *hugetlb_cgroup_css_alloc(struct cgroup *cgrou ...@@ -97,11 +96,11 @@ static struct cgroup_subsys_state *hugetlb_cgroup_css_alloc(struct cgroup *cgrou
return &h_cgroup->css; return &h_cgroup->css;
} }
static void hugetlb_cgroup_css_free(struct cgroup *cgroup) static void hugetlb_cgroup_css_free(struct cgroup_subsys_state *css)
{ {
struct hugetlb_cgroup *h_cgroup; struct hugetlb_cgroup *h_cgroup;
h_cgroup = hugetlb_cgroup_from_cgroup(cgroup); h_cgroup = hugetlb_cgroup_from_css(css);
kfree(h_cgroup); kfree(h_cgroup);
} }
...@@ -150,9 +149,9 @@ static void hugetlb_cgroup_move_parent(int idx, struct hugetlb_cgroup *h_cg, ...@@ -150,9 +149,9 @@ static void hugetlb_cgroup_move_parent(int idx, struct hugetlb_cgroup *h_cg,
* Force the hugetlb cgroup to empty the hugetlb resources by moving them to * Force the hugetlb cgroup to empty the hugetlb resources by moving them to
* the parent cgroup. * the parent cgroup.
*/ */
static void hugetlb_cgroup_css_offline(struct cgroup *cgroup) static void hugetlb_cgroup_css_offline(struct cgroup_subsys_state *css)
{ {
struct hugetlb_cgroup *h_cg = hugetlb_cgroup_from_cgroup(cgroup); struct hugetlb_cgroup *h_cg = hugetlb_cgroup_from_css(css);
struct hstate *h; struct hstate *h;
struct page *page; struct page *page;
int idx = 0; int idx = 0;
......
mm/memcontrol.c
浏览文件 @ eb95419b
...@@ -6211,7 +6211,7 @@ static void __init mem_cgroup_soft_limit_tree_init(void) ...@@ -6211,7 +6211,7 @@ static void __init mem_cgroup_soft_limit_tree_init(void)
} }
static struct cgroup_subsys_state * __ref static struct cgroup_subsys_state * __ref
mem_cgroup_css_alloc(struct cgroup *cont) mem_cgroup_css_alloc(struct cgroup_subsys_state *parent_css)
{ {
struct mem_cgroup *memcg; struct mem_cgroup *memcg;
long error = -ENOMEM; long error = -ENOMEM;
...@@ -6226,7 +6226,7 @@ mem_cgroup_css_alloc(struct cgroup *cont) ...@@ -6226,7 +6226,7 @@ mem_cgroup_css_alloc(struct cgroup *cont)
goto free_out; goto free_out;
/* root ? */ /* root ? */
if (cont->parent == NULL) { if (parent_css == NULL) {
root_mem_cgroup = memcg; root_mem_cgroup = memcg;
res_counter_init(&memcg->res, NULL); res_counter_init(&memcg->res, NULL);
res_counter_init(&memcg->memsw, NULL); res_counter_init(&memcg->memsw, NULL);
...@@ -6248,10 +6248,10 @@ mem_cgroup_css_alloc(struct cgroup *cont) ...@@ -6248,10 +6248,10 @@ mem_cgroup_css_alloc(struct cgroup *cont)
} }
static int static int
mem_cgroup_css_online(struct cgroup *cont) mem_cgroup_css_online(struct cgroup_subsys_state *css)
{ {
struct mem_cgroup *memcg = mem_cgroup_from_cont(cont); struct mem_cgroup *memcg = mem_cgroup_from_css(css);
struct mem_cgroup *parent = mem_cgroup_from_css(css_parent(&memcg->css)); struct mem_cgroup *parent = mem_cgroup_from_css(css_parent(css));
int error = 0; int error = 0;
if (!parent) if (!parent)
...@@ -6308,9 +6308,9 @@ static void mem_cgroup_invalidate_reclaim_iterators(struct mem_cgroup *memcg) ...@@ -6308,9 +6308,9 @@ static void mem_cgroup_invalidate_reclaim_iterators(struct mem_cgroup *memcg)
mem_cgroup_iter_invalidate(root_mem_cgroup); mem_cgroup_iter_invalidate(root_mem_cgroup);
} }
static void mem_cgroup_css_offline(struct cgroup *cont) static void mem_cgroup_css_offline(struct cgroup_subsys_state *css)
{ {
struct mem_cgroup *memcg = mem_cgroup_from_cont(cont); struct mem_cgroup *memcg = mem_cgroup_from_css(css);
kmem_cgroup_css_offline(memcg); kmem_cgroup_css_offline(memcg);
...@@ -6319,9 +6319,9 @@ static void mem_cgroup_css_offline(struct cgroup *cont) ...@@ -6319,9 +6319,9 @@ static void mem_cgroup_css_offline(struct cgroup *cont)
mem_cgroup_destroy_all_caches(memcg); mem_cgroup_destroy_all_caches(memcg);
} }
static void mem_cgroup_css_free(struct cgroup *cont) static void mem_cgroup_css_free(struct cgroup_subsys_state *css)
{ {
struct mem_cgroup *memcg = mem_cgroup_from_cont(cont); struct mem_cgroup *memcg = mem_cgroup_from_css(css);
memcg_destroy_kmem(memcg); memcg_destroy_kmem(memcg);
__mem_cgroup_free(memcg); __mem_cgroup_free(memcg);
...@@ -6691,12 +6691,12 @@ static void mem_cgroup_clear_mc(void) ...@@ -6691,12 +6691,12 @@ static void mem_cgroup_clear_mc(void)
mem_cgroup_end_move(from); mem_cgroup_end_move(from);
} }
static int mem_cgroup_can_attach(struct cgroup *cgroup, static int mem_cgroup_can_attach(struct cgroup_subsys_state *css,
struct cgroup_taskset *tset) struct cgroup_taskset *tset)
{ {
struct task_struct *p = cgroup_taskset_first(tset); struct task_struct *p = cgroup_taskset_first(tset);
int ret = 0; int ret = 0;
struct mem_cgroup *memcg = mem_cgroup_from_cont(cgroup); struct mem_cgroup *memcg = mem_cgroup_from_css(css);
unsigned long move_charge_at_immigrate; unsigned long move_charge_at_immigrate;
/* /*
...@@ -6738,7 +6738,7 @@ static int mem_cgroup_can_attach(struct cgroup *cgroup, ...@@ -6738,7 +6738,7 @@ static int mem_cgroup_can_attach(struct cgroup *cgroup,
return ret; return ret;
} }
static void mem_cgroup_cancel_attach(struct cgroup *cgroup, static void mem_cgroup_cancel_attach(struct cgroup_subsys_state *css,
struct cgroup_taskset *tset) struct cgroup_taskset *tset)
{ {
mem_cgroup_clear_mc(); mem_cgroup_clear_mc();
...@@ -6886,7 +6886,7 @@ static void mem_cgroup_move_charge(struct mm_struct *mm) ...@@ -6886,7 +6886,7 @@ static void mem_cgroup_move_charge(struct mm_struct *mm)
up_read(&mm->mmap_sem); up_read(&mm->mmap_sem);
} }
static void mem_cgroup_move_task(struct cgroup *cont, static void mem_cgroup_move_task(struct cgroup_subsys_state *css,
struct cgroup_taskset *tset) struct cgroup_taskset *tset)
{ {
struct task_struct *p = cgroup_taskset_first(tset); struct task_struct *p = cgroup_taskset_first(tset);
...@@ -6901,16 +6901,16 @@ static void mem_cgroup_move_task(struct cgroup *cont, ...@@ -6901,16 +6901,16 @@ static void mem_cgroup_move_task(struct cgroup *cont,
mem_cgroup_clear_mc(); mem_cgroup_clear_mc();
} }
#else /* !CONFIG_MMU */ #else /* !CONFIG_MMU */
static int mem_cgroup_can_attach(struct cgroup *cgroup, static int mem_cgroup_can_attach(struct cgroup_subsys_state *css,
struct cgroup_taskset *tset) struct cgroup_taskset *tset)
{ {
return 0; return 0;
} }
static void mem_cgroup_cancel_attach(struct cgroup *cgroup, static void mem_cgroup_cancel_attach(struct cgroup_subsys_state *css,
struct cgroup_taskset *tset) struct cgroup_taskset *tset)
{ {
} }
static void mem_cgroup_move_task(struct cgroup *cont, static void mem_cgroup_move_task(struct cgroup_subsys_state *css,
struct cgroup_taskset *tset) struct cgroup_taskset *tset)
{ {
} }
...@@ -6920,15 +6920,15 @@ static void mem_cgroup_move_task(struct cgroup *cont, ...@@ -6920,15 +6920,15 @@ static void mem_cgroup_move_task(struct cgroup *cont,
* Cgroup retains root cgroups across [un]mount cycles making it necessary * Cgroup retains root cgroups across [un]mount cycles making it necessary
* to verify sane_behavior flag on each mount attempt. * to verify sane_behavior flag on each mount attempt.
*/ */
static void mem_cgroup_bind(struct cgroup *root) static void mem_cgroup_bind(struct cgroup_subsys_state *root_css)
{ {
/* /*
* use_hierarchy is forced with sane_behavior. cgroup core * use_hierarchy is forced with sane_behavior. cgroup core
* guarantees that @root doesn't have any children, so turning it * guarantees that @root doesn't have any children, so turning it
* on for the root memcg is enough. * on for the root memcg is enough.
*/ */
if (cgroup_sane_behavior(root)) if (cgroup_sane_behavior(root_css->cgroup))
mem_cgroup_from_cont(root)->use_hierarchy = true; mem_cgroup_from_css(root_css)->use_hierarchy = true;
} }
struct cgroup_subsys mem_cgroup_subsys = { struct cgroup_subsys mem_cgroup_subsys = {
......
net/core/netprio_cgroup.c
浏览文件 @ eb95419b
...@@ -126,7 +126,8 @@ static int netprio_set_prio(struct cgroup_subsys_state *css, ...@@ -126,7 +126,8 @@ static int netprio_set_prio(struct cgroup_subsys_state *css,
return 0; return 0;
} }
static struct cgroup_subsys_state *cgrp_css_alloc(struct cgroup *cgrp) static struct cgroup_subsys_state *
cgrp_css_alloc(struct cgroup_subsys_state *parent_css)
{ {
struct cgroup_subsys_state *css; struct cgroup_subsys_state *css;
...@@ -137,16 +138,14 @@ static struct cgroup_subsys_state *cgrp_css_alloc(struct cgroup *cgrp) ...@@ -137,16 +138,14 @@ static struct cgroup_subsys_state *cgrp_css_alloc(struct cgroup *cgrp)
return css; return css;
} }
static int cgrp_css_online(struct cgroup *cgrp) static int cgrp_css_online(struct cgroup_subsys_state *css)
{ {
struct cgroup_subsys_state *css = cgroup_css(cgrp, net_prio_subsys_id); struct cgroup_subsys_state *parent_css = css_parent(css);
struct cgroup_subsys_state *parent_css;
struct net_device *dev; struct net_device *dev;
int ret = 0; int ret = 0;
if (!cgrp->parent) if (!parent_css)
return 0; return 0;
parent_css = cgroup_css(cgrp->parent, net_prio_subsys_id);
rtnl_lock(); rtnl_lock();
/* /*
...@@ -164,9 +163,9 @@ static int cgrp_css_online(struct cgroup *cgrp) ...@@ -164,9 +163,9 @@ static int cgrp_css_online(struct cgroup *cgrp)
return ret; return ret;
} }
static void cgrp_css_free(struct cgroup *cgrp) static void cgrp_css_free(struct cgroup_subsys_state *css)
{ {
kfree(cgroup_css(cgrp, net_prio_subsys_id)); kfree(css);
} }
static u64 read_prioidx(struct cgroup *cgrp, struct cftype *cft) static u64 read_prioidx(struct cgroup *cgrp, struct cftype *cft)
...@@ -221,12 +220,13 @@ static int update_netprio(const void *v, struct file *file, unsigned n) ...@@ -221,12 +220,13 @@ static int update_netprio(const void *v, struct file *file, unsigned n)
return 0; return 0;
} }
static void net_prio_attach(struct cgroup *cgrp, struct cgroup_taskset *tset) static void net_prio_attach(struct cgroup_subsys_state *css,
struct cgroup_taskset *tset)
{ {
struct task_struct *p; struct task_struct *p;
void *v; void *v;
cgroup_taskset_for_each(p, cgrp, tset) { cgroup_taskset_for_each(p, css->cgroup, tset) {
task_lock(p); task_lock(p);
v = (void *)(unsigned long)task_netprioidx(p); v = (void *)(unsigned long)task_netprioidx(p);
iterate_fd(p->files, 0, update_netprio, v); iterate_fd(p->files, 0, update_netprio, v);
......
net/sched/cls_cgroup.c
浏览文件 @ eb95419b
...@@ -38,7 +38,8 @@ static inline struct cgroup_cls_state *task_cls_state(struct task_struct *p) ...@@ -38,7 +38,8 @@ static inline struct cgroup_cls_state *task_cls_state(struct task_struct *p)
return css_cls_state(task_css(p, net_cls_subsys_id)); return css_cls_state(task_css(p, net_cls_subsys_id));
} }
static struct cgroup_subsys_state *cgrp_css_alloc(struct cgroup *cgrp) static struct cgroup_subsys_state *
cgrp_css_alloc(struct cgroup_subsys_state *parent_css)
{ {
struct cgroup_cls_state *cs; struct cgroup_cls_state *cs;
...@@ -48,19 +49,19 @@ static struct cgroup_subsys_state *cgrp_css_alloc(struct cgroup *cgrp) ...@@ -48,19 +49,19 @@ static struct cgroup_subsys_state *cgrp_css_alloc(struct cgroup *cgrp)
return &cs->css; return &cs->css;
} }
static int cgrp_css_online(struct cgroup *cgrp) static int cgrp_css_online(struct cgroup_subsys_state *css)
{ {
struct cgroup_cls_state *cs = cgrp_cls_state(cgrp); struct cgroup_cls_state *cs = css_cls_state(css);
struct cgroup_cls_state *parent = css_cls_state(css_parent(&cs->css)); struct cgroup_cls_state *parent = css_cls_state(css_parent(css));
if (parent) if (parent)
cs->classid = parent->classid; cs->classid = parent->classid;
return 0; return 0;
} }
static void cgrp_css_free(struct cgroup *cgrp) static void cgrp_css_free(struct cgroup_subsys_state *css)
{ {
kfree(cgrp_cls_state(cgrp)); kfree(css_cls_state(css));
} }
static int update_classid(const void *v, struct file *file, unsigned n) static int update_classid(const void *v, struct file *file, unsigned n)
...@@ -72,12 +73,13 @@ static int update_classid(const void *v, struct file *file, unsigned n) ...@@ -72,12 +73,13 @@ static int update_classid(const void *v, struct file *file, unsigned n)
return 0; return 0;
} }
static void cgrp_attach(struct cgroup *cgrp, struct cgroup_taskset *tset) static void cgrp_attach(struct cgroup_subsys_state *css,
struct cgroup_taskset *tset)
{ {
struct task_struct *p; struct task_struct *p;
void *v; void *v;
cgroup_taskset_for_each(p, cgrp, tset) { cgroup_taskset_for_each(p, css->cgroup, tset) {
task_lock(p); task_lock(p);
v = (void *)(unsigned long)task_cls_classid(p); v = (void *)(unsigned long)task_cls_classid(p);
iterate_fd(p->files, 0, update_classid, v); iterate_fd(p->files, 0, update_classid, v);
......
security/device_cgroup.c
浏览文件 @ eb95419b
...@@ -68,7 +68,7 @@ static inline struct dev_cgroup *task_devcgroup(struct task_struct *task) ...@@ -68,7 +68,7 @@ static inline struct dev_cgroup *task_devcgroup(struct task_struct *task)
struct cgroup_subsys devices_subsys; struct cgroup_subsys devices_subsys;
static int devcgroup_can_attach(struct cgroup *new_cgrp, static int devcgroup_can_attach(struct cgroup_subsys_state *new_css,
struct cgroup_taskset *set) struct cgroup_taskset *set)
{ {
struct task_struct *task = cgroup_taskset_first(set); struct task_struct *task = cgroup_taskset_first(set);
...@@ -193,13 +193,13 @@ static inline bool is_devcg_online(const struct dev_cgroup *devcg) ...@@ -193,13 +193,13 @@ static inline bool is_devcg_online(const struct dev_cgroup *devcg)
/** /**
* devcgroup_online - initializes devcgroup's behavior and exceptions based on * devcgroup_online - initializes devcgroup's behavior and exceptions based on
* parent's * parent's
* @cgroup: cgroup getting online * @css: css getting online
* returns 0 in case of success, error code otherwise * returns 0 in case of success, error code otherwise
*/ */
static int devcgroup_online(struct cgroup *cgroup) static int devcgroup_online(struct cgroup_subsys_state *css)
{ {
struct dev_cgroup *dev_cgroup = cgroup_to_devcgroup(cgroup); struct dev_cgroup *dev_cgroup = css_to_devcgroup(css);
struct dev_cgroup *parent_dev_cgroup = css_to_devcgroup(css_parent(&dev_cgroup->css)); struct dev_cgroup *parent_dev_cgroup = css_to_devcgroup(css_parent(css));
int ret = 0; int ret = 0;
mutex_lock(&devcgroup_mutex); mutex_lock(&devcgroup_mutex);
...@@ -217,9 +217,9 @@ static int devcgroup_online(struct cgroup *cgroup) ...@@ -217,9 +217,9 @@ static int devcgroup_online(struct cgroup *cgroup)
return ret; return ret;
} }
static void devcgroup_offline(struct cgroup *cgroup) static void devcgroup_offline(struct cgroup_subsys_state *css)
{ {
struct dev_cgroup *dev_cgroup = cgroup_to_devcgroup(cgroup); struct dev_cgroup *dev_cgroup = css_to_devcgroup(css);
mutex_lock(&devcgroup_mutex); mutex_lock(&devcgroup_mutex);
dev_cgroup->behavior = DEVCG_DEFAULT_NONE; dev_cgroup->behavior = DEVCG_DEFAULT_NONE;
...@@ -229,7 +229,8 @@ static void devcgroup_offline(struct cgroup *cgroup) ...@@ -229,7 +229,8 @@ static void devcgroup_offline(struct cgroup *cgroup)
/* /*
* called from kernel/cgroup.c with cgroup_lock() held. * called from kernel/cgroup.c with cgroup_lock() held.
*/ */
static struct cgroup_subsys_state *devcgroup_css_alloc(struct cgroup *cgroup) static struct cgroup_subsys_state *
devcgroup_css_alloc(struct cgroup_subsys_state *parent_css)
{ {
struct dev_cgroup *dev_cgroup; struct dev_cgroup *dev_cgroup;
...@@ -242,11 +243,10 @@ static struct cgroup_subsys_state *devcgroup_css_alloc(struct cgroup *cgroup) ...@@ -242,11 +243,10 @@ static struct cgroup_subsys_state *devcgroup_css_alloc(struct cgroup *cgroup)
return &dev_cgroup->css; return &dev_cgroup->css;
} }
static void devcgroup_css_free(struct cgroup *cgroup) static void devcgroup_css_free(struct cgroup_subsys_state *css)
{ {
struct dev_cgroup *dev_cgroup; struct dev_cgroup *dev_cgroup = css_to_devcgroup(css);
dev_cgroup = cgroup_to_devcgroup(cgroup);
__dev_exception_clean(dev_cgroup); __dev_exception_clean(dev_cgroup);
kfree(dev_cgroup); kfree(dev_cgroup);
} }
......
Markdown is supported
0% 或 .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑！
想要评论请 注册 或