From: alexs@kernel.org
To: Vitaly Wool <vitaly.wool@konsulko.com>,
Miaohe Lin <linmiaohe@huawei.com>,
Andrew Morton <akpm@linux-foundation.org>,
linux-kernel@vger.kernel.org, linux-mm@kvack.org,
minchan@kernel.org, willy@infradead.org,
senozhatsky@chromium.org, david@redhat.com, 42.hyeyoo@gmail.com,
Yosry Ahmed <yosryahmed@google.com>,
nphamcs@gmail.com
Cc: Alex Shi <alexs@kernel.org>
Subject: [PATCH v2 11/20] mm/zsmalloc: rename reset_page to reset_zpdesc and use zpdesc in it
Date: Wed, 3 Jul 2024 12:06:01 +0800 [thread overview]
Message-ID: <20240703040613.681396-12-alexs@kernel.org> (raw)
In-Reply-To: <20240703040613.681396-1-alexs@kernel.org>
From: Alex Shi <alexs@kernel.org>
zpdesc.zspage matches with page.private, zpdesc.next matches with
page.index. They will be reset in reset_page() wich is called prior to
free base pages of a zspage.
Use zpdesc to replace page struct and rename it to reset_zpdesc(), few
page helper still left since they are used too widely.
Signed-off-by: Alex Shi <alexs@kernel.org>
---
mm/zsmalloc.c | 12 +++++++-----
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/mm/zsmalloc.c b/mm/zsmalloc.c
index 9bc9b14187ed..6d1971836391 100644
--- a/mm/zsmalloc.c
+++ b/mm/zsmalloc.c
@@ -865,12 +865,14 @@ static inline bool obj_allocated(struct zpdesc *zpdesc, void *obj,
return true;
}
-static void reset_page(struct page *page)
+static void reset_zpdesc(struct zpdesc *zpdesc)
{
+ struct page *page = zpdesc_page(zpdesc);
+
__ClearPageMovable(page);
ClearPagePrivate(page);
- set_page_private(page, 0);
- page->index = 0;
+ zpdesc->zspage = NULL;
+ zpdesc->next = NULL;
reset_first_obj_offset(page);
__ClearPageZsmalloc(page);
}
@@ -910,7 +912,7 @@ static void __free_zspage(struct zs_pool *pool, struct size_class *class,
do {
VM_BUG_ON_PAGE(!PageLocked(page), page);
next = get_next_page(page);
- reset_page(page);
+ reset_zpdesc(page_zpdesc(page));
unlock_page(page);
dec_zone_page_state(page, NR_ZSPAGES);
put_page(page);
@@ -1899,7 +1901,7 @@ static int zs_page_migrate(struct page *newpage, struct page *page,
zpdesc_inc_zone_page_state(newzpdesc);
}
- reset_page(page);
+ reset_zpdesc(zpdesc);
zpdesc_put(zpdesc);
return MIGRATEPAGE_SUCCESS;
--
2.43.0
next prev parent reply other threads:[~2024-07-03 4:01 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-03 4:05 [PATCH v2 00/20] mm/zsmalloc: add zpdesc memory descriptor for zswap.zpool alexs
2024-07-03 4:05 ` [PATCH v2 01/20] " alexs
2024-07-05 13:41 ` kernel test robot
2024-07-05 14:02 ` kernel test robot
2024-07-03 4:05 ` [PATCH v2 02/20] mm/zsmalloc: use zpdesc in trylock_zspage/lock_zspage alexs
2024-07-03 4:05 ` [PATCH v2 03/20] mm/zsmalloc: convert __zs_map_object/__zs_unmap_object to use zpdesc alexs
2024-07-03 4:05 ` [PATCH v2 04/20] mm/zsmalloc: add and use pfn/zpdesc seeking funcs alexs
2024-07-03 4:05 ` [PATCH v2 05/20] mm/zsmalloc: convert obj_malloc() to use zpdesc alexs
2024-07-03 4:05 ` [PATCH v2 06/20] mm/zsmalloc: convert create_page_chain() and its users " alexs
2024-07-03 4:05 ` [PATCH v2 07/20] mm/zsmalloc: convert obj_allocated() and related helpers " alexs
2024-07-03 4:05 ` [PATCH v2 08/20] mm/zsmalloc: convert init_zspage() " alexs
2024-07-03 4:05 ` [PATCH v2 09/20] mm/zsmalloc: convert obj_to_page() and zs_free() " alexs
2024-07-03 4:06 ` [PATCH v2 10/20] mm/zsmalloc: add zpdesc_is_isolated/zpdesc_zone helper for zs_page_migrate alexs
2024-07-03 4:06 ` alexs [this message]
2024-07-03 4:06 ` [PATCH v2 12/20] mm/zsmalloc: convert __free_zspage() to use zdsesc alexs
2024-07-03 4:06 ` [PATCH v2 13/20] mm/zsmalloc: convert location_to_obj() to take zpdesc alexs
2024-07-03 4:06 ` [PATCH v2 14/20] mm/zsmalloc: convert migrate_zspage() to use zpdesc alexs
2024-07-03 4:06 ` [PATCH v2 15/20] mm/zsmalloc: convert get_zspage() to take zpdesc alexs
2024-07-03 4:06 ` [PATCH v2 16/20] mm/zsmalloc: convert SetZsPageMovable and remove unused funcs alexs
2024-07-03 4:06 ` [PATCH v2 17/20] mm/zsmalloc: convert get/set_first_obj_offset() to take zpdesc alexs
2024-07-05 14:43 ` kernel test robot
2024-07-05 17:15 ` Nathan Chancellor
2024-07-03 4:06 ` [PATCH v2 18/20] mm/zsmalloc: introduce __zpdesc_clear_movable alexs
2024-07-03 4:06 ` [PATCH v2 19/20] mm/zsmalloc: introduce __zpdesc_clear_zsmalloc alexs
2024-07-03 4:06 ` [PATCH v2 20/20] mm/zsmalloc: introduce __zpdesc_set_zsmalloc() alexs
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240703040613.681396-12-alexs@kernel.org \
--to=alexs@kernel.org \
--cc=42.hyeyoo@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=david@redhat.com \
--cc=linmiaohe@huawei.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=minchan@kernel.org \
--cc=nphamcs@gmail.com \
--cc=senozhatsky@chromium.org \
--cc=vitaly.wool@konsulko.com \
--cc=willy@infradead.org \
--cc=yosryahmed@google.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.