Messages in this thread | | | From | David Howells <> | Subject | Re: [PATCH v17 02/14] splice: Make do_splice_to() generic and export it | Date | Tue, 14 Mar 2023 21:15:54 +0000 |
| |
Christoph Hellwig <hch@infradead.org> wrote:
> > -static long do_splice_to(struct file *in, loff_t *ppos, > > - struct pipe_inode_info *pipe, size_t len, > > - unsigned int flags) > > +long vfs_splice_read(struct file *in, loff_t *ppos, > > The (pre-existing) long here is odd given that ->splice_read > returns a ssize_t. This might be a good time to fix that up.
There's a while lot of places in splice.c that use long rather than ssize_t. I can produce a separate patch that changes them all, but that should perhaps be separate to this.
David
| |