On 2013/12/9 20:38, Wei Yongjun wrote:
> From: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx>
>
> Add the missing unlock before return from function cgroup_load_subsys()
> in the error handling case.
>
> Signed-off-by: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx>
Acked-by: Li Zefan <lizefan@xxxxxxxxxx>
but it conflicts with Vladimir's bug fix.
> ---
> kernel/cgroup.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/kernel/cgroup.c b/kernel/cgroup.c
> index fb1193b..402f7aa 100644
> --- a/kernel/cgroup.c
> +++ b/kernel/cgroup.c
> @@ -4618,6 +4618,7 @@ int __init_or_module cgroup_load_subsys(struct cgroup_subsys *ss)
> if (IS_ERR(css)) {
> /* failure case - need to deassign the cgroup_subsys[] slot. */
> cgroup_subsys[ss->subsys_id] = NULL;
> + mutex_unlock(&cgroup_root_mutex);
> mutex_unlock(&cgroup_mutex);
> return PTR_ERR(css);
> }
>
>
_______________________________________________
Containers mailing list
Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linuxfoundation.org/mailman/listinfo/containers
