> - if (WARN_ON(iomap.offset > pos)) > + if (WARN_ON(iomap.offset > pos) || WARN_ON(iomap.length == 0)) Btw, shouldn't this be one WARN_ON with both conditions inside? It's not like we can spot which one it was based on the output anyway. Alternatively just have two conditionals, which allows us to spot what went wrong. -- 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
