On Fri, Dec 20, 2013 at 06:37:27PM -0500, Mike Snitzer wrote: > +static void handle_unserviceable_bio(struct pool *pool, struct bio *bio) > +{ > + /* > + * When pool is read-only, no cell locking is needed because > + * nothing is changing. > + */ > + WARN_ON_ONCE(get_pool_mode(pool) != PM_READ_ONLY); > + > + if (pool->no_free_space) > + retry_on_resume(bio); > + else > + bio_io_error(bio); > +} Could we rename no_free_space please? (requeue_unserviceable_bios?) -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel