diff mbox

[2/2] blk-mq-debugfs: Show zone locking information

Message ID 20180228003214.3194-3-bart.vanassche@wdc.com (mailing list archive)
State New, archived
Headers show

Commit Message

Bart Van Assche Feb. 28, 2018, 12:32 a.m. UTC
When debugging the ZBC code in the mq-deadline scheduler it is very
important to know which zones are locked and which zones are not
locked. Hence this patch that exports the zone locking information
through debugfs.

Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>
Cc: Omar Sandoval <osandov@fb.com>
Cc: Damien Le Moal <damien.lemoal@wdc.com>
Cc: Ming Lei <ming.lei@redhat.com>
Cc: Hannes Reinecke <hare@suse.de>
Cc: Johannes Thumshirn <jthumshirn@suse.de>
---
 block/blk-mq-debugfs.c | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

Comments

Damien Le Moal Feb. 28, 2018, 1:52 a.m. UTC | #1
Bart,

On 2018/02/27 16:32, Bart Van Assche wrote:
> When debugging the ZBC code in the mq-deadline scheduler it is very

> important to know which zones are locked and which zones are not

> locked. Hence this patch that exports the zone locking information

> through debugfs.

> 

> Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>

> Cc: Omar Sandoval <osandov@fb.com>

> Cc: Damien Le Moal <damien.lemoal@wdc.com>

> Cc: Ming Lei <ming.lei@redhat.com>

> Cc: Hannes Reinecke <hare@suse.de>

> Cc: Johannes Thumshirn <jthumshirn@suse.de>


It would be nice to add the same to legacy deadline too.

For this patch:

Reviewed-by: Damien Le Moal <damien.lemoal@wdc.com>

Tested-by: Damien Le Moal <damien.lemoal@wdc.com>



-- 
Damien Le Moal
Western Digital Research
diff mbox

Patch

diff --git a/block/blk-mq-debugfs.c b/block/blk-mq-debugfs.c
index eaf282e239b0..1be2116e09b5 100644
--- a/block/blk-mq-debugfs.c
+++ b/block/blk-mq-debugfs.c
@@ -206,11 +206,27 @@  static ssize_t queue_write_hint_store(void *data, const char __user *buf,
 	return count;
 }
 
+static int queue_zone_wlock_show(void *data, struct seq_file *m)
+{
+	struct request_queue *q = data;
+	unsigned int i;
+
+	if (!q->seq_zones_wlock)
+		return 0;
+
+	for (i = 0; i < blk_queue_nr_zones(q); i++)
+		if (test_bit(i, q->seq_zones_wlock))
+			seq_printf(m, "%u\n", i);
+
+	return 0;
+}
+
 static const struct blk_mq_debugfs_attr blk_mq_debugfs_queue_attrs[] = {
 	{ "poll_stat", 0400, queue_poll_stat_show },
 	{ "requeue_list", 0400, .seq_ops = &queue_requeue_list_seq_ops },
 	{ "state", 0600, queue_state_show, queue_state_write },
 	{ "write_hints", 0600, queue_write_hint_show, queue_write_hint_store },
+	{ "zone_wlock", 0400, queue_zone_wlock_show, NULL },
 	{ },
 };