> > - /* DIO must be aligned to device logical sector size */ > > - if ((iocb->ki_pos | count) & target->bt_logical_sectormask) { > > - if (iocb->ki_pos == isize) > > - return 0; > > - return -EINVAL; > > - } > > Do we not still need this, or is it checked elsewhere? > > The rest looks sane to me, but I need to test and I haven't grabbed your > tree yet.. iomap_dio_actor checks it. -- 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