On Jul 30 00:49, Minwoo Im wrote:
> Klaus,
>
> On 20-07-20 13:37:38, Klaus Jensen wrote:
> > From: Klaus Jensen
> >
> > Pass along the NvmeRequest in various functions since it is very useful
> > for tracing.
>
> One doubt here.
> This patch has put NvmeRequest argument to the nvme_map_prp() to trace
> the request's command id. But can we just trace the cid before this
> kind of prp mapping, somewhere like nvme_process_sq() level. Then we
> can figure out the tracing for the prp mapping is from which request.
>
> Tracing for cid is definitely great, but feels like too much cost to
> pass argument to trace 'cid' in the middle of the dma mapping stage.
>
Good point Minwoo.
I ended up dropping the patch and just replacing it with a patch that
adds tracing to nvme_map_prp.