[PATCH 5/5] zram: Set initial disksize to some default value

From: Nitin Gupta
Date: Fri Sep 09 2011 - 19:01:46 EST


Currently, we set initial disksize as 0, which forces
user to write some value to corresponding zram device's
sysfs node, before the device can be used. Now, we avoid
this step by providing some default size initially.

To change the disksize, user must:
- Reset disk.
Ex: echo 1 > /sys/block/zram0/reset
(NOTE: disksize is set to the default value after reset)

- Set new disksize.
Ex: echo $((256*1024*1024)) > /sys/block/zram0/disksize

Signed-off-by: Nitin Gupta <ngupta@xxxxxxxxxx>
Reviewed-by: Jerome Marchand <jmarchan@xxxxxxxxxx>
---
drivers/staging/zram/zram_drv.c | 13 +++++++------
1 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/zram/zram_drv.c b/drivers/staging/zram/zram_drv.c
index b1c4abd..9451d07 100644
--- a/drivers/staging/zram/zram_drv.c
+++ b/drivers/staging/zram/zram_drv.c
@@ -599,7 +599,7 @@ void zram_reset_device(struct zram *zram)
/* Reset stats */
memset(&zram->stats, 0, sizeof(zram->stats));

- zram->disksize = 0;
+ zram_set_disksize(zram, zram_default_disksize_bytes());
mutex_unlock(&zram->init_lock);
}

@@ -615,9 +615,6 @@ int zram_init_device(struct zram *zram)
return 0;
}

- if (!zram->disksize)
- zram_set_disksize(zram, zram_default_disksize_bytes());
-
zram->compress_workmem = kzalloc(LZO1X_MEM_COMPRESS, GFP_KERNEL);
if (!zram->compress_workmem) {
pr_err("Error allocating compressor working memory!\n");
@@ -717,8 +714,12 @@ static int create_device(struct zram *zram, int device_id)
zram->disk->private_data = zram;
snprintf(zram->disk->disk_name, 16, "zram%d", device_id);

- /* Actual capacity set using syfs (/sys/block/zram<id>/disksize */
- zram_set_disksize(zram, 0);
+ /*
+ * Set some default disksize. To set another disksize, user
+ * must reset the device and then write a new disksize to
+ * corresponding device's sysfs node.
+ */
+ zram_set_disksize(zram, zram_default_disksize_bytes());

/*
* To ensure that we always get PAGE_SIZE aligned
--
1.7.6

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/