Messages in this thread | | | Date | Sun, 8 Mar 2015 23:39:46 -0400 | From | Tejun Heo <> | Subject | Re: [PATCH v4 2/2] cgroups: add a pids subsystem |
| |
On Sun, Mar 08, 2015 at 11:34:05PM -0400, Tejun Heo wrote: > > + for (p = pids; p; p = p->parent) { > > + long new; > > + > > + new = atomic_long_add_return(num, &p->counter); > > + > > + if (p->limit == PIDS_UNLIMITED) > > + continue; > > Huh? So, the counter stays out of sync if unlimited? What happens > when it gets set to something else later?
Oops, I misread the code, but why is PIDS_UNLIMITED a special case? Just make it a number which always makes the condition true?
Thanks.
-- tejun
| |