diff options
| author | Johannes Weiner <hannes@cmpxchg.org> | 2024-01-29 20:36:45 -0500 |
|---|---|---|
| committer | Andrew Morton <akpm@linux-foundation.org> | 2024-02-22 10:24:43 -0800 |
| commit | 06ed22890cf9749b0e45d59c5cdc6e46dbd7339d (patch) | |
| tree | 3e9e14dc192fbb0ef08d2cbcddfcbcb64c44e250 /mm | |
| parent | be7fc97c52835b931682d79121530773b67e8307 (diff) | |
| download | linux-06ed22890cf9749b0e45d59c5cdc6e46dbd7339d.tar.gz | |
mm: zswap: simplify zswap_invalidate()
The branching is awkward and duplicates code. The comment about
writeback is also misleading: yes, the entry might have been written
back. Or it might have never been stored in zswap to begin with due to
a rejection - zswap_invalidate() is called on all exiting swap entries.
Link: https://lkml.kernel.org/r/20240130014208.565554-10-hannes@cmpxchg.org
Signed-off-by: Johannes Weiner <hannes@cmpxchg.org>
Reviewed-by: Nhat Pham <nphamcs@gmail.com>
Acked-by: Yosry Ahmed <yosryahmed@google.com>
Reviewed-by: Chengming Zhou <zhouchengming@bytedance.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Diffstat (limited to 'mm')
| -rw-r--r-- | mm/zswap.c | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/mm/zswap.c b/mm/zswap.c index 1a86659e1173d..732b0a701b77f 100644 --- a/mm/zswap.c +++ b/mm/zswap.c @@ -1739,15 +1739,10 @@ void zswap_invalidate(int type, pgoff_t offset) struct zswap_tree *tree = swap_zswap_tree(swp_entry(type, offset)); struct zswap_entry *entry; - /* find */ spin_lock(&tree->lock); entry = zswap_rb_search(&tree->rbroot, offset); - if (!entry) { - /* entry was written back */ - spin_unlock(&tree->lock); - return; - } - zswap_invalidate_entry(tree, entry); + if (entry) + zswap_invalidate_entry(tree, entry); spin_unlock(&tree->lock); } |