Re: [PATCH v2] memcg: Add memory.pressure_level events

2013-02-21 Thread Minchan Kim
On Thu, Feb 21, 2013 at 10:55:52PM -0800, Anton Vorontsov wrote: > On Fri, Feb 22, 2013 at 08:56:08AM +0900, Minchan Kim wrote: > > [...] The my point is that you have a plan to support? Why I have a > > question is that you said your goal is to replace lowmemory killer > > In short: yes, of

Re: [PATCH v2] memcg: Add memory.pressure_level events

2013-02-21 Thread Anton Vorontsov
On Fri, Feb 22, 2013 at 08:56:08AM +0900, Minchan Kim wrote: > [...] The my point is that you have a plan to support? Why I have a > question is that you said your goal is to replace lowmemory killer In short: yes, of course, if the non-memcg interface will be in demand. > but android don't have

Re: [PATCH v2] memcg: Add memory.pressure_level events

2013-02-21 Thread Minchan Kim
Hi Anton, On Thu, Feb 21, 2013 at 03:04:26PM -0800, Anton Vorontsov wrote: > On Tue, Feb 19, 2013 at 04:21:28PM -0800, Tejun Heo wrote: > > On Tue, Feb 19, 2013 at 4:17 PM, Minchan Kim wrote: > > > Should we really enable memcg for just pressure notificaion in embedded > > > side? > > > I

Re: [PATCH v2] memcg: Add memory.pressure_level events

2013-02-21 Thread Anton Vorontsov
On Tue, Feb 19, 2013 at 04:21:28PM -0800, Tejun Heo wrote: > On Tue, Feb 19, 2013 at 4:17 PM, Minchan Kim wrote: > > Should we really enable memcg for just pressure notificaion in embedded > > side? > > I didn't check the size(cgroup + memcg) and performance penalty but I don't > > want > > to

Re: [PATCH v2] memcg: Add memory.pressure_level events

2013-02-21 Thread Anton Vorontsov
On Tue, Feb 19, 2013 at 04:21:28PM -0800, Tejun Heo wrote: On Tue, Feb 19, 2013 at 4:17 PM, Minchan Kim minc...@kernel.org wrote: Should we really enable memcg for just pressure notificaion in embedded side? I didn't check the size(cgroup + memcg) and performance penalty but I don't

Re: [PATCH v2] memcg: Add memory.pressure_level events

2013-02-21 Thread Minchan Kim
Hi Anton, On Thu, Feb 21, 2013 at 03:04:26PM -0800, Anton Vorontsov wrote: On Tue, Feb 19, 2013 at 04:21:28PM -0800, Tejun Heo wrote: On Tue, Feb 19, 2013 at 4:17 PM, Minchan Kim minc...@kernel.org wrote: Should we really enable memcg for just pressure notificaion in embedded side? I

Re: [PATCH v2] memcg: Add memory.pressure_level events

2013-02-21 Thread Anton Vorontsov
On Fri, Feb 22, 2013 at 08:56:08AM +0900, Minchan Kim wrote: [...] The my point is that you have a plan to support? Why I have a question is that you said your goal is to replace lowmemory killer In short: yes, of course, if the non-memcg interface will be in demand. but android don't have

Re: [PATCH v2] memcg: Add memory.pressure_level events

2013-02-21 Thread Minchan Kim
On Thu, Feb 21, 2013 at 10:55:52PM -0800, Anton Vorontsov wrote: On Fri, Feb 22, 2013 at 08:56:08AM +0900, Minchan Kim wrote: [...] The my point is that you have a plan to support? Why I have a question is that you said your goal is to replace lowmemory killer In short: yes, of course, if

Re: [PATCH v2] memcg: Add memory.pressure_level events

2013-02-19 Thread Tejun Heo
Hello, On Tue, Feb 19, 2013 at 4:17 PM, Minchan Kim wrote: > Should we really enable memcg for just pressure notificaion in embedded side? > I didn't check the size(cgroup + memcg) and performance penalty but I don't > want > to add unnecessary overhead if it is possible. > Do you have a plan

Re: [PATCH v2] memcg: Add memory.pressure_level events

2013-02-19 Thread Minchan Kim
Hi Anton, On Mon, Feb 18, 2013 at 08:40:12PM -0800, Anton Vorontsov wrote: > With this patch userland applications that want to maintain the > interactivity/memory allocation cost can use the pressure level > notifications. The levels are defined like this: > > The "low" level means that the

Re: [PATCH v2] memcg: Add memory.pressure_level events

2013-02-19 Thread Minchan Kim
Hi Anton, On Mon, Feb 18, 2013 at 08:40:12PM -0800, Anton Vorontsov wrote: With this patch userland applications that want to maintain the interactivity/memory allocation cost can use the pressure level notifications. The levels are defined like this: The low level means that the system is

Re: [PATCH v2] memcg: Add memory.pressure_level events

2013-02-19 Thread Tejun Heo
Hello, On Tue, Feb 19, 2013 at 4:17 PM, Minchan Kim minc...@kernel.org wrote: Should we really enable memcg for just pressure notificaion in embedded side? I didn't check the size(cgroup + memcg) and performance penalty but I don't want to add unnecessary overhead if it is possible. Do you

[PATCH v2] memcg: Add memory.pressure_level events

2013-02-18 Thread Anton Vorontsov
With this patch userland applications that want to maintain the interactivity/memory allocation cost can use the pressure level notifications. The levels are defined like this: The "low" level means that the system is reclaiming memory for new allocations. Monitoring this reclaiming activity

[PATCH v2] memcg: Add memory.pressure_level events

2013-02-18 Thread Anton Vorontsov
With this patch userland applications that want to maintain the interactivity/memory allocation cost can use the pressure level notifications. The levels are defined like this: The low level means that the system is reclaiming memory for new allocations. Monitoring this reclaiming activity might