Since zswap_rb_erase was added to the final (when refcount == 0) zswap_put_entry, there is no need to call zswap_rb_erase before calling zswap_put_entry. Signed-off-by: Dan Streetman <ddstreet@xxxxxxxx> --- mm/zswap.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/mm/zswap.c b/mm/zswap.c index e154f1e..f4fbbd5 100644 --- a/mm/zswap.c +++ b/mm/zswap.c @@ -711,8 +711,6 @@ static int zswap_frontswap_store(unsigned type, pgoff_t offset, ret = zswap_rb_insert(&tree->rbroot, entry, &dupentry); if (ret == -EEXIST) { zswap_duplicate_entry++; - /* remove from rbtree */ - zswap_rb_erase(&tree->rbroot, dupentry); zswap_entry_put(tree, dupentry); } } while (ret == -EEXIST); @@ -787,9 +785,6 @@ static void zswap_frontswap_invalidate_page(unsigned type, pgoff_t offset) return; } - /* remove from rbtree */ - zswap_rb_erase(&tree->rbroot, entry); - /* drop the initial reference from entry creation */ zswap_entry_put(tree, entry); -- 1.8.3.1 -- 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>