On 12/22/25 3:28 PM, Jens Axboe wrote:

quoted

quoted

  	 */
  	strreplace(buts->name, '/', '_');
  
+	if (version == 1 && IS_ENABLED(CONFIG_BLK_DEV_ZONED)) {
+		pr_info("%s: blktrace events for REQ_OP_ZONE_XXX will be dropped\n",
+				name);
+		pr_info("use blktrace tools version >= 2 to track REQ_OP_ZONE_XXX\n");

Please change REQ_OP_ZONE_XXX to "zone operations" in these messages. That is a
little more general, so better I think since we also trace zone write
plug/unplug events, which are not REQ_OP_ZONE_XXX.

Agree, REQ_OP_ZONE_XXX means nothing in userspace.

-- Jens Axboe

Thanks Damien and Jens for comment.

Will send out V3 soon.

-ck