Re: [RFC 00/18] Present useful limits to user

[Konstantin Khlebnikov]

On Wed, Jun 15, 2016 at 5:47 PM, Austin S. Hemmelgarn
<ahferroin7@xxxxxxxxx> wrote:
> On 2016-06-14 15:03, Konstantin Khlebnikov wrote:
>>
>> I don't like the idea of this patchset.
>>
>> All limitations are context dependent and that context changes rapidly.
>> You'll never dump enough information for predicting future errors or
>> investigating reson of errors in past. You could try to reproduce all
>> kernel logic but model always will be aproximate.
>
> It's still better than what we have now, and there is one particular use for
> the cgroup stuff that I find intriguing, you can create a cgroup, populate
> it, set no limits, and then run a simulated workload against it and see how
> it reacts.  This in general will probably provide a better starting point
> for what to actually set the limits to than just making an arbitrary guess.
> Certain applications in particular come to mind which will just hang when
> they can't start a new thread or process (Dropbox is particularly guilty of
> this).  In such cases, setting the limit too low doesn't result in a crash,
> it results in the program just not appearing to work yet still running
> otherwise normally.
>
> In general, I could see the rlimit stuff being in the same situation, it's
> not for figuring out why something failed (good software will tell you
> somewhere), but figuring out limits so it doesn't fail but still is
> reasonably contained.  A lot of things that seem at face value like they
> shouldn't need specific exceptions to limits do.  Most normal users probably
> wouldn't guess that acpid needs a RLIMIT_NPROC count of at least 4 or more
> to work with the default rules.  Similarly, there's probably not many normal
> users who know that the Dropbox daemon spawns an insanely large thread pool
> and preallocates significant amounts of memory and will just hang if either
> of these fail.  By having a way to get running max counts of resource usage,
> it makes it easier for people to know what the minimum limit they need to
> put on something is.

Rlimits work only if resource usage could be estimated apriori.
They allows app limit itself to prevent failures is something goes wrong.

Rlimits are useless for controlling resource destribition: just use
cgroups for that.

>>
>> If you want to track origin of failures in user space applications when it
>> hits
>> some limit you should track errors. For example rlimits and other
>> limitation
>> subsystems could provide resonable amount of tracepoints which could
>> tell what exactly happened before error. If you need highwater of some
>> values you could track it in userspace, or maybe tracing subsystem could
>> provide postpocessing for tracepoint parameters. Anyway, systemtap and
>> other monsters can do this right now.
>
> Userspace tracking of some things just isn't practical.  Take RLIMIT_NPROC
> for example.  There's not really any reliable way to track this from
> userspace without modifying the process which is being tracked, which is not
> a user friendly way of doing things, and in some cases is functionally
> impossible for an end user to do.

You cannot get reliable upper bound for nr-proc from black box observations.
Highwater mark is very racy - tiny timing shifts can change it drammaticaly.

>
>>
>> On Mon, Jun 13, 2016 at 10:44 PM, Topi Miettinen <toiwoton@xxxxxxxxx>
>> wrote:
>>>
>>> Hello,
>>>
>>> There are many basic ways to control processes, including capabilities,
>>> cgroups and resource limits. However, there are far fewer ways to find
>>> out
>>> useful values for the limits, except blind trial and error.
>>>
>>> This patch series attempts to fix that by giving at least a nice starting
>>> point from the actual maximum values. I looked where each limit is
>>> checked
>>> and added a call to limit bump nearby.
>>>
>>>
>>> Capabilities
>>> [RFC 01/18] capabilities: track actually used capabilities
>>>
>>> Currently, there is no way to know which capabilities are actually used.
>>> Even
>>> the source code is only implicit, in-depth knowledge of each capability
>>> must
>>> be used when analyzing a program to judge which capabilities the program
>>> will
>>> exercise.
>>>
>>> Cgroups
>>> [RFC 02/18] cgroup_pids: track maximum pids
>>> [RFC 03/18] memcontrol: present maximum used memory also for
>>> [RFC 04/18] device_cgroup: track and present accessed devices
>>>
>>> For tasks and memory cgroup limits the situation is somewhat better as
>>> the
>>> current tasks and memory status can be easily seen with ps(1). However,
>>> any
>>> transient tasks or temporary higher memory use might slip from the view.
>>> Device use may be seen with advanced MAC tools, like TOMOYO, but there is
>>> no
>>> universal method. Program sources typically give no useful indication
>>> about
>>> memory use or how many tasks there could be.
>>>
>>> Resource limits
>>> [RFC 05/18] limits: track and present RLIMIT_NOFILE actual max
>>> [RFC 06/18] limits: present RLIMIT_CPU and RLIMIT_RTTIMER current
>>> [RFC 07/18] limits: track RLIMIT_FSIZE actual max
>>> [RFC 08/18] limits: track RLIMIT_DATA actual max
>>> [RFC 09/18] limits: track RLIMIT_CORE actual max
>>> [RFC 10/18] limits: track RLIMIT_STACK actual max
>>> [RFC 11/18] limits: track and present RLIMIT_NPROC actual max
>>> [RFC 12/18] limits: track RLIMIT_MEMLOCK actual max
>>> [RFC 13/18] limits: track RLIMIT_AS actual max
>>> [RFC 14/18] limits: track RLIMIT_SIGPENDING actual max
>>> [RFC 15/18] limits: track RLIMIT_MSGQUEUE actual max
>>> [RFC 16/18] limits: track RLIMIT_NICE actual max
>>> [RFC 17/18] limits: track RLIMIT_RTPRIO actual max
>>> [RFC 18/18] proc: present VM_LOCKED memory in /proc/self/maps
>>>
>>> Current number of files and current VM usage (data pages, address space
>>> size)
>>> could be calculated from available /proc files. Again, any temporarily
>>> higher
>>> values could be easily missed. For many limits, there is no way to see
>>> what
>>> is the current situation and source code is mostly useless.
>>>
>>> As a side note, the resouce limits seem to be in bad shape. For example,
>>> RLIMIT_MEMLOCK is used incoherently and I think VM statistics can miss
>>> some changes. Adding RLIMIT_CODE could be useful.
>>>
>>> The current maximum values for the resource limits are now shown in
>>> /proc/task/limits. If this is deemed too confusing for the existing
>>> programs which rely on the exact format, I can change that to a new file.
>>>
>>>
>>> Finally, the patches work in my testing but I have probably missed finer
>>> lock/RCU details.
>>>
>>> -Topi
>>>
>