On 2020-05-29 09:03, Coly Li wrote: > +static inline unsigned int bio_aligned_discard_max_sectors( > + struct request_queue *q) > +{ > + return round_down(UINT_MAX, (q->limits.max_discard_sectors << 9)) >> 9; > +} How about changing '9' into 'SECTOR_SHIFT'? Thanks, Bart.