Re: [PATCH v1 4/5] zram: add swap full hint

2014-10-08 Thread Dan Streetman
On Mon, Oct 6, 2014 at 7:46 PM, Minchan Kim wrote: > On Tue, Oct 07, 2014 at 08:36:08AM +0900, Minchan Kim wrote: >> Hello Dan, >> >> Sorry for the delay. I had internal works which should be handled >> urgent. I hope you don't lose your interest due to my bad response >> latency. >> >> On Thu, Se

Re: [PATCH v1 4/5] zram: add swap full hint

2014-10-06 Thread Minchan Kim
On Tue, Oct 07, 2014 at 08:36:08AM +0900, Minchan Kim wrote: > Hello Dan, > > Sorry for the delay. I had internal works which should be handled > urgent. I hope you don't lose your interest due to my bad response > latency. > > On Thu, Sep 25, 2014 at 11:52:22AM -0400, Dan Streetman wrote: > > On

Re: [PATCH v1 4/5] zram: add swap full hint

2014-10-06 Thread Minchan Kim
Hello Dan, Sorry for the delay. I had internal works which should be handled urgent. I hope you don't lose your interest due to my bad response latency. On Thu, Sep 25, 2014 at 11:52:22AM -0400, Dan Streetman wrote: > On Wed, Sep 24, 2014 at 9:02 PM, Minchan Kim wrote: > > On Wed, Sep 24, 2014 a

Re: [PATCH v1 4/5] zram: add swap full hint

2014-09-25 Thread Dan Streetman
On Wed, Sep 24, 2014 at 9:02 PM, Minchan Kim wrote: > On Wed, Sep 24, 2014 at 10:01:03AM -0400, Dan Streetman wrote: >> On Sun, Sep 21, 2014 at 8:03 PM, Minchan Kim wrote: >> > This patch implement SWAP_FULL handler in zram so that VM can >> > know whether zram is full or not and use it to stop a

Re: [PATCH v1 4/5] zram: add swap full hint

2014-09-24 Thread Minchan Kim
On Wed, Sep 24, 2014 at 05:10:22PM +0200, Jerome Marchand wrote: > On 09/23/2014 11:17 PM, Andrew Morton wrote: > > On Tue, 23 Sep 2014 13:56:02 +0900 Minchan Kim wrote: > > > >>> > +#define ZRAM_FULLNESS_PERCENT 80 > >>> > >>> We've had problems in the past where 1% is just too large an inc

Re: [PATCH v1 4/5] zram: add swap full hint

2014-09-24 Thread Minchan Kim
On Wed, Sep 24, 2014 at 10:01:03AM -0400, Dan Streetman wrote: > On Sun, Sep 21, 2014 at 8:03 PM, Minchan Kim wrote: > > This patch implement SWAP_FULL handler in zram so that VM can > > know whether zram is full or not and use it to stop anonymous > > page reclaim. > > > > How to judge fullness i

Re: [PATCH v1 4/5] zram: add swap full hint

2014-09-24 Thread Jerome Marchand
On 09/23/2014 11:17 PM, Andrew Morton wrote: > On Tue, 23 Sep 2014 13:56:02 +0900 Minchan Kim wrote: > >>> +#define ZRAM_FULLNESS_PERCENT 80 >>> >>> We've had problems in the past where 1% is just too large an increment >>> for large systems. >> >> So, do you want fullness_bytes like dirty_b

Re: [PATCH v1 4/5] zram: add swap full hint

2014-09-24 Thread Dan Streetman
On Sun, Sep 21, 2014 at 8:03 PM, Minchan Kim wrote: > This patch implement SWAP_FULL handler in zram so that VM can > know whether zram is full or not and use it to stop anonymous > page reclaim. > > How to judge fullness is below, > > fullness = (100 * used space / total space) > > It means the h

Re: [PATCH v1 4/5] zram: add swap full hint

2014-09-24 Thread Minchan Kim
On Tue, Sep 23, 2014 at 02:17:55PM -0700, Andrew Morton wrote: > On Tue, 23 Sep 2014 13:56:02 +0900 Minchan Kim wrote: > > > > > > > > +#define ZRAM_FULLNESS_PERCENT 80 > > > > > > We've had problems in the past where 1% is just too large an increment > > > for large systems. > > > > So, do yo

Re: [PATCH v1 4/5] zram: add swap full hint

2014-09-23 Thread Andrew Morton
On Tue, 23 Sep 2014 13:56:02 +0900 Minchan Kim wrote: > > > > > +#define ZRAM_FULLNESS_PERCENT 80 > > > > We've had problems in the past where 1% is just too large an increment > > for large systems. > > So, do you want fullness_bytes like dirty_bytes? Firstly I'd like you to think about whet

Re: [PATCH v1 4/5] zram: add swap full hint

2014-09-22 Thread Minchan Kim
On Mon, Sep 22, 2014 at 02:11:18PM -0700, Andrew Morton wrote: > On Mon, 22 Sep 2014 09:03:10 +0900 Minchan Kim wrote: > > > This patch implement SWAP_FULL handler in zram so that VM can > > know whether zram is full or not and use it to stop anonymous > > page reclaim. > > > > How to judge full

Re: [PATCH v1 4/5] zram: add swap full hint

2014-09-22 Thread Andrew Morton
On Mon, 22 Sep 2014 09:03:10 +0900 Minchan Kim wrote: > This patch implement SWAP_FULL handler in zram so that VM can > know whether zram is full or not and use it to stop anonymous > page reclaim. > > How to judge fullness is below, > > fullness = (100 * used space / total space) > > It means