On 12/22/25 3:28 PM, Jens Axboe wrote:
>>> */
>>> 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
