On 6/2/20 10:14 PM, Christoph Hellwig wrote: > The status can be trivially derived from the bio itself. That also avoid > callers like NVMe to incorrectly pass a blk_status_t instead of the errno, > and the overhead of translating the blk_status_t to the errno in the I/O > completion fast path when no tracing is enabled. > > Fixes: 35fe0d12c8a3 ("nvme: trace bio completion") > Signed-off-by: Christoph Hellwig<hch@xxxxxx> Looks good to me. Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@xxxxxxx>