Re: [RFC] mm: add support for zsmalloc and zcache

2012-11-02 Thread Konrad Rzeszutek Wilk
On Fri, Oct 26, 2012 at 04:45:14PM -0500, Seth Jennings wrote: > On 10/02/2012 01:17 PM, Dan Magenheimer wrote: > > If so, and move forward? What do you see as next steps? > > I've been reviewing the changes between zcache and zcache2 and getting > a feel for the scope and direction of those

Re: [RFC] mm: add support for zsmalloc and zcache

2012-11-02 Thread Konrad Rzeszutek Wilk
On Fri, Oct 26, 2012 at 04:45:14PM -0500, Seth Jennings wrote: On 10/02/2012 01:17 PM, Dan Magenheimer wrote: If so, shake hands and move forward? What do you see as next steps? I've been reviewing the changes between zcache and zcache2 and getting a feel for the scope and direction of

Re: [RFC] mm: add support for zsmalloc and zcache

2012-10-26 Thread Seth Jennings
On 10/02/2012 01:17 PM, Dan Magenheimer wrote: > If so, and move forward? What do you see as next steps? I've been reviewing the changes between zcache and zcache2 and getting a feel for the scope and direction of those changes. - Getting the community engaged to review zcache1 at ~2300SLOC

Re: [RFC] mm: add support for zsmalloc and zcache

2012-10-26 Thread Seth Jennings
On 10/02/2012 01:17 PM, Dan Magenheimer wrote: If so, shake hands and move forward? What do you see as next steps? I've been reviewing the changes between zcache and zcache2 and getting a feel for the scope and direction of those changes. - Getting the community engaged to review zcache1 at

Re: [RFC] mm: add support for zsmalloc and zcache

2012-10-04 Thread Seth Jennings
On 10/02/2012 01:17 PM, Dan Magenheimer wrote: > If so, and move forward? What do you see as next steps? I'll need to get up to speed on the new codebase before I can answer this. I should be able to answer by early next week. Seth -- To unsubscribe from this list: send the line "unsubscribe

Re: [RFC] mm: add support for zsmalloc and zcache

2012-10-04 Thread Seth Jennings
On 10/02/2012 01:17 PM, Dan Magenheimer wrote: If so, shake hands and move forward? What do you see as next steps? I'll need to get up to speed on the new codebase before I can answer this. I should be able to answer by early next week. Seth -- To unsubscribe from this list: send the line

RE: [RFC] mm: add support for zsmalloc and zcache

2012-10-02 Thread Dan Magenheimer
> From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] > Subject: Re: [RFC] mm: add support for zsmalloc and zcache > > On 09/27/2012 05:07 PM, Dan Magenheimer wrote: > > Of course, I'm of the opinion that neither zcache1 nor > > zcache2 would be likely to be promo

Re: [RFC] mm: add support for zsmalloc and zcache

2012-10-02 Thread Seth Jennings
On 09/27/2012 05:07 PM, Dan Magenheimer wrote: > Of course, I'm of the opinion that neither zcache1 nor > zcache2 would be likely to be promoted for at least another > cycle or two, so if you go with zcache2+zsmalloc as the compromise > and it still takes six months for promotion, I hope you don't

Re: [RFC/PATCH] zcache2 on PPC64 (Was: [RFC] mm: add support for zsmalloc and zcache)

2012-10-02 Thread Seth Jennings
On 09/28/2012 08:31 AM, Mel Gorman wrote: > On Tue, Sep 25, 2012 at 04:31:01PM -0700, Dan Magenheimer wrote: >> Attached patch applies to staging-next and I _think_ should >> fix the reported problem where zbud in zcache2 does not >> work on a PPC64 with PAGE_SIZE!=12. I do not have a machine >>

Re: [RFC/PATCH] zcache2 on PPC64 (Was: [RFC] mm: add support for zsmalloc and zcache)

2012-10-02 Thread Seth Jennings
On 09/28/2012 08:31 AM, Mel Gorman wrote: On Tue, Sep 25, 2012 at 04:31:01PM -0700, Dan Magenheimer wrote: Attached patch applies to staging-next and I _think_ should fix the reported problem where zbud in zcache2 does not work on a PPC64 with PAGE_SIZE!=12. I do not have a machine to test

Re: [RFC] mm: add support for zsmalloc and zcache

