Re: [PATCH V3] nommu: add anonymous page memcg accounting

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Steven J. Magnani
Date: Friday, October 22, 2010 - 6:26 am

On Fri, 2010-10-22 at 12:20 +0900, KAMEZAWA Hiroyuki wrote:

No - for what I need (simple sandboxing) just running my 'problem'
process in a memory cgroup is sufficient. I might even be able to get
away with oom_kill_allocating_task and no cgroup, but since that would
allow dosfsck to run the system completely out of memory there's no
guarantee that it would be the one that pushes the system over the edge.

What do you mean by "NOMMU memory limit"? (Is there some other way to
achieve the same functionality?)

I looked into David's initial suggestion of using ulimit to create a
sandbox but it seems that nommu.c doesn't respect RLIMIT_AS. When I can
find some time I'll try to cook up a patch for that.

Also it seems that nommu.c doesn't ever decrement mm->total_vm, which if
I'm reading the code correctly (before the 2.6.36 OOM-killer rewrite)
could throw off badness calculations for processes that do lots of
malloc/free operations. In 2.6.36 it doesn't look to me like this would
have any ill effects.

Thanks for all the feedback. I fully agree that maintenance should be a
strong consideration when merging new code.

Regards,
------------------------------------------------------------------------
 Steven J. Magnani               "I claim this network for MARS!
 www.digidescorp.com              Earthling, return my space modulator!"

 #include <standard.disclaimer>


--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH V3] nommu: add anonymous page memcg accounting, Steven J. Magnani, (Thu Oct 21, 5:28 am)
Re: [PATCH V3] nommu: add anonymous page memcg accounting, KAMEZAWA Hiroyuki, (Thu Oct 21, 8:20 pm)
Re: [PATCH V3] nommu: add anonymous page memcg accounting, KAMEZAWA Hiroyuki, (Thu Oct 21, 9:34 pm)
Re: [PATCH V3] nommu: add anonymous page memcg accounting, Steven J. Magnani, (Fri Oct 22, 6:26 am)
Re: [PATCH V3] nommu: add anonymous page memcg accounting, KAMEZAWA Hiroyuki, (Sun Oct 24, 5:13 pm)