On Wed, Apr 29, 2020 at 10:12:29PM +0800, Ming Lei wrote: > > +static int > > +loop_validate_size(loff_t size) > > +{ > > + if ((loff_t)(sector_t)size != size) > > + return -EFBIG; > > + > > + return 0; > > +} > > + > > Now sector_t has been switched to u64 unconditionally, do we still need such > validation? Oops, completely forgot about that. Yes, we can just kill the checks.