2012-10-02 Thread Seth Jennings
On 09/27/2012 05:07 PM, Dan Magenheimer wrote: Of course, I'm of the opinion that neither zcache1 nor zcache2 would be likely to be promoted for at least another cycle or two, so if you go with zcache2+zsmalloc as the compromise and it still takes six months for promotion, I hope you don't

RE: [RFC] mm: add support for zsmalloc and zcache

2012-10-02 Thread Dan Magenheimer
From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] Subject: Re: [RFC] mm: add support for zsmalloc and zcache On 09/27/2012 05:07 PM, Dan Magenheimer wrote: Of course, I'm of the opinion that neither zcache1 nor zcache2 would be likely to be promoted for at least another cycle

Re: [RFC/PATCH] zcache2 on PPC64 (Was: [RFC] mm: add support for zsmalloc and zcache)

2012-09-28 Thread Mel Gorman
On Tue, Sep 25, 2012 at 04:31:01PM -0700, Dan Magenheimer wrote: > Attached patch applies to staging-next and I _think_ should > fix the reported problem where zbud in zcache2 does not > work on a PPC64 with PAGE_SIZE!=12. I do not have a machine > to test this so testing by others would be

Re: [RFC/PATCH] zcache2 on PPC64 (Was: [RFC] mm: add support for zsmalloc and zcache)

2012-09-28 Thread Mel Gorman
On Tue, Sep 25, 2012 at 04:31:01PM -0700, Dan Magenheimer wrote: Attached patch applies to staging-next and I _think_ should fix the reported problem where zbud in zcache2 does not work on a PPC64 with PAGE_SIZE!=12. I do not have a machine to test this so testing by others would be

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-27 Thread Dan Magenheimer
> From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] > Subject: Re: [RFC] mm: add support for zsmalloc and zcache > > On 09/24/2012 02:17 PM, Dan Magenheimer wrote: > >> From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] > >> Subject: Re: [RFC] mm: add s

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-27 Thread Seth Jennings
On 09/24/2012 02:17 PM, Dan Magenheimer wrote: >> From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] >> Subject: Re: [RFC] mm: add support for zsmalloc and zcache > > Once again, you have completely ignored a reasonable > compromise proposal. Why? We have use

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-27 Thread Seth Jennings
On 09/24/2012 02:17 PM, Dan Magenheimer wrote: From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] Subject: Re: [RFC] mm: add support for zsmalloc and zcache Once again, you have completely ignored a reasonable compromise proposal. Why? We have users who are interested in zcache

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-27 Thread Dan Magenheimer
From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] Subject: Re: [RFC] mm: add support for zsmalloc and zcache On 09/24/2012 02:17 PM, Dan Magenheimer wrote: From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] Subject: Re: [RFC] mm: add support for zsmalloc and zcache Once

[RFC/PATCH] zcache2 on PPC64 (Was: [RFC] mm: add support for zsmalloc and zcache)

2012-09-25 Thread Dan Magenheimer
Attached patch applies to staging-next and I _think_ should fix the reported problem where zbud in zcache2 does not work on a PPC64 with PAGE_SIZE!=12. I do not have a machine to test this so testing by others would be appreciated. Ideally there should also be a BUILD_BUG_ON to ensure PAGE_SHIFT

[RFC/PATCH] zsmalloc added back to zcache2 (Was: [RFC] mm: add support for zsmalloc and zcache)

