diff options
| author | Ming Lei <[email protected]> | 2021-10-25 10:54:23 +0800 |
|---|---|---|
| committer | Jens Axboe <[email protected]> | 2021-11-02 14:43:12 -0600 |
| commit | 6f1637795f2827d36aec9e0246487f5852e8abf7 (patch) | |
| tree | a91d5e4e8ea97e1cc5a4f04ce9c313260ac38b79 | |
| parent | 494dbee341e7a02529ce776ee9a5e0b7733ca280 (diff) | |
zram: fix race between zram_reset_device() and disksize_store()
When the ->init_lock is released in zram_reset_device(), disksize_store()
can come in and try to allocate meta, but zram_reset_device() is freeing
free meta, so cause races.
Link: https://lore.kernel.org/linux-block/[email protected]/T/#mc617f865a3fa2778e40f317ddf48f6447c20c073
Reported-by: Luis Chamberlain <[email protected]>
Reviewed-by: Luis Chamberlain <[email protected]>
Signed-off-by: Ming Lei <[email protected]>
Acked-by: Minchan Kim <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Jens Axboe <[email protected]>
| -rw-r--r-- | drivers/block/zram/zram_drv.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/block/zram/zram_drv.c b/drivers/block/zram/zram_drv.c index 876bf191e2ca..9b38e5f749c6 100644 --- a/drivers/block/zram/zram_drv.c +++ b/drivers/block/zram/zram_drv.c @@ -1704,12 +1704,13 @@ static void zram_reset_device(struct zram *zram) set_capacity_and_notify(zram->disk, 0); part_stat_set_all(zram->disk->part0, 0); - up_write(&zram->init_lock); /* I/O operation under all of CPU are done so let's free */ zram_meta_free(zram, disksize); memset(&zram->stats, 0, sizeof(zram->stats)); zcomp_destroy(comp); reset_bdev(zram); + + up_write(&zram->init_lock); } static ssize_t disksize_store(struct device *dev, |