> > - /* Skip holes. */ > + /* No holes. */ > if (iomap->type == IOMAP_HOLE) > - goto out; > + goto err; If we end up not allowing holes this check can just be removed entirely. We already check for the allow types below. I have to admit I would move that type check above the bdev check, though.