mbox series

[v3,0/4] zswap same-filled and limit checking cleanups

Message ID 20240413022407.785696-1-yosryahmed@google.com (mailing list archive)
Headers show
Series zswap same-filled and limit checking cleanups | expand

Message

Yosry Ahmed April 13, 2024, 2:24 a.m. UTC
Miscellaneous cleanups for limit checking and same-filled handling in
the store path. This series was broken out of the "zswap: store
zero-filled pages more efficiently" series [1]. It contains the cleanups
and drops the main functional changes.

[1]https://lore.kernel.org/lkml/20240325235018.2028408-1-yosryahmed@google.com/

v2 -> v3:
- Dropped "mm: zswap: calculate limits only when updated" patch after
  discussions with David Hildenbrand about the complexity of adding
  notifiers for totalram_pages changes.
- Restored "mm: zswap: refactor limit checking from zswap_store()" to
  the v1 version, and addressed Johannes's comment about deferring the
  call to zswap_accept_thr_pages() to avoid the division when not
  needed. Consequently, dropped the Ack and Review tags from v2.
- Added a separate comment header for same-filled pags in patch 2.
- Collected Acks and Review tags (thanks!).
- Rebased on top of the latest mm-unstable.

v2: https://lore.kernel.org/lkml/20240405053510.1948982-1-yosryahmed@google.com/

Yosry Ahmed (4):
  mm: zswap: always shrink in zswap_store() if zswap_pool_reached_full
  mm: zswap: refactor limit checking from zswap_store()
  mm: zswap: move more same-filled pages checks outside of zswap_store()
  mm: zswap: remove same_filled module params

 mm/zswap.c | 98 +++++++++++++++++++++++-------------------------------
 1 file changed, 41 insertions(+), 57 deletions(-)