Re: [PATCH] oom: create a resource limit for oom_adj

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: David Rientjes
Date: Thursday, November 11, 2010 - 1:57 pm

On Thu, 11 Nov 2010, Mandeep Singh Baines wrote:


Hmm, at first glance that seems potentially dangerous if the current tab 
generates a burt of memory allocations and it ends up killing all other 
tabs before finally targeting the culprit whereas currently the heuristic 
should do a good job of finding this problematic tab and killing it 
instantly.

Perhaps that can't happen and it probably doesn't even matter: 
oom_score_adj allows users to determine which process to kill regardless 
of the underlying reason.


So as more and more tabs get used, the least recently used tab gets its 
oom_score_adj raised higher and higher until it is reused itself and then 
it gets reset back to 0 for the current tab?

Is there a reason you don't want to give the underlying browser session 
process CAP_SYS_RESOURCE?  Will it not be enforcing resource limits to 
ensure tabs don't deplete all memory when certain sites are opened?  Are 
you concerned that it may deplete all memory itself (for which case you 
could raise its own oom_score_adj, which is a proportion of available 
memory so you can define where that point of depletiong is)?
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] oom: create a resource limit for oom_adj, Mandeep Singh Baines, (Wed Nov 10, 9:35 pm)
Re: [PATCH] oom: create a resource limit for oom_adj, David Rientjes, (Thu Nov 11, 12:35 am)
Re: [PATCH] oom: create a resource limit for oom_adj, Mandeep Singh Baines, (Thu Nov 11, 11:30 am)
Re: [PATCH] oom: create a resource limit for oom_adj, David Rientjes, (Thu Nov 11, 1:57 pm)
Re: [PATCH] oom: create a resource limit for oom_adj, Mandeep Singh Baines, (Thu Nov 11, 3:25 pm)
Re: [PATCH] oom: create a resource limit for oom_adj, David Rientjes, (Thu Nov 11, 4:19 pm)
Re: [PATCH] oom: create a resource limit for oom_adj, Mandeep Singh Baines, (Thu Nov 11, 4:56 pm)
[PATCH] oom: allow a non-CAP_SYS_RESOURCE proces to oom_sc ..., Mandeep Singh Baines, (Fri Nov 12, 5:46 pm)
Re: [PATCH] oom: create a resource limit for oom_adj, KOSAKI Motohiro, (Sat Nov 13, 10:07 pm)
[PATCH v2] oom: allow a non-CAP_SYS_RESOURCE proces to oom ..., Mandeep Singh Baines, (Mon Nov 15, 3:01 pm)
[PATCH v3] oom: allow a non-CAP_SYS_RESOURCE proces to oom ..., Mandeep Singh Baines, (Mon Nov 15, 5:03 pm)