RE: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-25 Thread Dan Magenheimer
> From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] > Subject: Re: [PATCHv5 1/8] zsmalloc: add to mm/ > > On 02/25/2013 11:05 AM, Dan Magenheimer wrote: > >> From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] > >> Sent: Friday, February 22, 201

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-25 Thread Seth Jennings
On 02/25/2013 11:05 AM, Dan Magenheimer wrote: >> From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] >> Sent: Friday, February 22, 2013 1:04 PM >> To: Joonsoo Kim >> Subject: Re: [PATCHv5 1/8] zsmalloc: add to mm/ >> >> On 02/22/2013 03:24 AM, Joonsoo Kim

RE: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-25 Thread Dan Magenheimer
> From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] > Sent: Friday, February 22, 2013 1:04 PM > To: Joonsoo Kim > Subject: Re: [PATCHv5 1/8] zsmalloc: add to mm/ > > On 02/22/2013 03:24 AM, Joonsoo Kim wrote: > > > > It's my quick thought. So there is no c

RE: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-25 Thread Dan Magenheimer
From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] Sent: Friday, February 22, 2013 1:04 PM To: Joonsoo Kim Subject: Re: [PATCHv5 1/8] zsmalloc: add to mm/ On 02/22/2013 03:24 AM, Joonsoo Kim wrote: It's my quick thought. So there is no concrete idea. As Seth said, with a FULL

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-25 Thread Seth Jennings
On 02/25/2013 11:05 AM, Dan Magenheimer wrote: From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] Sent: Friday, February 22, 2013 1:04 PM To: Joonsoo Kim Subject: Re: [PATCHv5 1/8] zsmalloc: add to mm/ On 02/22/2013 03:24 AM, Joonsoo Kim wrote: It's my quick thought. So

