On Mon, Jan 16, 2023 at 11:08:17PM +0000, David Howells wrote: > Use information other than the iterator direction to determine the > direction of the I/O: > > (*) If a kiocb is available, use the IOCB_WRITE flag. > > (*) If an iomap_iter is available, use the IOMAP_WRITE flag. > > (*) If a request is available, use op_is_write(). The really should be three independent patches. Plus another one to drop the debug checks in cifs. The changes themselves look good to me. > > +static unsigned char iov_iter_rw(const struct iov_iter *i) > +{ > + return i->data_source ? WRITE : READ; > +} It might as well make sense to just open code this in the only caller as well (yet another patch).