Hi Thank you for careful review. >> + struct task_cgroup *taskcg; >> + >> + if ((max_tasks > INT_MAX) || >> + (max_tasks < INT_MIN)) > > It should be < -1 I think. OK. I'll fix it at next post. >> + spin_lock(&taskcg->lock); >> + if (max_tasks < taskcg->nr_tasks) >> + return -EBUSY; > > Shouldn't you drop the lock here? you are right. Thanks. > How does this controller affect performance? Do you have some numbers? No. but I have plan to mesure it at nearly future. Thanks. _______________________________________________ Containers mailing list Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/containers