Re: [PATCH] blk-mq: fix the cmd_flag_name array

[Bart Van Assche]

On Thu, 2019-01-24 at 09:22 -0700, Jens Axboe wrote:
+AD4 On 1/24/19 8:09 AM, Jeff Moyer wrote:
+AD4 +AD4 Jianchao Wang +ADw-jianchao.w.wang+AEA-oracle.com+AD4 writes:
+AD4 +AD4 
+AD4 +AD4 +AD4 Swap REQ+AF8-NOWAIT and REQ+AF8-NOUNMAP and add REQ+AF8-HIPRI.
+AD4 +AD4 +AD4 
+AD4 +AD4 +AD4 Signed-off-by: Jianchao Wang +ADw-jianchao.w.wang+AEA-oracle.com+AD4
+AD4 +AD4 +AD4 ---
+AD4 +AD4 +AD4  block/blk-mq-debugfs.c +AHw 3 +-+--
+AD4 +AD4 +AD4  1 file changed, 2 insertions(+-), 1 deletion(-)
+AD4 +AD4 +AD4 
+AD4 +AD4 +AD4 diff --git a/block/blk-mq-debugfs.c b/block/blk-mq-debugfs.c
+AD4 +AD4 +AD4 index 90d6876..f812083 100644
+AD4 +AD4 +AD4 --- a/block/blk-mq-debugfs.c
+AD4 +AD4 +AD4 +-+-+- b/block/blk-mq-debugfs.c
+AD4 +AD4 +AD4 +AEAAQA -308,8 +-308,9 +AEAAQA static const char +ACo-const cmd+AF8-flag+AF8-name+AFsAXQ +AD0 +AHs
+AD4 +AD4 +AD4  	CMD+AF8-FLAG+AF8-NAME(PREFLUSH),
+AD4 +AD4 +AD4  	CMD+AF8-FLAG+AF8-NAME(RAHEAD),
+AD4 +AD4 +AD4  	CMD+AF8-FLAG+AF8-NAME(BACKGROUND),
+AD4 +AD4 +AD4 -	CMD+AF8-FLAG+AF8-NAME(NOUNMAP),
+AD4 +AD4 +AD4  	CMD+AF8-FLAG+AF8-NAME(NOWAIT),
+AD4 +AD4 +AD4 +-	CMD+AF8-FLAG+AF8-NAME(NOUNMAP),
+AD4 +AD4 +AD4 +-	CMD+AF8-FLAG+AF8-NAME(HIPRI),
+AD4 +AD4 +AD4  +AH0AOw
+AD4 +AD4 +AD4  +ACM-undef CMD+AF8-FLAG+AF8-NAME
+AD4 +AD4 
+AD4 +AD4 Acked-by: Jeff Moyer +ADw-jmoyer+AEA-redhat.com+AD4
+AD4 +AD4 
+AD4 +AD4 You might consider also adding a comment above the req+AF8-flag+AF8-bits enum
+AD4 +AD4 noting that modifications also need to be propagated to cmd+AF8-flag+AF8-name.
+AD4 
+AD4 Agree... These things are notoriously difficult to keep in sync, this
+AD4 isn't the first time we've had a fixup. Not sure if that situation
+AD4 is improvable, but at least a comment might help.

How about making the build system derive the CMD+AF8-FLAG+AF8-NAME() declarations
from the include/linux/blk+ACo header files? It's not that hard to do that e.g.
with the help of sed. The output of sed could be written into a .h file and
that file could be +ACM-included from blk-mq-debugfs.c.

Bart.