On Wed, May 16, 2018 at 09:45:50AM -0700, Omar Sandoval wrote:
> + if ((iomap->type != IOMAP_MAPPED && iomap->type != IOMAP_UNWRITTEN) ||
> + iomap->addr == IOMAP_NULL_ADDR) {
> + pr_err("swapon: file has unallocated extents\n");
> + return -EINVAL;
> + }
The two are really different cases - IOMAP_NULL_ADDR shouldn't really
happen for any of the above. Although we might have to move the
inline check before the type check above for the message to make sense.
I have a patch in the local queue that makes inline a type instead of
a flag, btw as it really isn't a flag.
--
To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