2012-09-25 Thread Dan Magenheimer
Attached patch applies to staging-next and adds zsmalloc support, optionally at compile-time and run-time, back into zcache (aka zcache2). It is only lightly tested and does not provide some of the debug info from old zcache (aka zcache1) because it needs to be converted from sysfs to debugfs.

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-25 Thread Dan Magenheimer
> From: Sasha Levin [mailto:levinsasha...@gmail.com] > Subject: Re: [RFC] mm: add support for zsmalloc and zcache Sorry for delayed response! > On 09/22/2012 03:31 PM, Sasha Levin wrote: > > On 09/21/2012 09:14 PM, Dan Magenheimer wrote: > >>>> +#define MAX_CLIENT

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-25 Thread James Bottomley
On Mon, 2012-09-24 at 12:25 -0500, Seth Jennings wrote: > In summary, I really don't understand the objection to > promoting zcache and integrating zcache2 improvements and > features incrementally. It seems very natural and > straightforward to me. Rewrites can even happen in > mainline, as

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-25 Thread Mel Gorman
On Mon, Sep 24, 2012 at 01:36:48PM -0700, Dan Magenheimer wrote: > > From: Mel Gorman [mailto:mgor...@suse.de] > > Subject: Re: [RFC] mm: add support for zsmalloc and zcache > > > > On Sat, Sep 22, 2012 at 02:18:44PM -0700, Dan Magenheimer wrote: > > > > From

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-25 Thread Mel Gorman
On Mon, Sep 24, 2012 at 01:36:48PM -0700, Dan Magenheimer wrote: From: Mel Gorman [mailto:mgor...@suse.de] Subject: Re: [RFC] mm: add support for zsmalloc and zcache On Sat, Sep 22, 2012 at 02:18:44PM -0700, Dan Magenheimer wrote: From: Mel Gorman [mailto:mgor...@suse.de] Subject

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-25 Thread James Bottomley
On Mon, 2012-09-24 at 12:25 -0500, Seth Jennings wrote: In summary, I really don't understand the objection to promoting zcache and integrating zcache2 improvements and features incrementally. It seems very natural and straightforward to me. Rewrites can even happen in mainline, as James

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-25 Thread Dan Magenheimer
From: Sasha Levin [mailto:levinsasha...@gmail.com] Subject: Re: [RFC] mm: add support for zsmalloc and zcache Sorry for delayed response! On 09/22/2012 03:31 PM, Sasha Levin wrote: On 09/21/2012 09:14 PM, Dan Magenheimer wrote: +#define MAX_CLIENTS 16 Seems a bit arbitrary. Why 16

[RFC/PATCH] zsmalloc added back to zcache2 (Was: [RFC] mm: add support for zsmalloc and zcache)

2012-09-25 Thread Dan Magenheimer
Attached patch applies to staging-next and adds zsmalloc support, optionally at compile-time and run-time, back into zcache (aka zcache2). It is only lightly tested and does not provide some of the debug info from old zcache (aka zcache1) because it needs to be converted from sysfs to debugfs.

[RFC/PATCH] zcache2 on PPC64 (Was: [RFC] mm: add support for zsmalloc and zcache)

