diff options
| author | Amit Pundir <amit.pundir@linaro.org> | 2016-10-29 13:25:35 +0800 |
|---|---|---|
| committer | Amit Pundir <amit.pundir@linaro.org> | 2016-10-29 13:54:22 +0800 |
| commit | f19f4f75b6a86abdb43df1884de3a9038c2bc01a (patch) | |
| tree | 00bb31bea2bbb8c8d3cf3580cab6b44b0deaf01f /include | |
| parent | 1c03eb64cf3cf44f46195700cbe168515c8955c8 (diff) | |
Revert "cgroup: Fix issues in allow_attach callback"
This reverts commit fdc5f72f024f98e9718d3f3ce8b5fa22984dedeb.
Part of Chromium commit series:
CHROMIUM: remove Android's cgroup generic permissions checks
The implementation is utterly broken, resulting in all processes being
allows to move tasks between sets (as long as they have access to the
"tasks" attribute), and upstream is heading towards checking only
capability anyway, so let's get rid of this code.
BUG=b:31790445,chromium:647994
TEST=Boot android container, examine logcat
Signed-off-by: Dmitry Torokhov <dtor@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/394967
Reviewed-by: Ricky Zhou <rickyz@chromium.org>
Reviewed-by: John Stultz <john.stultz@linaro.org>
Signed-off-by: Amit Pundir <amit.pundir@linaro.org>
Diffstat (limited to 'include')
| -rw-r--r-- | include/linux/cgroup.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/cgroup.h b/include/linux/cgroup.h index a198b82d881f..4245f753f5c3 100644 --- a/include/linux/cgroup.h +++ b/include/linux/cgroup.h @@ -564,7 +564,7 @@ static inline int cgroup_init(void) { return 0; } static inline int subsys_cgroup_allow_attach(struct cgroup_subsys_state *css, struct cgroup_taskset *tset) { - return -EINVAL; + return 0; } #endif /* !CONFIG_CGROUPS */ |