diff options
| author | Shakeel Butt <shakeelb@google.com> | 2019-04-03 16:03:54 -0700 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2019-04-04 13:39:42 -0700 |
| commit | d6e486ee0ef2f99a4069d9186e53dac61b28cb3c (patch) | |
| tree | 7b5697d3acd6c362b74c756caedad007db576194 /kernel/cgroup | |
| parent | 145f47c7381d43c789cbad55d4dbfd28fc6c46a4 (diff) | |
| download | linux-d6e486ee0ef2f99a4069d9186e53dac61b28cb3c.tar.bz2 | |
cgroup: remove extra cgroup_migrate_finish() call
The callers of cgroup_migrate_prepare_dst() correctly call
cgroup_migrate_finish() for success and failure cases both. No need to
call it in cgroup_migrate_prepare_dst() in failure case.
Signed-off-by: Shakeel Butt <shakeelb@google.com>
Reviewed-by: Daniel Jordan <daniel.m.jordan@oracle.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'kernel/cgroup')
| -rw-r--r-- | kernel/cgroup/cgroup.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/kernel/cgroup/cgroup.c b/kernel/cgroup/cgroup.c index 3f2b4bde0f9c..f219c195a9a5 100644 --- a/kernel/cgroup/cgroup.c +++ b/kernel/cgroup/cgroup.c @@ -2602,7 +2602,7 @@ int cgroup_migrate_prepare_dst(struct cgroup_mgctx *mgctx) dst_cset = find_css_set(src_cset, src_cset->mg_dst_cgrp); if (!dst_cset) - goto err; + return -ENOMEM; WARN_ON_ONCE(src_cset->mg_dst_cset || dst_cset->mg_dst_cset); @@ -2634,9 +2634,6 @@ int cgroup_migrate_prepare_dst(struct cgroup_mgctx *mgctx) } return 0; -err: - cgroup_migrate_finish(mgctx); - return -ENOMEM; } /** |