On (09/23/15 14:53), a...@linux-foundation.org wrote: > ------------------------------------------------------ > From: Hui Zhu <zhu...@xiaomi.com> > Subject: zsmalloc: add comments for ->inuse to zspage > > Signed-off-by: Hui Zhu <zhu...@xiaomi.com> > Cc: Sergey Senozhatsky <sergey.senozhat...@gmail.com> > Cc: Dan Streetman <ddstr...@ieee.org> > Cc: Minchan Kim <minc...@kernel.org> > Signed-off-by: Andrew Morton <a...@linux-foundation.org> > --- > > mm/zsmalloc.c | 1 + > 1 file changed, 1 insertion(+) > > diff -puN mm/zsmalloc.c~zsmalloc-add-comments-for-inuse-to-zspage > mm/zsmalloc.c > --- a/mm/zsmalloc.c~zsmalloc-add-comments-for-inuse-to-zspage > +++ a/mm/zsmalloc.c > @@ -38,6 +38,7 @@ > * page->lru: links together first pages of various zspages. > * Basically forming list of zspages in a fullness group. > * page->mapping: class index and fullness group of the zspage > + * page->inuse: the pages number that is used in this zspage
Probably arrived last night to my another mailbox, so haven't seen this one. The comment is wrong. page ->inuse represents the number of objects in use in this zspage; not "the pages number". usage example: zspage_full() : page->inuse == page->objects obj_malloc() : first_page->inuse++ obj_free() : first_page->inuse-- -ss -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/