RE: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-25 Thread Dan Magenheimer
From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] Subject: Re: [PATCHv5 1/8] zsmalloc: add to mm/ On 02/25/2013 11:05 AM, Dan Magenheimer wrote: From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] Sent: Friday, February 22, 2013 1:04 PM To: Joonsoo Kim Subject: Re

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-22 Thread Seth Jennings
On 02/22/2013 03:24 AM, Joonsoo Kim wrote: > On Wed, Feb 20, 2013 at 08:37:33AM +0900, Minchan Kim wrote: >> On Tue, Feb 19, 2013 at 11:54:21AM -0600, Seth Jennings wrote: >>> On 02/19/2013 03:18 AM, Joonsoo Kim wrote: Hello, Seth. I'm not sure that this is right time to review, because

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-22 Thread Joonsoo Kim
On Wed, Feb 20, 2013 at 08:37:33AM +0900, Minchan Kim wrote: > On Tue, Feb 19, 2013 at 11:54:21AM -0600, Seth Jennings wrote: > > On 02/19/2013 03:18 AM, Joonsoo Kim wrote: > > > Hello, Seth. > > > I'm not sure that this is right time to review, because I already have > > > seen many effort of

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-22 Thread Joonsoo Kim
On Wed, Feb 20, 2013 at 08:37:33AM +0900, Minchan Kim wrote: On Tue, Feb 19, 2013 at 11:54:21AM -0600, Seth Jennings wrote: On 02/19/2013 03:18 AM, Joonsoo Kim wrote: Hello, Seth. I'm not sure that this is right time to review, because I already have seen many effort of various people

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-22 Thread Seth Jennings
On 02/22/2013 03:24 AM, Joonsoo Kim wrote: On Wed, Feb 20, 2013 at 08:37:33AM +0900, Minchan Kim wrote: On Tue, Feb 19, 2013 at 11:54:21AM -0600, Seth Jennings wrote: On 02/19/2013 03:18 AM, Joonsoo Kim wrote: Hello, Seth. I'm not sure that this is right time to review, because I already have

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-19 Thread Nitin Gupta
On 02/19/2013 09:54 AM, Seth Jennings wrote: > On 02/19/2013 03:18 AM, Joonsoo Kim wrote: >> Hello, Seth. >> I'm not sure that this is right time to review, because I already have >> seen many effort of various people to promote zxxx series. I don't want to >> be a stopper to promote these. :) >

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-19 Thread Minchan Kim
On Tue, Feb 19, 2013 at 11:54:21AM -0600, Seth Jennings wrote: > On 02/19/2013 03:18 AM, Joonsoo Kim wrote: > > Hello, Seth. > > I'm not sure that this is right time to review, because I already have > > seen many effort of various people to promote zxxx series. I don't want to > > be a stopper to

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-19 Thread Seth Jennings
On 02/19/2013 03:18 AM, Joonsoo Kim wrote: > Hello, Seth. > I'm not sure that this is right time to review, because I already have > seen many effort of various people to promote zxxx series. I don't want to > be a stopper to promote these. :) Any time is good review time :) Thanks for your

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-19 Thread Joonsoo Kim
Hello, Seth. I'm not sure that this is right time to review, because I already have seen many effort of various people to promote zxxx series. I don't want to be a stopper to promote these. :) But, I read the code, now, and then some comments below. On Wed, Feb 13, 2013 at 12:38:44PM -0600, Seth

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-19 Thread Joonsoo Kim
Hello, Seth. I'm not sure that this is right time to review, because I already have seen many effort of various people to promote zxxx series. I don't want to be a stopper to promote these. :) But, I read the code, now, and then some comments below. On Wed, Feb 13, 2013 at 12:38:44PM -0600, Seth

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-19 Thread Seth Jennings
On 02/19/2013 03:18 AM, Joonsoo Kim wrote: Hello, Seth. I'm not sure that this is right time to review, because I already have seen many effort of various people to promote zxxx series. I don't want to be a stopper to promote these. :) Any time is good review time :) Thanks for your review!

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-19 Thread Minchan Kim
On Tue, Feb 19, 2013 at 11:54:21AM -0600, Seth Jennings wrote: On 02/19/2013 03:18 AM, Joonsoo Kim wrote: Hello, Seth. I'm not sure that this is right time to review, because I already have seen many effort of various people to promote zxxx series. I don't want to be a stopper to promote

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-19 Thread Nitin Gupta
On 02/19/2013 09:54 AM, Seth Jennings wrote: On 02/19/2013 03:18 AM, Joonsoo Kim wrote: Hello, Seth. I'm not sure that this is right time to review, because I already have seen many effort of various people to promote zxxx series. I don't want to be a stopper to promote these. :) Any time

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-18 Thread Seth Jennings
On 02/15/2013 09:26 PM, Ric Mason wrote: > On 02/14/2013 02:38 AM, Seth Jennings wrote: >> = >> DO NOT MERGE, FOR REVIEW ONLY >> This patch introduces zsmalloc as new code, however, it already >> exists in drivers/staging. In order to build successfully, you >> must select EITHER to

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-18 Thread Seth Jennings
On 02/15/2013 09:26 PM, Ric Mason wrote: On 02/14/2013 02:38 AM, Seth Jennings wrote: = DO NOT MERGE, FOR REVIEW ONLY This patch introduces zsmalloc as new code, however, it already exists in drivers/staging. In order to build successfully, you must select EITHER to driver/staging

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-15 Thread Ric Mason
On 02/14/2013 02:38 AM, Seth Jennings wrote: = DO NOT MERGE, FOR REVIEW ONLY This patch introduces zsmalloc as new code, however, it already exists in drivers/staging. In order to build successfully, you must select EITHER to driver/staging version OR this version. Once zsmalloc is

Re: [PATCHv5 1/8] zsmalloc: add to mm/

2013-02-15 Thread Ric Mason
On 02/14/2013 02:38 AM, Seth Jennings wrote: = DO NOT MERGE, FOR REVIEW ONLY This patch introduces zsmalloc as new code, however, it already exists in drivers/staging. In order to build successfully, you must select EITHER to driver/staging version OR this version. Once zsmalloc is