On Wed, Jun 28, 2023 at 08:45:44PM +0800, chengming.zhou@xxxxxxxxx wrote: > After these cleanup, __blk_mq_alloc_requests() is the only entry to > alloc and init rq. I find the code a little hard to follow now, due to the optional setting of the ctx. We also introduce really odd behavior here if the caller for a hctx-specific allocation doesn't have free tags, as we'll now run into the normal retry path. Is this really needed for your timestamp changes? If not I'd prefer to skip it.