Skip to content

Commit

Permalink
blk-mq: expose write hints through debugfs
Browse files Browse the repository at this point in the history
Useful to verify that things are working the way they should.
Reading the file will return number of kb written with each
write hint. Writing the file will reset the statistics. No care
is taken to ensure that we don't race on updates.

Drivers will write to q->write_hints[] if they handle a given
write hint.

Reviewed-by: Andreas Dilger <[email protected]>
Reviewed-by: Martin K. Petersen <[email protected]>
Reviewed-by: Christoph Hellwig <[email protected]>
Signed-off-by: Jens Axboe <[email protected]>
  • Loading branch information
axboe committed Jun 27, 2017
1 parent cb6934f commit f793dfd
Show file tree
Hide file tree
Showing 2 changed files with 27 additions and 0 deletions.
24 changes: 24 additions & 0 deletions block/blk-mq-debugfs.c
Original file line number Diff line number Diff line change
Expand Up @@ -135,6 +135,29 @@ static void print_stat(struct seq_file *m, struct blk_rq_stat *stat)
}
}

static int queue_write_hint_show(void *data, struct seq_file *m)
{
struct request_queue *q = data;
int i;

for (i = 0; i < BLK_MAX_WRITE_HINTS; i++)
seq_printf(m, "hint%d: %llu\n", i, q->write_hints[i]);

return 0;
}

static ssize_t queue_write_hint_store(void *data, const char __user *buf,
size_t count, loff_t *ppos)
{
struct request_queue *q = data;
int i;

for (i = 0; i < BLK_MAX_WRITE_HINTS; i++)
q->write_hints[i] = 0;

return count;
}

static int queue_poll_stat_show(void *data, struct seq_file *m)
{
struct request_queue *q = data;
Expand Down Expand Up @@ -730,6 +753,7 @@ 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},
{},
};

Expand Down
3 changes: 3 additions & 0 deletions include/linux/blkdev.h
Original file line number Diff line number Diff line change
Expand Up @@ -596,6 +596,9 @@ struct request_queue {
void *rq_alloc_data;

struct work_struct release_work;

#define BLK_MAX_WRITE_HINTS 5
u64 write_hints[BLK_MAX_WRITE_HINTS];
};

#define QUEUE_FLAG_QUEUED 1 /* uses generic tag queueing */
Expand Down

0 comments on commit f793dfd

Please sign in to comment.