zram: free meta table in zram_meta_free
authorGanesh Mahendran <opensource.ganesh@gmail.com>
Thu, 12 Feb 2015 23:00:33 +0000 (15:00 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 13 Feb 2015 02:54:11 +0000 (18:54 -0800)
zram_meta_alloc() and zram_meta_free() are a pair.  In
zram_meta_alloc(), meta table is allocated.  So it it better to free it
in zram_meta_free().

Signed-off-by: Ganesh Mahendran <opensource.ganesh@gmail.com>
Acked-by: Minchan Kim <minchan@kernel.org>
Acked-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Nitin Gupta <ngupta@vflare.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/block/zram/zram_drv.c

index 369fe564279973bec0f33f2c40a185cf08c3185e..0e07652cf7c12c3b8ec4a36fe17dbeacf6c99125 100644 (file)
@@ -307,8 +307,21 @@ static inline int valid_io_request(struct zram *zram,
        return 1;
 }
 
-static void zram_meta_free(struct zram_meta *meta)
+static void zram_meta_free(struct zram_meta *meta, u64 disksize)
 {
+       size_t num_pages = disksize >> PAGE_SHIFT;
+       size_t index;
+
+       /* Free all pages that are still in this zram device */
+       for (index = 0; index < num_pages; index++) {
+               unsigned long handle = meta->table[index].handle;
+
+               if (!handle)
+                       continue;
+
+               zs_free(meta->mem_pool, handle);
+       }
+
        zs_destroy_pool(meta->mem_pool);
        vfree(meta->table);
        kfree(meta);
@@ -704,9 +717,6 @@ static void zram_bio_discard(struct zram *zram, u32 index,
 
 static void zram_reset_device(struct zram *zram, bool reset_capacity)
 {
-       size_t index;
-       struct zram_meta *meta;
-
        down_write(&zram->init_lock);
 
        zram->limit_pages = 0;
@@ -716,20 +726,9 @@ static void zram_reset_device(struct zram *zram, bool reset_capacity)
                return;
        }
 
-       meta = zram->meta;
-       /* Free all pages that are still in this zram device */
-       for (index = 0; index < zram->disksize >> PAGE_SHIFT; index++) {
-               unsigned long handle = meta->table[index].handle;
-               if (!handle)
-                       continue;
-
-               zs_free(meta->mem_pool, handle);
-       }
-
        zcomp_destroy(zram->comp);
        zram->max_comp_streams = 1;
-
-       zram_meta_free(zram->meta);
+       zram_meta_free(zram->meta, zram->disksize);
        zram->meta = NULL;
        /* Reset stats */
        memset(&zram->stats, 0, sizeof(zram->stats));
@@ -801,7 +800,7 @@ out_destroy_comp:
        up_write(&zram->init_lock);
        zcomp_destroy(comp);
 out_free_meta:
-       zram_meta_free(meta);
+       zram_meta_free(meta, disksize);
        return err;
 }