On Fri, Sep 23, 2016 at 12:03 PM, Al Viro <viro@xxxxxxxxxxxxxxxxxx> wrote:
> @@ -1421,8 +1406,25 @@ static long do_splice(struct file *in, loff_t __user *off_in,
> + ret = 0;
> + pipe_lock(opipe);
> + bogus_count = opipe->buffers;
> + do {
> + bogus_count += opipe->nrbufs;
> + ret = do_splice_to(in, &offset, opipe, len, flags);
> + if (ret > 0) {
> + total += ret;
> + len -= ret;
> + }
> + bogus_count -= opipe->nrbufs;
> + if (bogus_count <= 0)
> + break;
I was like "oh, I'm sure this is some temporary hack, it will be gone
by the end of the series".
It wasn't gone by the end.
There's two copies of that pattern, and at the very least it needs a
big comment about what this pattern does and why.
But other than that reaction, I didn't get any hives from this. I
didn't *test* it, only looking at patches, but no red flags I could
notice.
Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
