Re: [PATCH v13 03/12] splice: Do splice read from a buffered file without using ITER_PIPE

From: David Howells
Date: Mon Feb 13 2023 - 05:12:24 EST


Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote:

> Also why doesn't this use init_sync_kiocb?

I'm not sure I want ki_flags.

> > if (in->f_flags & O_DIRECT)
> > return generic_file_direct_splice_read(in, ppos, pipe, len, flags);
> > + return generic_file_buffered_splice_read(in, ppos, pipe, len, flags);
>
> Btw, can we drop the verbose generic_file_ prefix here?

Probably. Note that at some point cifs, for example, running in "unbuffered"
mode might want to call [generic_file_]direct_splice_read() directly.

David