diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-01-04 02:24:29 -0500 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2012-01-04 10:37:48 -0500 |
commit | 305f3c8b20ba1ca94829329acdbf22e689304dba (patch) | |
tree | fdf794766dfba0393324f3c79a6f642dda328042 /kernel/cgroup.c | |
parent | 7e3aa30ac8c904a706518b725c451bb486daaae9 (diff) |
cgroup: move assignement out of condition in cgroup_attach_proc()
Gcc complains about this: "kernel/cgroup.c:2179:4: warning: suggest
parentheses around assignment used as truth value [-Wparentheses]"
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'kernel/cgroup.c')
-rw-r--r-- | kernel/cgroup.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/kernel/cgroup.c b/kernel/cgroup.c index 548d8d4e86d0..bab5c17e7781 100644 --- a/kernel/cgroup.c +++ b/kernel/cgroup.c | |||
@@ -2175,9 +2175,12 @@ static int cgroup_attach_proc(struct cgroup *cgrp, struct task_struct *leader) | |||
2175 | oldcg = tc->task->cgroups; | 2175 | oldcg = tc->task->cgroups; |
2176 | 2176 | ||
2177 | /* if we don't already have it in the list get a new one */ | 2177 | /* if we don't already have it in the list get a new one */ |
2178 | if (!css_set_check_fetched(cgrp, tc->task, oldcg, &newcg_list)) | 2178 | if (!css_set_check_fetched(cgrp, tc->task, oldcg, |
2179 | if (retval = css_set_prefetch(cgrp, oldcg, &newcg_list)) | 2179 | &newcg_list)) { |
2180 | retval = css_set_prefetch(cgrp, oldcg, &newcg_list); | ||
2181 | if (retval) | ||
2180 | goto out_list_teardown; | 2182 | goto out_list_teardown; |
2183 | } | ||
2181 | } | 2184 | } |
2182 | 2185 | ||
2183 | /* | 2186 | /* |