On Mon, Mar 25, 2019 at 05:10:20AM +0000, Chaitanya Kulkarni wrote: > > +/* > > + * Return the first bvec that contains integrity data. In general only > > + * drivers that are limited to a single integrity segment should use this > > + * helper. > > + */ > > +static inline struct bio_vec *rq_integrity_vec(struct request *rq) > > +{ > Wrt comment, should we add a check here to make sure underlaying driver > has limited single integrity segment ? I've added a WARN_ON_ONCE.