[PATCH v4] zram: add num_{discard_req, discarded} for discard stat

From: Chao Yu
Date: Tue Aug 26 2014 - 23:04:31 EST


Since we have supported handling discard request in this commit
f4659d8e620d08bd1a84a8aec5d2f5294a242764 (zram: support REQ_DISCARD), zram got
one more chance to free unused memory whenever received discard request. But
without stating for discard request, there is no method for user to know whether
discard request has been handled by zram or how many blocks were discarded by
zram when user wants to know the effect of discard.

In this patch, we add num_discard_req to stat discard request and add
num_discarded to stat real discarded blocks, and export them to sysfs for users.

* From v1
* Update zram document to show num_discards in statistics list.

* From v2
* Update description of this patch with clear goal.

* From v3
* Stat discard request and discarded pages separately as "previous stat
indicates lots of free page discarded without real freeing, so the stat makes
our user's misunderstanding" pointed out by Minchan Kim.

Signed-off-by: Chao Yu <chao2.yu@xxxxxxxxxxx>
---
Documentation/ABI/testing/sysfs-block-zram | 17 +++++++++++++++++
Documentation/blockdev/zram.txt | 2 ++
drivers/block/zram/zram_drv.c | 17 ++++++++++++++---
drivers/block/zram/zram_drv.h | 2 ++
4 files changed, 35 insertions(+), 3 deletions(-)

diff --git a/Documentation/ABI/testing/sysfs-block-zram b/Documentation/ABI/testing/sysfs-block-zram
index 70ec992..805fb11 100644
--- a/Documentation/ABI/testing/sysfs-block-zram
+++ b/Documentation/ABI/testing/sysfs-block-zram
@@ -57,6 +57,23 @@ Description:
The failed_writes file is read-only and specifies the number of
failed writes happened on this device.

+What: /sys/block/zram<id>/num_discard_req
+Date: August 2014
+Contact: Chao Yu <chao2.yu@xxxxxxxxxxx>
+Description:
+ The num_discard_req file is read-only and specifies the number
+ of requests received by this device. These requests are sent by
+ swap layer or filesystem when they want to free blocks which are
+ no longer used.
+
+What: /sys/block/zram<id>/num_discarded
+Date: August 2014
+Contact: Chao Yu <chao2.yu@xxxxxxxxxxx>
+Description:
+ The num_discarded file is read-only and specifies the number of
+ real discarded blocks (pages which are really freed) in this
+ device after discard request is sent to this device.
+
What: /sys/block/zram<id>/max_comp_streams
Date: February 2014
Contact: Sergey Senozhatsky <sergey.senozhatsky@xxxxxxxxx>
diff --git a/Documentation/blockdev/zram.txt b/Documentation/blockdev/zram.txt
index 0595c3f..f9c1e41 100644
--- a/Documentation/blockdev/zram.txt
+++ b/Documentation/blockdev/zram.txt
@@ -89,6 +89,8 @@ size of the disk when not in use so a huge zram is wasteful.
num_writes
failed_reads
failed_writes
+ num_discard_req
+ num_discarded
invalid_io
notify_free
zero_pages
diff --git a/drivers/block/zram/zram_drv.c b/drivers/block/zram/zram_drv.c
index d00831c..1d012e8 100644
--- a/drivers/block/zram/zram_drv.c
+++ b/drivers/block/zram/zram_drv.c
@@ -322,7 +322,7 @@ static void handle_zero_page(struct bio_vec *bvec)
* caller should hold this table index entry's bit_spinlock to
* indicate this index entry is accessing.
*/
-static void zram_free_page(struct zram *zram, size_t index)
+static bool zram_free_page(struct zram *zram, size_t index)
{
struct zram_meta *meta = zram->meta;
unsigned long handle = meta->table[index].handle;
@@ -336,7 +336,7 @@ static void zram_free_page(struct zram *zram, size_t index)
zram_clear_flag(meta, index, ZRAM_ZERO);
atomic64_dec(&zram->stats.zero_pages);
}
- return;
+ return false;
}

zs_free(meta->mem_pool, handle);
@@ -347,6 +347,7 @@ static void zram_free_page(struct zram *zram, size_t index)

meta->table[index].handle = 0;
zram_set_obj_size(meta, index, 0);
+ return true;
}

static int zram_decompress_page(struct zram *zram, char *mem, u32 index)
@@ -603,12 +604,18 @@ static void zram_bio_discard(struct zram *zram, u32 index,
}

while (n >= PAGE_SIZE) {
+ bool discarded;
+
bit_spin_lock(ZRAM_ACCESS, &meta->table[index].value);
- zram_free_page(zram, index);
+ discarded = zram_free_page(zram, index);
bit_spin_unlock(ZRAM_ACCESS, &meta->table[index].value);
+ if (discarded)
+ atomic64_inc(&zram->stats.num_discarded);
index++;
n -= PAGE_SIZE;
}
+
+ atomic64_inc(&zram->stats.num_discard_req);
}

static void zram_reset_device(struct zram *zram, bool reset_capacity)
@@ -866,6 +873,8 @@ ZRAM_ATTR_RO(num_reads);
ZRAM_ATTR_RO(num_writes);
ZRAM_ATTR_RO(failed_reads);
ZRAM_ATTR_RO(failed_writes);
+ZRAM_ATTR_RO(num_discard_req);
+ZRAM_ATTR_RO(num_discarded);
ZRAM_ATTR_RO(invalid_io);
ZRAM_ATTR_RO(notify_free);
ZRAM_ATTR_RO(zero_pages);
@@ -879,6 +888,8 @@ static struct attribute *zram_disk_attrs[] = {
&dev_attr_num_writes.attr,
&dev_attr_failed_reads.attr,
&dev_attr_failed_writes.attr,
+ &dev_attr_num_discard_req.attr,
+ &dev_attr_num_discarded.attr,
&dev_attr_invalid_io.attr,
&dev_attr_notify_free.attr,
&dev_attr_zero_pages.attr,
diff --git a/drivers/block/zram/zram_drv.h b/drivers/block/zram/zram_drv.h
index e0f725c..49f91aa 100644
--- a/drivers/block/zram/zram_drv.h
+++ b/drivers/block/zram/zram_drv.h
@@ -86,6 +86,8 @@ struct zram_stats {
atomic64_t num_writes; /* --do-- */
atomic64_t failed_reads; /* can happen when memory is too low */
atomic64_t failed_writes; /* can happen when memory is too low */
+ atomic64_t num_discard_req; /* no. of discard req */
+ atomic64_t num_discarded; /* no. of discarded pages */
atomic64_t invalid_io; /* non-page-aligned I/O requests */
atomic64_t notify_free; /* no. of swap slot free notifications */
atomic64_t zero_pages; /* no. of zero filled pages */
--
2.0.1.474.g72c7794


--
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/