2012-09-25 Thread Dan Magenheimer
Attached patch applies to staging-next and I _think_ should fix the reported problem where zbud in zcache2 does not work on a PPC64 with PAGE_SIZE!=12. I do not have a machine to test this so testing by others would be appreciated. Ideally there should also be a BUILD_BUG_ON to ensure PAGE_SHIFT

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-24 Thread Dan Magenheimer
> From: Mel Gorman [mailto:mgor...@suse.de] > Subject: Re: [RFC] mm: add support for zsmalloc and zcache > > On Sat, Sep 22, 2012 at 02:18:44PM -0700, Dan Magenheimer wrote: > > > From: Mel Gorman [mailto:mgor...@suse.de] > > > Subject: Re: [RFC] mm: add s

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-24 Thread Dan Magenheimer
> From: James Bottomley [mailto:james.bottom...@hansenpartnership.com] > Subject: Re: [RFC] mm: add support for zsmalloc and zcache > On Sat, 2012-09-22 at 02:07 +0100, Mel Gorman wrote: > > > The two proposals: > > > A) Recreate all the work done for z

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-24 Thread Dan Magenheimer
> From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] > Subject: Re: [RFC] mm: add support for zsmalloc and zcache Once again, you have completely ignored a reasonable compromise proposal. Why? > According to Greg's staging-next, ramster adds 6000 lines of > new co

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-24 Thread Seth Jennings
On 09/21/2012 03:35 PM, Dan Magenheimer wrote: >> From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] >> Subject: Re: [RFC] mm: add support for zsmalloc and zcache >> >> On 09/21/2012 01:02 PM, Konrad Rzeszutek Wilk wrote: >>> On Fri, Sep 21, 2012 at 0

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-24 Thread Mel Gorman
On Sat, Sep 22, 2012 at 02:18:44PM -0700, Dan Magenheimer wrote: > > From: Mel Gorman [mailto:mgor...@suse.de] > > Subject: Re: [RFC] mm: add support for zsmalloc and zcache > > > > On Fri, Sep 21, 2012 at 01:35:15PM -0700, Dan Magenheimer wrote: > > > &g

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-24 Thread Mel Gorman
On Sat, Sep 22, 2012 at 02:18:44PM -0700, Dan Magenheimer wrote: From: Mel Gorman [mailto:mgor...@suse.de] Subject: Re: [RFC] mm: add support for zsmalloc and zcache On Fri, Sep 21, 2012 at 01:35:15PM -0700, Dan Magenheimer wrote: From: Seth Jennings [mailto:sjenn

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-24 Thread Seth Jennings
On 09/21/2012 03:35 PM, Dan Magenheimer wrote: From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] Subject: Re: [RFC] mm: add support for zsmalloc and zcache On 09/21/2012 01:02 PM, Konrad Rzeszutek Wilk wrote: On Fri, Sep 21, 2012 at 05:12:52PM +0100, Mel Gorman wrote: On Tue, Sep 04

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-24 Thread Dan Magenheimer
From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] Subject: Re: [RFC] mm: add support for zsmalloc and zcache Once again, you have completely ignored a reasonable compromise proposal. Why? According to Greg's staging-next, ramster adds 6000 lines of new code to zcache

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-24 Thread Dan Magenheimer
From: James Bottomley [mailto:james.bottom...@hansenpartnership.com] Subject: Re: [RFC] mm: add support for zsmalloc and zcache On Sat, 2012-09-22 at 02:07 +0100, Mel Gorman wrote: The two proposals: A) Recreate all the work done for zcache2 as a proper sequence of independent

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-24 Thread Dan Magenheimer
From: Mel Gorman [mailto:mgor...@suse.de] Subject: Re: [RFC] mm: add support for zsmalloc and zcache On Sat, Sep 22, 2012 at 02:18:44PM -0700, Dan Magenheimer wrote: From: Mel Gorman [mailto:mgor...@suse.de] Subject: Re: [RFC] mm: add support for zsmalloc and zcache On Fri, Sep 21

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-23 Thread James Bottomley
On Sat, 2012-09-22 at 02:07 +0100, Mel Gorman wrote: > > The two proposals: > > A) Recreate all the work done for zcache2 as a proper sequence of > >independent patches and apply them to zcache1. (Seth/Konrad) > > B) Add zsmalloc back in to zcache2 as an alternative allocator > >for

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-23 Thread James Bottomley
On Sat, 2012-09-22 at 02:07 +0100, Mel Gorman wrote: The two proposals: A) Recreate all the work done for zcache2 as a proper sequence of independent patches and apply them to zcache1. (Seth/Konrad) B) Add zsmalloc back in to zcache2 as an alternative allocator for frontswap pages.

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-22 Thread Dan Magenheimer
> From: Mel Gorman [mailto:mgor...@suse.de] > Subject: Re: [RFC] mm: add support for zsmalloc and zcache > > On Fri, Sep 21, 2012 at 01:35:15PM -0700, Dan Magenheimer wrote: > > > From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] > > > Subject: Re: [RFC

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-22 Thread Sasha Levin
On 09/22/2012 03:31 PM, Sasha Levin wrote: > On 09/21/2012 09:14 PM, Dan Magenheimer wrote: +#define MAX_CLIENTS 16 Seems a bit arbitrary. Why 16? >> Sasha Levin posted a patch to fix this but it was tied in to >> the proposed KVM implementation, so was never merged. >> > > My

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-22 Thread Sasha Levin
On 09/21/2012 09:14 PM, Dan Magenheimer wrote: >>> +#define MAX_CLIENTS 16 >> > >> > Seems a bit arbitrary. Why 16? > Sasha Levin posted a patch to fix this but it was tied in to > the proposed KVM implementation, so was never merged. > My patch changed the max pools per client, not the maximum

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-22 Thread Sasha Levin
On 09/21/2012 09:14 PM, Dan Magenheimer wrote: +#define MAX_CLIENTS 16 Seems a bit arbitrary. Why 16? Sasha Levin posted a patch to fix this but it was tied in to the proposed KVM implementation, so was never merged. My patch changed the max pools per client, not the maximum amount of

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-22 Thread Sasha Levin
On 09/22/2012 03:31 PM, Sasha Levin wrote: On 09/21/2012 09:14 PM, Dan Magenheimer wrote: +#define MAX_CLIENTS 16 Seems a bit arbitrary. Why 16? Sasha Levin posted a patch to fix this but it was tied in to the proposed KVM implementation, so was never merged. My patch changed the max

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-22 Thread Dan Magenheimer
From: Mel Gorman [mailto:mgor...@suse.de] Subject: Re: [RFC] mm: add support for zsmalloc and zcache On Fri, Sep 21, 2012 at 01:35:15PM -0700, Dan Magenheimer wrote: From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] Subject: Re: [RFC] mm: add support for zsmalloc and zcache

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-21 Thread Mel Gorman
On Fri, Sep 21, 2012 at 01:35:15PM -0700, Dan Magenheimer wrote: > > From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] > > Subject: Re: [RFC] mm: add support for zsmalloc and zcache > > > > On 09/21/2012 01:02 PM, Konrad Rzeszutek Wilk wrote: > > > On Fri,

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-21 Thread Mel Gorman
On Fri, Sep 21, 2012 at 12:14:39PM -0700, Dan Magenheimer wrote: > Hi Mel -- > > Wow! An incredibly wonderfully detailed response! Thank you very > much for taking the time to read through all of zcache! > My pleasure. > Your comments run the gamut from nit and code style, to design, >

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-21 Thread Dan Magenheimer
> From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] > Subject: Re: [RFC] mm: add support for zsmalloc and zcache > > On 09/21/2012 01:02 PM, Konrad Rzeszutek Wilk wrote: > > On Fri, Sep 21, 2012 at 05:12:52PM +0100, Mel Gorman wrote: > >> On Tue, Sep 04, 201

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-21 Thread Seth Jennings
On 09/21/2012 11:12 AM, Mel Gorman wrote: > That said, my initial feeling still stands. I think that this needs to move > out of staging because it's in limbo where it is but Andrew may disagree > because of the reservations. If my reservations are accurate then they > should at least be *clearly*

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-21 Thread Dan Magenheimer
Hi Mel -- Wow! An incredibly wonderfully detailed response! Thank you very much for taking the time to read through all of zcache! Your comments run the gamut from nit and code style, to design, architecture and broad naming. Until the choice-of-codebase issue is resolved, I'll avoid the nits

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-21 Thread Seth Jennings
On 09/21/2012 01:02 PM, Konrad Rzeszutek Wilk wrote: > On Fri, Sep 21, 2012 at 05:12:52PM +0100, Mel Gorman wrote: >> On Tue, Sep 04, 2012 at 04:34:46PM -0500, Seth Jennings wrote: >>> zcache is the remaining piece of code required to support in-kernel >>> memory compression. The other two

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-21 Thread Konrad Rzeszutek Wilk
On Fri, Sep 21, 2012 at 05:12:52PM +0100, Mel Gorman wrote: > On Tue, Sep 04, 2012 at 04:34:46PM -0500, Seth Jennings wrote: > > zcache is the remaining piece of code required to support in-kernel > > memory compression. The other two features, cleancache and frontswap, > > have been promoted to

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-21 Thread Konrad Rzeszutek Wilk
On Fri, Sep 21, 2012 at 05:12:52PM +0100, Mel Gorman wrote: On Tue, Sep 04, 2012 at 04:34:46PM -0500, Seth Jennings wrote: zcache is the remaining piece of code required to support in-kernel memory compression. The other two features, cleancache and frontswap, have been promoted to

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-21 Thread Seth Jennings
On 09/21/2012 01:02 PM, Konrad Rzeszutek Wilk wrote: On Fri, Sep 21, 2012 at 05:12:52PM +0100, Mel Gorman wrote: On Tue, Sep 04, 2012 at 04:34:46PM -0500, Seth Jennings wrote: zcache is the remaining piece of code required to support in-kernel memory compression. The other two features,

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-21 Thread Dan Magenheimer
Hi Mel -- Wow! An incredibly wonderfully detailed response! Thank you very much for taking the time to read through all of zcache! Your comments run the gamut from nit and code style, to design, architecture and broad naming. Until the choice-of-codebase issue is resolved, I'll avoid the nits

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-21 Thread Seth Jennings
On 09/21/2012 11:12 AM, Mel Gorman wrote: That said, my initial feeling still stands. I think that this needs to move out of staging because it's in limbo where it is but Andrew may disagree because of the reservations. If my reservations are accurate then they should at least be *clearly*

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-21 Thread Dan Magenheimer
From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] Subject: Re: [RFC] mm: add support for zsmalloc and zcache On 09/21/2012 01:02 PM, Konrad Rzeszutek Wilk wrote: On Fri, Sep 21, 2012 at 05:12:52PM +0100, Mel Gorman wrote: On Tue, Sep 04, 2012 at 04:34:46PM -0500, Seth Jennings

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-21 Thread Mel Gorman
On Fri, Sep 21, 2012 at 12:14:39PM -0700, Dan Magenheimer wrote: Hi Mel -- Wow! An incredibly wonderfully detailed response! Thank you very much for taking the time to read through all of zcache! My pleasure. Your comments run the gamut from nit and code style, to design, architecture

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-21 Thread Mel Gorman
On Fri, Sep 21, 2012 at 01:35:15PM -0700, Dan Magenheimer wrote: From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] Subject: Re: [RFC] mm: add support for zsmalloc and zcache On 09/21/2012 01:02 PM, Konrad Rzeszutek Wilk wrote: On Fri, Sep 21, 2012 at 05:12:52PM +0100, Mel Gorman

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-17 Thread Nitin Gupta
On Mon, Sep 17, 2012 at 1:42 PM, Dan Magenheimer wrote: >> From: Nitin Gupta [mailto:ngu...@vflare.org] >> Subject: Re: [RFC] mm: add support for zsmalloc and zcache >> >> The problem is that zbud performs well only when a (compressed) page is >> either PAGE_SIZE/2

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-17 Thread Dan Magenheimer
> From: Nitin Gupta [mailto:ngu...@vflare.org] > Subject: Re: [RFC] mm: add support for zsmalloc and zcache > > The problem is that zbud performs well only when a (compressed) page is > either PAGE_SIZE/2 - e or PAGE_SIZE - e, where e is small. So, even if > the average compr

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-17 Thread Dan Magenheimer
From: Nitin Gupta [mailto:ngu...@vflare.org] Subject: Re: [RFC] mm: add support for zsmalloc and zcache The problem is that zbud performs well only when a (compressed) page is either PAGE_SIZE/2 - e or PAGE_SIZE - e, where e is small. So, even if the average compression ratio is 2x (which

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-17 Thread Nitin Gupta
On Mon, Sep 17, 2012 at 1:42 PM, Dan Magenheimer dan.magenhei...@oracle.com wrote: From: Nitin Gupta [mailto:ngu...@vflare.org] Subject: Re: [RFC] mm: add support for zsmalloc and zcache The problem is that zbud performs well only when a (compressed) page is either PAGE_SIZE/2 - e

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-08 Thread Nitin Gupta
On 09/07/2012 07:37 AM, Konrad Rzeszutek Wilk wrote: significant design challenges exist, many of which are already resolved in the new codebase ("zcache2"). These design issues include: .. snip.. Before other key mm maintainers read and comment on zcache, I think it would be most wise to

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-08 Thread Nitin Gupta
On 09/07/2012 07:37 AM, Konrad Rzeszutek Wilk wrote: significant design challenges exist, many of which are already resolved in the new codebase (zcache2). These design issues include: .. snip.. Before other key mm maintainers read and comment on zcache, I think it would be most wise to move

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-07 Thread Seth Jennings
On 09/06/2012 03:37 PM, Dan Magenheimer wrote: > In response to this RFC for zcache promotion, I've been asked to summarize > the concerns and objections which led me to NACK the previous zcache > promotion request. While I see great potential in zcache, I think some > significant design

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-07 Thread Konrad Rzeszutek Wilk
> significant design challenges exist, many of which are already resolved in > the new codebase ("zcache2"). These design issues include: .. snip.. > Before other key mm maintainers read and comment on zcache, I think > it would be most wise to move to a codebase which resolves the known design >

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-07 Thread Konrad Rzeszutek Wilk
significant design challenges exist, many of which are already resolved in the new codebase (zcache2). These design issues include: .. snip.. Before other key mm maintainers read and comment on zcache, I think it would be most wise to move to a codebase which resolves the known design

Re: [RFC] mm: add support for zsmalloc and zcache

2012-09-07 Thread Seth Jennings
On 09/06/2012 03:37 PM, Dan Magenheimer wrote: In response to this RFC for zcache promotion, I've been asked to summarize the concerns and objections which led me to NACK the previous zcache promotion request. While I see great potential in zcache, I think some significant design challenges

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-06 Thread Dan Magenheimer
In response to this RFC for zcache promotion, I've been asked to summarize the concerns and objections which led me to NACK the previous zcache promotion request. While I see great potential in zcache, I think some significant design challenges exist, many of which are already resolved in the new

RE: [RFC] mm: add support for zsmalloc and zcache

2012-09-06 Thread Dan Magenheimer
In response to this RFC for zcache promotion, I've been asked to summarize the concerns and objections which led me to NACK the previous zcache promotion request. While I see great potential in zcache, I think some significant design challenges exist, many of which are already resolved in the new