Re: [PATCH] zswap: update docs for runtime-changeable attributes

2015-08-24 Thread Vlastimil Babka
On 25.8.2015 6:22, Sergey Senozhatsky wrote: i'd argue that neither zbud nor zsmalloc are responsible for reacting to memory pressure, they just store the pages. It's zswap that has to limit its size, which it does with max_percent_pool. >>> >>> Yeah but it's zbud that tracks the ag

Re: [PATCH] zswap: update docs for runtime-changeable attributes

2015-08-24 Thread Sergey Senozhatsky
On (08/19/15 11:56), Dan Streetman wrote: [..] > > Ugh that's madness. Still, a documented madness is better than an > > undocumented one. > > heh, i'm not sure why it's madness, the alternative of > uncompressing/recompressing all pages into the new zpool and/or with > the new compressor seems m

Re: [PATCH] zswap: update docs for runtime-changeable attributes

2015-08-19 Thread Dan Streetman
On Wed, Aug 19, 2015 at 11:02 AM, Vlastimil Babka wrote: > On 08/19/2015 04:21 PM, Dan Streetman wrote: >> On Wed, Aug 19, 2015 at 10:02 AM, Vlastimil Babka wrote: >>> On 08/18/2015 09:07 PM, Dan Streetman wrote: +pages are freed. The pool is not preallocated. By default, a zpool of

Re: [PATCH] zswap: update docs for runtime-changeable attributes

2015-08-19 Thread Vlastimil Babka
On 08/19/2015 04:21 PM, Dan Streetman wrote: > On Wed, Aug 19, 2015 at 10:02 AM, Vlastimil Babka wrote: >> On 08/18/2015 09:07 PM, Dan Streetman wrote: >>> +pages are freed. The pool is not preallocated. By default, a zpool of >>> type >>> +zbud is created, but it can be selected at boot time b

Re: [PATCH] zswap: update docs for runtime-changeable attributes

2015-08-19 Thread Dan Streetman
On Wed, Aug 19, 2015 at 10:02 AM, Vlastimil Babka wrote: > On 08/18/2015 09:07 PM, Dan Streetman wrote: >> Change the Documentation/vm/zswap.txt doc to indicate that the "zpool" >> and "compressor" params are now changeable at runtime. >> >> Signed-off-by: Dan Streetman >> --- >> Documentation/v

Re: [PATCH] zswap: update docs for runtime-changeable attributes

2015-08-19 Thread Vlastimil Babka
On 08/18/2015 09:07 PM, Dan Streetman wrote: > Change the Documentation/vm/zswap.txt doc to indicate that the "zpool" > and "compressor" params are now changeable at runtime. > > Signed-off-by: Dan Streetman > --- > Documentation/vm/zswap.txt | 31 +++ > 1 file change

[PATCH] zswap: update docs for runtime-changeable attributes

2015-08-18 Thread Dan Streetman
Change the Documentation/vm/zswap.txt doc to indicate that the "zpool" and "compressor" params are now changeable at runtime. Signed-off-by: Dan Streetman --- Documentation/vm/zswap.txt | 31 +++ 1 file changed, 23 insertions(+), 8 deletions(-) diff --git a/Documenta