Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-28 Thread Joonsoo Kim
On Wed, Aug 27, 2014 at 04:28:19PM +0900, Minchan Kim wrote: > On Wed, Aug 27, 2014 at 02:04:38PM +0900, Joonsoo Kim wrote: > > On Wed, Aug 27, 2014 at 11:51:32AM +0900, Minchan Kim wrote: > > > Hey Joonsoo, > > > > > > On Wed, Aug 27, 2014 at 10:26:11AM +0900, Joonsoo Kim wrote: > > > > Hello, Mi

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-27 Thread Minchan Kim
On Wed, Aug 27, 2014 at 03:04:32PM -0400, Dan Streetman wrote: > On Wed, Aug 27, 2014 at 12:59 PM, David Horner wrote: > > On Wed, Aug 27, 2014 at 12:29 PM, Dan Streetman wrote: > >> On Wed, Aug 27, 2014 at 11:35 AM, David Horner wrote: > >>> On Wed, Aug 27, 2014 at 11:14 AM, Dan Streetman wrot

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-27 Thread Minchan Kim
On Wed, Aug 27, 2014 at 12:29:22PM -0400, Dan Streetman wrote: > On Wed, Aug 27, 2014 at 11:35 AM, David Horner wrote: > > On Wed, Aug 27, 2014 at 11:14 AM, Dan Streetman wrote: > >> On Wed, Aug 27, 2014 at 10:44 AM, David Horner wrote: > >>> On Wed, Aug 27, 2014 at 10:03 AM, Dan Streetman wrot

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-27 Thread Minchan Kim
Hello, On Wed, Aug 27, 2014 at 10:03:45AM -0400, Dan Streetman wrote: > On Tue, Aug 26, 2014 at 10:51 PM, Minchan Kim wrote: > > Hey Joonsoo, > > > > On Wed, Aug 27, 2014 at 10:26:11AM +0900, Joonsoo Kim wrote: > >> Hello, Minchan and David. > >> > >> On Tue, Aug 26, 2014 at 08:22:29AM -0400, Dav

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-27 Thread Dan Streetman
On Wed, Aug 27, 2014 at 12:59 PM, David Horner wrote: > On Wed, Aug 27, 2014 at 12:29 PM, Dan Streetman wrote: >> On Wed, Aug 27, 2014 at 11:35 AM, David Horner wrote: >>> On Wed, Aug 27, 2014 at 11:14 AM, Dan Streetman wrote: On Wed, Aug 27, 2014 at 10:44 AM, David Horner wrote: > On

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-27 Thread David Horner
On Wed, Aug 27, 2014 at 12:29 PM, Dan Streetman wrote: > On Wed, Aug 27, 2014 at 11:35 AM, David Horner wrote: >> On Wed, Aug 27, 2014 at 11:14 AM, Dan Streetman wrote: >>> On Wed, Aug 27, 2014 at 10:44 AM, David Horner wrote: On Wed, Aug 27, 2014 at 10:03 AM, Dan Streetman wrote: > O

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-27 Thread Dan Streetman
On Wed, Aug 27, 2014 at 11:35 AM, David Horner wrote: > On Wed, Aug 27, 2014 at 11:14 AM, Dan Streetman wrote: >> On Wed, Aug 27, 2014 at 10:44 AM, David Horner wrote: >>> On Wed, Aug 27, 2014 at 10:03 AM, Dan Streetman wrote: On Tue, Aug 26, 2014 at 10:51 PM, Minchan Kim wrote: > Hey

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-27 Thread David Horner
On Wed, Aug 27, 2014 at 11:14 AM, Dan Streetman wrote: > On Wed, Aug 27, 2014 at 10:44 AM, David Horner wrote: >> On Wed, Aug 27, 2014 at 10:03 AM, Dan Streetman wrote: >>> On Tue, Aug 26, 2014 at 10:51 PM, Minchan Kim wrote: Hey Joonsoo, On Wed, Aug 27, 2014 at 10:26:11AM +0900,

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-27 Thread Dan Streetman
On Wed, Aug 27, 2014 at 10:44 AM, David Horner wrote: > On Wed, Aug 27, 2014 at 10:03 AM, Dan Streetman wrote: >> On Tue, Aug 26, 2014 at 10:51 PM, Minchan Kim wrote: >>> Hey Joonsoo, >>> >>> On Wed, Aug 27, 2014 at 10:26:11AM +0900, Joonsoo Kim wrote: Hello, Minchan and David. On

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-27 Thread David Horner
On Wed, Aug 27, 2014 at 10:03 AM, Dan Streetman wrote: > On Tue, Aug 26, 2014 at 10:51 PM, Minchan Kim wrote: >> Hey Joonsoo, >> >> On Wed, Aug 27, 2014 at 10:26:11AM +0900, Joonsoo Kim wrote: >>> Hello, Minchan and David. >>> >>> On Tue, Aug 26, 2014 at 08:22:29AM -0400, David Horner wrote: >>>

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-27 Thread Dan Streetman
On Tue, Aug 26, 2014 at 10:51 PM, Minchan Kim wrote: > Hey Joonsoo, > > On Wed, Aug 27, 2014 at 10:26:11AM +0900, Joonsoo Kim wrote: >> Hello, Minchan and David. >> >> On Tue, Aug 26, 2014 at 08:22:29AM -0400, David Horner wrote: >> > On Tue, Aug 26, 2014 at 3:55 AM, Minchan Kim wrote: >> > > Hey

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-27 Thread Minchan Kim
On Wed, Aug 27, 2014 at 02:04:38PM +0900, Joonsoo Kim wrote: > On Wed, Aug 27, 2014 at 11:51:32AM +0900, Minchan Kim wrote: > > Hey Joonsoo, > > > > On Wed, Aug 27, 2014 at 10:26:11AM +0900, Joonsoo Kim wrote: > > > Hello, Minchan and David. > > > > > > On Tue, Aug 26, 2014 at 08:22:29AM -0400, D

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-26 Thread Joonsoo Kim
On Wed, Aug 27, 2014 at 11:51:32AM +0900, Minchan Kim wrote: > Hey Joonsoo, > > On Wed, Aug 27, 2014 at 10:26:11AM +0900, Joonsoo Kim wrote: > > Hello, Minchan and David. > > > > On Tue, Aug 26, 2014 at 08:22:29AM -0400, David Horner wrote: > > > On Tue, Aug 26, 2014 at 3:55 AM, Minchan Kim wrot

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-26 Thread Minchan Kim
Hey Joonsoo, On Wed, Aug 27, 2014 at 10:26:11AM +0900, Joonsoo Kim wrote: > Hello, Minchan and David. > > On Tue, Aug 26, 2014 at 08:22:29AM -0400, David Horner wrote: > > On Tue, Aug 26, 2014 at 3:55 AM, Minchan Kim wrote: > > > Hey Joonsoo, > > > > > > On Tue, Aug 26, 2014 at 04:37:30PM +0900,

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-26 Thread Joonsoo Kim
Hello, Minchan and David. On Tue, Aug 26, 2014 at 08:22:29AM -0400, David Horner wrote: > On Tue, Aug 26, 2014 at 3:55 AM, Minchan Kim wrote: > > Hey Joonsoo, > > > > On Tue, Aug 26, 2014 at 04:37:30PM +0900, Joonsoo Kim wrote: > >> On Mon, Aug 25, 2014 at 09:05:55AM +0900, Minchan Kim wrote: > >

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-26 Thread David Horner
On Tue, Aug 26, 2014 at 3:55 AM, Minchan Kim wrote: > Hey Joonsoo, > > On Tue, Aug 26, 2014 at 04:37:30PM +0900, Joonsoo Kim wrote: >> On Mon, Aug 25, 2014 at 09:05:55AM +0900, Minchan Kim wrote: >> > @@ -513,6 +540,14 @@ static int zram_bvec_write(struct zram *zram, struct >> > bio_vec *bvec, u3

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-26 Thread Minchan Kim
Hey Joonsoo, On Tue, Aug 26, 2014 at 04:37:30PM +0900, Joonsoo Kim wrote: > On Mon, Aug 25, 2014 at 09:05:55AM +0900, Minchan Kim wrote: > > @@ -513,6 +540,14 @@ static int zram_bvec_write(struct zram *zram, struct > > bio_vec *bvec, u32 index, > > ret = -ENOMEM; > > goto

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-26 Thread Joonsoo Kim
On Mon, Aug 25, 2014 at 09:05:55AM +0900, Minchan Kim wrote: > @@ -513,6 +540,14 @@ static int zram_bvec_write(struct zram *zram, struct > bio_vec *bvec, u32 index, > ret = -ENOMEM; > goto out; > } > + > + if (zram->limit_pages && > + zs_get_total_

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-25 Thread Minchan Kim
Hey Sergey, On Mon, Aug 25, 2014 at 08:09:27PM +0900, Sergey Senozhatsky wrote: > On (08/25/14 09:05), Minchan Kim wrote: > > Since zram has no control feature to limit memory usage, > > it makes hard to manage system memrory. > > > > This patch adds new knob "mem_limit" via sysfs to set up the >

Re: [PATCH v5 3/4] zram: zram memory size limitation

2014-08-25 Thread Sergey Senozhatsky
On (08/25/14 09:05), Minchan Kim wrote: > Since zram has no control feature to limit memory usage, > it makes hard to manage system memrory. > > This patch adds new knob "mem_limit" via sysfs to set up the > a limit so that zram could fail allocation once it reaches > the limit. > > In addition,