Messages in this thread | ![/](/images/icornerl.gif) | | Date | Thu, 03 Sep 2009 15:47:06 +0200 | From | Jiri Slaby <> | Subject | Re: [PATCH 2/2] core: allow setrlimit to non-current tasks |
| |
On 09/02/2009 11:51 PM, Oleg Nesterov wrote: > On 09/02, Jiri Slaby wrote: >> I can't think of anything else than doing all the checks and updates >> under alloc_lock, introducing coarse grained static mutex in setrlimit >> to protect it, > > Oh, please don't ;) > > Or I missed your point? > > > But if you mean this series, then yes, I agree.
Yes, I meant those. But I don't know what do you agree with :).
> We should try to do something > to ensure that at least rlim_max can be always lowered when admin writes to > /proc/pid/limits.
Yes, that's what I asked about when I wrote the three options which I was able to think of above. So any other ideas about how to elegantly protect against sys_setrlimit vs. admin+/proc/*/limits race?
Thanks a heap.
| ![\](/images/icornerr.gif) |