diff mbox series

block: decode QUEUE_FLAG_HCTX_ACTIVE in debugfs output

Message ID 4351076388918075bd80ef07756f9d2ce63be12c.1633332053.git.johannes.thumshirn@wdc.com (mailing list archive)
State New, archived
Headers show
Series block: decode QUEUE_FLAG_HCTX_ACTIVE in debugfs output | expand

Commit Message

Johannes Thumshirn Oct. 4, 2021, 7:22 a.m. UTC
While debugging an issue we've found that $DEBUGFS/block/$disk/state
doesn't decode QUEUE_FLAG_HCTX_ACTIVE but only displays its numerical
value.

Add QUEUE_FLAG(HCTX_ACTIVE) to the blk_queue_flag_name array so it'll get
decoded properly.

Signed-off-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
---
 block/blk-mq-debugfs.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Jens Axboe Oct. 4, 2021, 12:59 p.m. UTC | #1
On 10/4/21 1:22 AM, Johannes Thumshirn wrote:
> While debugging an issue we've found that $DEBUGFS/block/$disk/state
> doesn't decode QUEUE_FLAG_HCTX_ACTIVE but only displays its numerical
> value.
> 
> Add QUEUE_FLAG(HCTX_ACTIVE) to the blk_queue_flag_name array so it'll get
> decoded properly.

Applied, thanks.

We really should have some way to catch this at build time, this isn't the
first (and won't be the last) time we run into this.
diff mbox series

Patch

diff --git a/block/blk-mq-debugfs.c b/block/blk-mq-debugfs.c
index 4b66d2776eda..3b38d15723de 100644
--- a/block/blk-mq-debugfs.c
+++ b/block/blk-mq-debugfs.c
@@ -129,6 +129,7 @@  static const char *const blk_queue_flag_name[] = {
 	QUEUE_FLAG_NAME(PCI_P2PDMA),
 	QUEUE_FLAG_NAME(ZONE_RESETALL),
 	QUEUE_FLAG_NAME(RQ_ALLOC_TIME),
+	QUEUE_FLAG_NAME(HCTX_ACTIVE),
 	QUEUE_FLAG_NAME(NOWAIT),
 };
 #undef QUEUE_FLAG_NAME