zram: remove redundant checks from zram_recompress()
authorSergey Senozhatsky <senozhatsky@chromium.org>
Wed, 9 Nov 2022 11:50:43 +0000 (20:50 +0900)
committerAndrew Morton <akpm@linux-foundation.org>
Wed, 30 Nov 2022 23:58:52 +0000 (15:58 -0800)
Size class index comparison is powerful enough so we can remove object
size comparisons.

Link: https://lkml.kernel.org/r/20221109115047.2921851-10-senozhatsky@chromium.org
Signed-off-by: Sergey Senozhatsky <senozhatsky@chromium.org>
Acked-by: Minchan Kim <minchan@kernel.org>
Cc: Alexey Romanov <avromanov@sberdevices.ru>
Cc: Nhat Pham <nphamcs@gmail.com>
Cc: Nitin Gupta <ngupta@vflare.org>
Cc: Suleiman Souhlal <suleiman@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
drivers/block/zram/zram_drv.c

index 66659f1..72beb33 100644 (file)
@@ -1722,9 +1722,7 @@ static int zram_recompress(struct zram *zram, u32 index, struct page *page,
                                                        comp_len_new);
 
                /* Continue until we make progress */
-               if (comp_len_new >= huge_class_size ||
-                   comp_len_new >= comp_len_old ||
-                   class_index_new >= class_index_old ||
+               if (class_index_new >= class_index_old ||
                    (threshold && comp_len_new >= threshold)) {
                        zcomp_stream_put(zram->comps[prio]);
                        continue;
@@ -1747,9 +1745,7 @@ static int zram_recompress(struct zram *zram, u32 index, struct page *page,
         * that would save memory, mark the object as incompressible so that
         * we will not try to compress it again.
         */
-       if (comp_len_new >= huge_class_size ||
-           comp_len_new >= comp_len_old ||
-           class_index_new >= class_index_old) {
+       if (class_index_new >= class_index_old) {
                zram_set_flag(zram, index, ZRAM_INCOMPRESSIBLE);
                return 0;
        }