On Thu, Jan 27, 2022 at 08:02:26AM +0000, Chaitanya Kulkarni wrote: > > + __field( unsigned int, nr_sector ) > > + __field( int, error ) > > + __array( char, rwbs, RWBS_LEN ) > > + ), > > + > > + TP_fast_assign( > > + __entry->dev = rq->q->disk ? disk_devt(rq->q->disk) : 0; > > + __assign_str(name, rq->q->disk ? rq->q->disk->disk_name : "?"); > > Did you verify that rq->q->disk NULL checks are must in this checkout > for blk_update_request() ? Yes, we need this.