On (02/14/17 20:48), Yisheng Xie wrote: > We had used page->lru to link the component pages (except the first > page) of a zspage, and used INIT_LIST_HEAD(&page->lru) to init it. > Therefore, to get the last page's next page, which is NULL, we had to > use page flag PG_Private_2 to identify it. > > But now, we use page->freelist to link all of the pages in zspage and > init the page->freelist as NULL for last page, so no need to use > PG_Private_2 anymore. > > This patch is to remove redundant SetPagePrivate2 in create_page_chain > and ClearPagePrivate2 in reset_page(). Maybe can save few cycles for > migration of zsmalloc page :) > > Signed-off-by: Yisheng Xie <xieyisheng1@xxxxxxxxxx> looks good to me. Reviewed-by: Sergey Senozhatsky <sergey.senozhatsky@xxxxxxxxx> -ss -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>
