Re: [PATCH 6/8] mm/zsmalloc: keep comments consistent with code
On Mon, Jul 04, 2016 at 09:05:16AM +0900, Minchan Kim wrote: > On Fri, Jul 01, 2016 at 02:41:04PM +0800, Ganesh Mahendran wrote: > > some minor change of comments: > > 1). update zs_malloc(),zs_create_pool() function header > > 2). update "Usage of struct page fields" > > > > Signed-off-by: Ganesh Mahendran> > --- > > mm/zsmalloc.c | 7 +++ > > 1 file changed, 3 insertions(+), 4 deletions(-) > > > > diff --git a/mm/zsmalloc.c b/mm/zsmalloc.c > > index 2690914..6fc631a 100644 > > --- a/mm/zsmalloc.c > > +++ b/mm/zsmalloc.c > > @@ -20,6 +20,7 @@ > > * page->freelist(index): links together all component pages of a zspage > > * For the huge page, this is always 0, so we use this field > > * to store handle. > > + * page->units: first object index in a subpage of zspage > > Hmm, I want to use offset instead of index. Yes, it should be offset here. I mixed it with obj index. :) Thanks
Re: [PATCH 6/8] mm/zsmalloc: keep comments consistent with code
On Mon, Jul 04, 2016 at 09:05:16AM +0900, Minchan Kim wrote: > On Fri, Jul 01, 2016 at 02:41:04PM +0800, Ganesh Mahendran wrote: > > some minor change of comments: > > 1). update zs_malloc(),zs_create_pool() function header > > 2). update "Usage of struct page fields" > > > > Signed-off-by: Ganesh Mahendran > > --- > > mm/zsmalloc.c | 7 +++ > > 1 file changed, 3 insertions(+), 4 deletions(-) > > > > diff --git a/mm/zsmalloc.c b/mm/zsmalloc.c > > index 2690914..6fc631a 100644 > > --- a/mm/zsmalloc.c > > +++ b/mm/zsmalloc.c > > @@ -20,6 +20,7 @@ > > * page->freelist(index): links together all component pages of a zspage > > * For the huge page, this is always 0, so we use this field > > * to store handle. > > + * page->units: first object index in a subpage of zspage > > Hmm, I want to use offset instead of index. Yes, it should be offset here. I mixed it with obj index. :) Thanks
Re: [PATCH 6/8] mm/zsmalloc: keep comments consistent with code
On Fri, Jul 01, 2016 at 02:41:04PM +0800, Ganesh Mahendran wrote: > some minor change of comments: > 1). update zs_malloc(),zs_create_pool() function header > 2). update "Usage of struct page fields" > > Signed-off-by: Ganesh Mahendran> --- > mm/zsmalloc.c | 7 +++ > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/mm/zsmalloc.c b/mm/zsmalloc.c > index 2690914..6fc631a 100644 > --- a/mm/zsmalloc.c > +++ b/mm/zsmalloc.c > @@ -20,6 +20,7 @@ > * page->freelist(index): links together all component pages of a zspage > * For the huge page, this is always 0, so we use this field > * to store handle. > + * page->units: first object index in a subpage of zspage Hmm, I want to use offset instead of index.
Re: [PATCH 6/8] mm/zsmalloc: keep comments consistent with code
On Fri, Jul 01, 2016 at 02:41:04PM +0800, Ganesh Mahendran wrote: > some minor change of comments: > 1). update zs_malloc(),zs_create_pool() function header > 2). update "Usage of struct page fields" > > Signed-off-by: Ganesh Mahendran > --- > mm/zsmalloc.c | 7 +++ > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/mm/zsmalloc.c b/mm/zsmalloc.c > index 2690914..6fc631a 100644 > --- a/mm/zsmalloc.c > +++ b/mm/zsmalloc.c > @@ -20,6 +20,7 @@ > * page->freelist(index): links together all component pages of a zspage > * For the huge page, this is always 0, so we use this field > * to store handle. > + * page->units: first object index in a subpage of zspage Hmm, I want to use offset instead of index.
[PATCH 6/8] mm/zsmalloc: keep comments consistent with code
some minor change of comments: 1). update zs_malloc(),zs_create_pool() function header 2). update "Usage of struct page fields" Signed-off-by: Ganesh Mahendran--- mm/zsmalloc.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/mm/zsmalloc.c b/mm/zsmalloc.c index 2690914..6fc631a 100644 --- a/mm/zsmalloc.c +++ b/mm/zsmalloc.c @@ -20,6 +20,7 @@ * page->freelist(index): links together all component pages of a zspage * For the huge page, this is always 0, so we use this field * to store handle. + * page->units: first object index in a subpage of zspage * * Usage of struct page flags: * PG_private: identifies the first component page @@ -140,9 +141,6 @@ */ #define ZS_SIZE_CLASS_DELTA(PAGE_SIZE >> CLASS_BITS) -/* - * We do not maintain any list for completely empty or full pages - */ enum fullness_group { ZS_EMPTY, ZS_ALMOST_EMPTY, @@ -1540,6 +1538,7 @@ static unsigned long obj_malloc(struct size_class *class, * zs_malloc - Allocate block of given size from pool. * @pool: pool to allocate from * @size: size of block to allocate + * @gfp: gfp flags when allocating object * * On success, handle to the allocated object is returned, * otherwise 0. @@ -2418,7 +2417,7 @@ static int zs_register_shrinker(struct zs_pool *pool) /** * zs_create_pool - Creates an allocation pool to work from. - * @flags: allocation flags used to allocate pool metadata + * @name: pool name to be created * * This function must be called before anything when using * the zsmalloc allocator. -- 1.9.1
[PATCH 6/8] mm/zsmalloc: keep comments consistent with code
some minor change of comments: 1). update zs_malloc(),zs_create_pool() function header 2). update "Usage of struct page fields" Signed-off-by: Ganesh Mahendran --- mm/zsmalloc.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/mm/zsmalloc.c b/mm/zsmalloc.c index 2690914..6fc631a 100644 --- a/mm/zsmalloc.c +++ b/mm/zsmalloc.c @@ -20,6 +20,7 @@ * page->freelist(index): links together all component pages of a zspage * For the huge page, this is always 0, so we use this field * to store handle. + * page->units: first object index in a subpage of zspage * * Usage of struct page flags: * PG_private: identifies the first component page @@ -140,9 +141,6 @@ */ #define ZS_SIZE_CLASS_DELTA(PAGE_SIZE >> CLASS_BITS) -/* - * We do not maintain any list for completely empty or full pages - */ enum fullness_group { ZS_EMPTY, ZS_ALMOST_EMPTY, @@ -1540,6 +1538,7 @@ static unsigned long obj_malloc(struct size_class *class, * zs_malloc - Allocate block of given size from pool. * @pool: pool to allocate from * @size: size of block to allocate + * @gfp: gfp flags when allocating object * * On success, handle to the allocated object is returned, * otherwise 0. @@ -2418,7 +2417,7 @@ static int zs_register_shrinker(struct zs_pool *pool) /** * zs_create_pool - Creates an allocation pool to work from. - * @flags: allocation flags used to allocate pool metadata + * @name: pool name to be created * * This function must be called before anything when using * the zsmalloc allocator. -- 1.9.1