On Wed, Jun 19, 2024 at 01:05:58AM -0700, Christoph Hellwig wrote: > On Wed, Jun 19, 2024 at 03:56:43PM +0800, Ming Lei wrote: > > If we add the check for all type of IO, it requires ->bi_sector to > > be meaningful for zero size bio. I am not sure if it is always true, > > such as RESET_ALL. > > meaningful or initialized to zero. Given that bio_init initializes it > to zero we should generally be fine (and are for BIO_OP_ZONE_RESET_ALL > for all callers in tree). Fine, let's fail this kind potential not-well-initialized bio, which is brittle anyway. Thanks, Ming