Christoph Hellwig <hch@xxxxxxxxxxxxx> writes: > On Sat, Apr 15, 2023 at 01:14:30PM +0530, Ritesh Harjani (IBM) wrote: >> Add iomap_dio_rw_queued and iomap_dio_complete tracepoint. >> iomap_dio_rw_queued is mostly only to know that the request was queued >> and -EIOCBQUEUED was returned. It is mostly iomap_dio_complete which has >> all the details. > > Everything that is here looks good to me. But it seems like we lost > the _begin tracepoint? Sorry, my bad, I might have only partially understood your review comment then. Will quickly send the next rev. So in the next rev. will only just add a _begin tracepoint in __iomap_dio_rw() function. Rest everything should be as is. Right?