| .. | .. |
|---|
| 1 | +// SPDX-License-Identifier: GPL-2.0-only |
|---|
| 1 | 2 | /* |
|---|
| 2 | 3 | * "splice": joining two ropes together by interweaving their strands. |
|---|
| 3 | 4 | * |
|---|
| .. | .. |
|---|
| 32 | 33 | #include <linux/security.h> |
|---|
| 33 | 34 | #include <linux/gfp.h> |
|---|
| 34 | 35 | #include <linux/socket.h> |
|---|
| 35 | | -#include <linux/compat.h> |
|---|
| 36 | 36 | #include <linux/sched/signal.h> |
|---|
| 37 | 37 | |
|---|
| 38 | 38 | #include "internal.h" |
|---|
| .. | .. |
|---|
| 43 | 43 | * addition of remove_mapping(). If success is returned, the caller may |
|---|
| 44 | 44 | * attempt to reuse this page for another destination. |
|---|
| 45 | 45 | */ |
|---|
| 46 | | -static int page_cache_pipe_buf_steal(struct pipe_inode_info *pipe, |
|---|
| 47 | | - struct pipe_buffer *buf) |
|---|
| 46 | +static bool page_cache_pipe_buf_try_steal(struct pipe_inode_info *pipe, |
|---|
| 47 | + struct pipe_buffer *buf) |
|---|
| 48 | 48 | { |
|---|
| 49 | 49 | struct page *page = buf->page; |
|---|
| 50 | 50 | struct address_space *mapping; |
|---|
| .. | .. |
|---|
| 75 | 75 | */ |
|---|
| 76 | 76 | if (remove_mapping(mapping, page)) { |
|---|
| 77 | 77 | buf->flags |= PIPE_BUF_FLAG_LRU; |
|---|
| 78 | | - return 0; |
|---|
| 78 | + return true; |
|---|
| 79 | 79 | } |
|---|
| 80 | 80 | } |
|---|
| 81 | 81 | |
|---|
| .. | .. |
|---|
| 85 | 85 | */ |
|---|
| 86 | 86 | out_unlock: |
|---|
| 87 | 87 | unlock_page(page); |
|---|
| 88 | | - return 1; |
|---|
| 88 | + return false; |
|---|
| 89 | 89 | } |
|---|
| 90 | 90 | |
|---|
| 91 | 91 | static void page_cache_pipe_buf_release(struct pipe_inode_info *pipe, |
|---|
| .. | .. |
|---|
| 138 | 138 | } |
|---|
| 139 | 139 | |
|---|
| 140 | 140 | const struct pipe_buf_operations page_cache_pipe_buf_ops = { |
|---|
| 141 | | - .can_merge = 0, |
|---|
| 142 | | - .confirm = page_cache_pipe_buf_confirm, |
|---|
| 143 | | - .release = page_cache_pipe_buf_release, |
|---|
| 144 | | - .steal = page_cache_pipe_buf_steal, |
|---|
| 145 | | - .get = generic_pipe_buf_get, |
|---|
| 141 | + .confirm = page_cache_pipe_buf_confirm, |
|---|
| 142 | + .release = page_cache_pipe_buf_release, |
|---|
| 143 | + .try_steal = page_cache_pipe_buf_try_steal, |
|---|
| 144 | + .get = generic_pipe_buf_get, |
|---|
| 146 | 145 | }; |
|---|
| 147 | 146 | |
|---|
| 148 | | -static int user_page_pipe_buf_steal(struct pipe_inode_info *pipe, |
|---|
| 149 | | - struct pipe_buffer *buf) |
|---|
| 147 | +static bool user_page_pipe_buf_try_steal(struct pipe_inode_info *pipe, |
|---|
| 148 | + struct pipe_buffer *buf) |
|---|
| 150 | 149 | { |
|---|
| 151 | 150 | if (!(buf->flags & PIPE_BUF_FLAG_GIFT)) |
|---|
| 152 | | - return 1; |
|---|
| 151 | + return false; |
|---|
| 153 | 152 | |
|---|
| 154 | 153 | buf->flags |= PIPE_BUF_FLAG_LRU; |
|---|
| 155 | | - return generic_pipe_buf_steal(pipe, buf); |
|---|
| 154 | + return generic_pipe_buf_try_steal(pipe, buf); |
|---|
| 156 | 155 | } |
|---|
| 157 | 156 | |
|---|
| 158 | 157 | static const struct pipe_buf_operations user_page_pipe_buf_ops = { |
|---|
| 159 | | - .can_merge = 0, |
|---|
| 160 | | - .confirm = generic_pipe_buf_confirm, |
|---|
| 161 | | - .release = page_cache_pipe_buf_release, |
|---|
| 162 | | - .steal = user_page_pipe_buf_steal, |
|---|
| 163 | | - .get = generic_pipe_buf_get, |
|---|
| 158 | + .release = page_cache_pipe_buf_release, |
|---|
| 159 | + .try_steal = user_page_pipe_buf_try_steal, |
|---|
| 160 | + .get = generic_pipe_buf_get, |
|---|
| 164 | 161 | }; |
|---|
| 165 | 162 | |
|---|
| 166 | 163 | static void wakeup_pipe_readers(struct pipe_inode_info *pipe) |
|---|
| 167 | 164 | { |
|---|
| 168 | 165 | smp_mb(); |
|---|
| 169 | | - if (waitqueue_active(&pipe->wait)) |
|---|
| 170 | | - wake_up_interruptible(&pipe->wait); |
|---|
| 166 | + if (waitqueue_active(&pipe->rd_wait)) |
|---|
| 167 | + wake_up_interruptible(&pipe->rd_wait); |
|---|
| 171 | 168 | kill_fasync(&pipe->fasync_readers, SIGIO, POLL_IN); |
|---|
| 172 | 169 | } |
|---|
| 173 | 170 | |
|---|
| .. | .. |
|---|
| 186 | 183 | struct splice_pipe_desc *spd) |
|---|
| 187 | 184 | { |
|---|
| 188 | 185 | unsigned int spd_pages = spd->nr_pages; |
|---|
| 186 | + unsigned int tail = pipe->tail; |
|---|
| 187 | + unsigned int head = pipe->head; |
|---|
| 188 | + unsigned int mask = pipe->ring_size - 1; |
|---|
| 189 | 189 | int ret = 0, page_nr = 0; |
|---|
| 190 | 190 | |
|---|
| 191 | 191 | if (!spd_pages) |
|---|
| .. | .. |
|---|
| 197 | 197 | goto out; |
|---|
| 198 | 198 | } |
|---|
| 199 | 199 | |
|---|
| 200 | | - while (pipe->nrbufs < pipe->buffers) { |
|---|
| 201 | | - int newbuf = (pipe->curbuf + pipe->nrbufs) & (pipe->buffers - 1); |
|---|
| 202 | | - struct pipe_buffer *buf = pipe->bufs + newbuf; |
|---|
| 200 | + while (!pipe_full(head, tail, pipe->max_usage)) { |
|---|
| 201 | + struct pipe_buffer *buf = &pipe->bufs[head & mask]; |
|---|
| 203 | 202 | |
|---|
| 204 | 203 | buf->page = spd->pages[page_nr]; |
|---|
| 205 | 204 | buf->offset = spd->partial[page_nr].offset; |
|---|
| .. | .. |
|---|
| 208 | 207 | buf->ops = spd->ops; |
|---|
| 209 | 208 | buf->flags = 0; |
|---|
| 210 | 209 | |
|---|
| 211 | | - pipe->nrbufs++; |
|---|
| 210 | + head++; |
|---|
| 211 | + pipe->head = head; |
|---|
| 212 | 212 | page_nr++; |
|---|
| 213 | 213 | ret += buf->len; |
|---|
| 214 | 214 | |
|---|
| .. | .. |
|---|
| 229 | 229 | |
|---|
| 230 | 230 | ssize_t add_to_pipe(struct pipe_inode_info *pipe, struct pipe_buffer *buf) |
|---|
| 231 | 231 | { |
|---|
| 232 | + unsigned int head = pipe->head; |
|---|
| 233 | + unsigned int tail = pipe->tail; |
|---|
| 234 | + unsigned int mask = pipe->ring_size - 1; |
|---|
| 232 | 235 | int ret; |
|---|
| 233 | 236 | |
|---|
| 234 | 237 | if (unlikely(!pipe->readers)) { |
|---|
| 235 | 238 | send_sig(SIGPIPE, current, 0); |
|---|
| 236 | 239 | ret = -EPIPE; |
|---|
| 237 | | - } else if (pipe->nrbufs == pipe->buffers) { |
|---|
| 240 | + } else if (pipe_full(head, tail, pipe->max_usage)) { |
|---|
| 238 | 241 | ret = -EAGAIN; |
|---|
| 239 | 242 | } else { |
|---|
| 240 | | - int newbuf = (pipe->curbuf + pipe->nrbufs) & (pipe->buffers - 1); |
|---|
| 241 | | - pipe->bufs[newbuf] = *buf; |
|---|
| 242 | | - pipe->nrbufs++; |
|---|
| 243 | + pipe->bufs[head & mask] = *buf; |
|---|
| 244 | + pipe->head = head + 1; |
|---|
| 243 | 245 | return buf->len; |
|---|
| 244 | 246 | } |
|---|
| 245 | 247 | pipe_buf_release(pipe, buf); |
|---|
| .. | .. |
|---|
| 253 | 255 | */ |
|---|
| 254 | 256 | int splice_grow_spd(const struct pipe_inode_info *pipe, struct splice_pipe_desc *spd) |
|---|
| 255 | 257 | { |
|---|
| 256 | | - unsigned int buffers = READ_ONCE(pipe->buffers); |
|---|
| 258 | + unsigned int max_usage = READ_ONCE(pipe->max_usage); |
|---|
| 257 | 259 | |
|---|
| 258 | | - spd->nr_pages_max = buffers; |
|---|
| 259 | | - if (buffers <= PIPE_DEF_BUFFERS) |
|---|
| 260 | + spd->nr_pages_max = max_usage; |
|---|
| 261 | + if (max_usage <= PIPE_DEF_BUFFERS) |
|---|
| 260 | 262 | return 0; |
|---|
| 261 | 263 | |
|---|
| 262 | | - spd->pages = kmalloc_array(buffers, sizeof(struct page *), GFP_KERNEL); |
|---|
| 263 | | - spd->partial = kmalloc_array(buffers, sizeof(struct partial_page), |
|---|
| 264 | + spd->pages = kmalloc_array(max_usage, sizeof(struct page *), GFP_KERNEL); |
|---|
| 265 | + spd->partial = kmalloc_array(max_usage, sizeof(struct partial_page), |
|---|
| 264 | 266 | GFP_KERNEL); |
|---|
| 265 | 267 | |
|---|
| 266 | 268 | if (spd->pages && spd->partial) |
|---|
| .. | .. |
|---|
| 299 | 301 | { |
|---|
| 300 | 302 | struct iov_iter to; |
|---|
| 301 | 303 | struct kiocb kiocb; |
|---|
| 302 | | - int idx, ret; |
|---|
| 304 | + unsigned int i_head; |
|---|
| 305 | + int ret; |
|---|
| 303 | 306 | |
|---|
| 304 | | - iov_iter_pipe(&to, ITER_PIPE | READ, pipe, len); |
|---|
| 305 | | - idx = to.idx; |
|---|
| 307 | + iov_iter_pipe(&to, READ, pipe, len); |
|---|
| 308 | + i_head = to.head; |
|---|
| 306 | 309 | init_sync_kiocb(&kiocb, in); |
|---|
| 307 | 310 | kiocb.ki_pos = *ppos; |
|---|
| 308 | 311 | ret = call_read_iter(in, &kiocb, &to); |
|---|
| .. | .. |
|---|
| 310 | 313 | *ppos = kiocb.ki_pos; |
|---|
| 311 | 314 | file_accessed(in); |
|---|
| 312 | 315 | } else if (ret < 0) { |
|---|
| 313 | | - to.idx = idx; |
|---|
| 316 | + to.head = i_head; |
|---|
| 314 | 317 | to.iov_offset = 0; |
|---|
| 315 | 318 | iov_iter_advance(&to, 0); /* to free what was emitted */ |
|---|
| 316 | 319 | /* |
|---|
| .. | .. |
|---|
| 323 | 326 | |
|---|
| 324 | 327 | return ret; |
|---|
| 325 | 328 | } |
|---|
| 326 | | -EXPORT_SYMBOL(generic_file_splice_read); |
|---|
| 329 | +EXPORT_SYMBOL_NS(generic_file_splice_read, ANDROID_GKI_VFS_EXPORT_ONLY); |
|---|
| 327 | 330 | |
|---|
| 328 | 331 | const struct pipe_buf_operations default_pipe_buf_ops = { |
|---|
| 329 | | - .can_merge = 0, |
|---|
| 330 | | - .confirm = generic_pipe_buf_confirm, |
|---|
| 331 | | - .release = generic_pipe_buf_release, |
|---|
| 332 | | - .steal = generic_pipe_buf_steal, |
|---|
| 333 | | - .get = generic_pipe_buf_get, |
|---|
| 332 | + .release = generic_pipe_buf_release, |
|---|
| 333 | + .try_steal = generic_pipe_buf_try_steal, |
|---|
| 334 | + .get = generic_pipe_buf_get, |
|---|
| 334 | 335 | }; |
|---|
| 335 | | - |
|---|
| 336 | | -int generic_pipe_buf_nosteal(struct pipe_inode_info *pipe, |
|---|
| 337 | | - struct pipe_buffer *buf) |
|---|
| 338 | | -{ |
|---|
| 339 | | - return 1; |
|---|
| 340 | | -} |
|---|
| 341 | 336 | |
|---|
| 342 | 337 | /* Pipe buffer operations for a socket and similar. */ |
|---|
| 343 | 338 | const struct pipe_buf_operations nosteal_pipe_buf_ops = { |
|---|
| 344 | | - .can_merge = 0, |
|---|
| 345 | | - .confirm = generic_pipe_buf_confirm, |
|---|
| 346 | | - .release = generic_pipe_buf_release, |
|---|
| 347 | | - .steal = generic_pipe_buf_nosteal, |
|---|
| 348 | | - .get = generic_pipe_buf_get, |
|---|
| 339 | + .release = generic_pipe_buf_release, |
|---|
| 340 | + .get = generic_pipe_buf_get, |
|---|
| 349 | 341 | }; |
|---|
| 350 | 342 | EXPORT_SYMBOL(nosteal_pipe_buf_ops); |
|---|
| 351 | | - |
|---|
| 352 | | -static ssize_t kernel_readv(struct file *file, const struct kvec *vec, |
|---|
| 353 | | - unsigned long vlen, loff_t offset) |
|---|
| 354 | | -{ |
|---|
| 355 | | - mm_segment_t old_fs; |
|---|
| 356 | | - loff_t pos = offset; |
|---|
| 357 | | - ssize_t res; |
|---|
| 358 | | - |
|---|
| 359 | | - old_fs = get_fs(); |
|---|
| 360 | | - set_fs(get_ds()); |
|---|
| 361 | | - /* The cast to a user pointer is valid due to the set_fs() */ |
|---|
| 362 | | - res = vfs_readv(file, (const struct iovec __user *)vec, vlen, &pos, 0); |
|---|
| 363 | | - set_fs(old_fs); |
|---|
| 364 | | - |
|---|
| 365 | | - return res; |
|---|
| 366 | | -} |
|---|
| 367 | | - |
|---|
| 368 | | -static ssize_t default_file_splice_read(struct file *in, loff_t *ppos, |
|---|
| 369 | | - struct pipe_inode_info *pipe, size_t len, |
|---|
| 370 | | - unsigned int flags) |
|---|
| 371 | | -{ |
|---|
| 372 | | - struct kvec *vec, __vec[PIPE_DEF_BUFFERS]; |
|---|
| 373 | | - struct iov_iter to; |
|---|
| 374 | | - struct page **pages; |
|---|
| 375 | | - unsigned int nr_pages; |
|---|
| 376 | | - size_t offset, base, copied = 0; |
|---|
| 377 | | - ssize_t res; |
|---|
| 378 | | - int i; |
|---|
| 379 | | - |
|---|
| 380 | | - if (pipe->nrbufs == pipe->buffers) |
|---|
| 381 | | - return -EAGAIN; |
|---|
| 382 | | - |
|---|
| 383 | | - /* |
|---|
| 384 | | - * Try to keep page boundaries matching to source pagecache ones - |
|---|
| 385 | | - * it probably won't be much help, but... |
|---|
| 386 | | - */ |
|---|
| 387 | | - offset = *ppos & ~PAGE_MASK; |
|---|
| 388 | | - |
|---|
| 389 | | - iov_iter_pipe(&to, ITER_PIPE | READ, pipe, len + offset); |
|---|
| 390 | | - |
|---|
| 391 | | - res = iov_iter_get_pages_alloc(&to, &pages, len + offset, &base); |
|---|
| 392 | | - if (res <= 0) |
|---|
| 393 | | - return -ENOMEM; |
|---|
| 394 | | - |
|---|
| 395 | | - nr_pages = DIV_ROUND_UP(res + base, PAGE_SIZE); |
|---|
| 396 | | - |
|---|
| 397 | | - vec = __vec; |
|---|
| 398 | | - if (nr_pages > PIPE_DEF_BUFFERS) { |
|---|
| 399 | | - vec = kmalloc_array(nr_pages, sizeof(struct kvec), GFP_KERNEL); |
|---|
| 400 | | - if (unlikely(!vec)) { |
|---|
| 401 | | - res = -ENOMEM; |
|---|
| 402 | | - goto out; |
|---|
| 403 | | - } |
|---|
| 404 | | - } |
|---|
| 405 | | - |
|---|
| 406 | | - pipe->bufs[to.idx].offset = offset; |
|---|
| 407 | | - pipe->bufs[to.idx].len -= offset; |
|---|
| 408 | | - |
|---|
| 409 | | - for (i = 0; i < nr_pages; i++) { |
|---|
| 410 | | - size_t this_len = min_t(size_t, len, PAGE_SIZE - offset); |
|---|
| 411 | | - vec[i].iov_base = page_address(pages[i]) + offset; |
|---|
| 412 | | - vec[i].iov_len = this_len; |
|---|
| 413 | | - len -= this_len; |
|---|
| 414 | | - offset = 0; |
|---|
| 415 | | - } |
|---|
| 416 | | - |
|---|
| 417 | | - res = kernel_readv(in, vec, nr_pages, *ppos); |
|---|
| 418 | | - if (res > 0) { |
|---|
| 419 | | - copied = res; |
|---|
| 420 | | - *ppos += res; |
|---|
| 421 | | - } |
|---|
| 422 | | - |
|---|
| 423 | | - if (vec != __vec) |
|---|
| 424 | | - kfree(vec); |
|---|
| 425 | | -out: |
|---|
| 426 | | - for (i = 0; i < nr_pages; i++) |
|---|
| 427 | | - put_page(pages[i]); |
|---|
| 428 | | - kvfree(pages); |
|---|
| 429 | | - iov_iter_advance(&to, copied); /* truncates and discards */ |
|---|
| 430 | | - return res; |
|---|
| 431 | | -} |
|---|
| 432 | 343 | |
|---|
| 433 | 344 | /* |
|---|
| 434 | 345 | * Send 'sd->len' bytes to socket from 'sd->file' at position 'sd->pos' |
|---|
| .. | .. |
|---|
| 446 | 357 | |
|---|
| 447 | 358 | more = (sd->flags & SPLICE_F_MORE) ? MSG_MORE : 0; |
|---|
| 448 | 359 | |
|---|
| 449 | | - if (sd->len < sd->total_len && pipe->nrbufs > 1) |
|---|
| 360 | + if (sd->len < sd->total_len && |
|---|
| 361 | + pipe_occupancy(pipe->head, pipe->tail) > 1) |
|---|
| 450 | 362 | more |= MSG_SENDPAGE_NOTLAST; |
|---|
| 451 | 363 | |
|---|
| 452 | 364 | return file->f_op->sendpage(file, buf->page, buf->offset, |
|---|
| .. | .. |
|---|
| 456 | 368 | static void wakeup_pipe_writers(struct pipe_inode_info *pipe) |
|---|
| 457 | 369 | { |
|---|
| 458 | 370 | smp_mb(); |
|---|
| 459 | | - if (waitqueue_active(&pipe->wait)) |
|---|
| 460 | | - wake_up_interruptible(&pipe->wait); |
|---|
| 371 | + if (waitqueue_active(&pipe->wr_wait)) |
|---|
| 372 | + wake_up_interruptible(&pipe->wr_wait); |
|---|
| 461 | 373 | kill_fasync(&pipe->fasync_writers, SIGIO, POLL_OUT); |
|---|
| 462 | 374 | } |
|---|
| 463 | 375 | |
|---|
| .. | .. |
|---|
| 484 | 396 | static int splice_from_pipe_feed(struct pipe_inode_info *pipe, struct splice_desc *sd, |
|---|
| 485 | 397 | splice_actor *actor) |
|---|
| 486 | 398 | { |
|---|
| 399 | + unsigned int head = pipe->head; |
|---|
| 400 | + unsigned int tail = pipe->tail; |
|---|
| 401 | + unsigned int mask = pipe->ring_size - 1; |
|---|
| 487 | 402 | int ret; |
|---|
| 488 | 403 | |
|---|
| 489 | | - while (pipe->nrbufs) { |
|---|
| 490 | | - struct pipe_buffer *buf = pipe->bufs + pipe->curbuf; |
|---|
| 404 | + while (!pipe_empty(head, tail)) { |
|---|
| 405 | + struct pipe_buffer *buf = &pipe->bufs[tail & mask]; |
|---|
| 491 | 406 | |
|---|
| 492 | 407 | sd->len = buf->len; |
|---|
| 493 | 408 | if (sd->len > sd->total_len) |
|---|
| .. | .. |
|---|
| 514 | 429 | |
|---|
| 515 | 430 | if (!buf->len) { |
|---|
| 516 | 431 | pipe_buf_release(pipe, buf); |
|---|
| 517 | | - pipe->curbuf = (pipe->curbuf + 1) & (pipe->buffers - 1); |
|---|
| 518 | | - pipe->nrbufs--; |
|---|
| 432 | + tail++; |
|---|
| 433 | + pipe->tail = tail; |
|---|
| 519 | 434 | if (pipe->files) |
|---|
| 520 | 435 | sd->need_wakeup = true; |
|---|
| 521 | 436 | } |
|---|
| .. | .. |
|---|
| 525 | 440 | } |
|---|
| 526 | 441 | |
|---|
| 527 | 442 | return 1; |
|---|
| 443 | +} |
|---|
| 444 | + |
|---|
| 445 | +/* We know we have a pipe buffer, but maybe it's empty? */ |
|---|
| 446 | +static inline bool eat_empty_buffer(struct pipe_inode_info *pipe) |
|---|
| 447 | +{ |
|---|
| 448 | + unsigned int tail = pipe->tail; |
|---|
| 449 | + unsigned int mask = pipe->ring_size - 1; |
|---|
| 450 | + struct pipe_buffer *buf = &pipe->bufs[tail & mask]; |
|---|
| 451 | + |
|---|
| 452 | + if (unlikely(!buf->len)) { |
|---|
| 453 | + pipe_buf_release(pipe, buf); |
|---|
| 454 | + pipe->tail = tail+1; |
|---|
| 455 | + return true; |
|---|
| 456 | + } |
|---|
| 457 | + |
|---|
| 458 | + return false; |
|---|
| 528 | 459 | } |
|---|
| 529 | 460 | |
|---|
| 530 | 461 | /** |
|---|
| .. | .. |
|---|
| 546 | 477 | if (signal_pending(current)) |
|---|
| 547 | 478 | return -ERESTARTSYS; |
|---|
| 548 | 479 | |
|---|
| 549 | | - while (!pipe->nrbufs) { |
|---|
| 480 | +repeat: |
|---|
| 481 | + while (pipe_empty(pipe->head, pipe->tail)) { |
|---|
| 550 | 482 | if (!pipe->writers) |
|---|
| 551 | 483 | return 0; |
|---|
| 552 | 484 | |
|---|
| 553 | | - if (!pipe->waiting_writers && sd->num_spliced) |
|---|
| 485 | + if (sd->num_spliced) |
|---|
| 554 | 486 | return 0; |
|---|
| 555 | 487 | |
|---|
| 556 | 488 | if (sd->flags & SPLICE_F_NONBLOCK) |
|---|
| .. | .. |
|---|
| 564 | 496 | sd->need_wakeup = false; |
|---|
| 565 | 497 | } |
|---|
| 566 | 498 | |
|---|
| 567 | | - pipe_wait(pipe); |
|---|
| 499 | + pipe_wait_readable(pipe); |
|---|
| 568 | 500 | } |
|---|
| 501 | + |
|---|
| 502 | + if (eat_empty_buffer(pipe)) |
|---|
| 503 | + goto repeat; |
|---|
| 569 | 504 | |
|---|
| 570 | 505 | return 1; |
|---|
| 571 | 506 | } |
|---|
| .. | .. |
|---|
| 689 | 624 | .pos = *ppos, |
|---|
| 690 | 625 | .u.file = out, |
|---|
| 691 | 626 | }; |
|---|
| 692 | | - int nbufs = pipe->buffers; |
|---|
| 627 | + int nbufs = pipe->max_usage; |
|---|
| 693 | 628 | struct bio_vec *array = kcalloc(nbufs, sizeof(struct bio_vec), |
|---|
| 694 | 629 | GFP_KERNEL); |
|---|
| 695 | 630 | ssize_t ret; |
|---|
| .. | .. |
|---|
| 702 | 637 | splice_from_pipe_begin(&sd); |
|---|
| 703 | 638 | while (sd.total_len) { |
|---|
| 704 | 639 | struct iov_iter from; |
|---|
| 640 | + unsigned int head, tail, mask; |
|---|
| 705 | 641 | size_t left; |
|---|
| 706 | | - int n, idx; |
|---|
| 642 | + int n; |
|---|
| 707 | 643 | |
|---|
| 708 | 644 | ret = splice_from_pipe_next(pipe, &sd); |
|---|
| 709 | 645 | if (ret <= 0) |
|---|
| 710 | 646 | break; |
|---|
| 711 | 647 | |
|---|
| 712 | | - if (unlikely(nbufs < pipe->buffers)) { |
|---|
| 648 | + if (unlikely(nbufs < pipe->max_usage)) { |
|---|
| 713 | 649 | kfree(array); |
|---|
| 714 | | - nbufs = pipe->buffers; |
|---|
| 650 | + nbufs = pipe->max_usage; |
|---|
| 715 | 651 | array = kcalloc(nbufs, sizeof(struct bio_vec), |
|---|
| 716 | 652 | GFP_KERNEL); |
|---|
| 717 | 653 | if (!array) { |
|---|
| .. | .. |
|---|
| 720 | 656 | } |
|---|
| 721 | 657 | } |
|---|
| 722 | 658 | |
|---|
| 659 | + head = pipe->head; |
|---|
| 660 | + tail = pipe->tail; |
|---|
| 661 | + mask = pipe->ring_size - 1; |
|---|
| 662 | + |
|---|
| 723 | 663 | /* build the vector */ |
|---|
| 724 | 664 | left = sd.total_len; |
|---|
| 725 | | - for (n = 0, idx = pipe->curbuf; left && n < pipe->nrbufs; n++, idx++) { |
|---|
| 726 | | - struct pipe_buffer *buf = pipe->bufs + idx; |
|---|
| 665 | + for (n = 0; !pipe_empty(head, tail) && left && n < nbufs; tail++, n++) { |
|---|
| 666 | + struct pipe_buffer *buf = &pipe->bufs[tail & mask]; |
|---|
| 727 | 667 | size_t this_len = buf->len; |
|---|
| 728 | 668 | |
|---|
| 729 | 669 | if (this_len > left) |
|---|
| 730 | 670 | this_len = left; |
|---|
| 731 | | - |
|---|
| 732 | | - if (idx == pipe->buffers - 1) |
|---|
| 733 | | - idx = -1; |
|---|
| 734 | 671 | |
|---|
| 735 | 672 | ret = pipe_buf_confirm(pipe, buf); |
|---|
| 736 | 673 | if (unlikely(ret)) { |
|---|
| .. | .. |
|---|
| 745 | 682 | left -= this_len; |
|---|
| 746 | 683 | } |
|---|
| 747 | 684 | |
|---|
| 748 | | - iov_iter_bvec(&from, ITER_BVEC | WRITE, array, n, |
|---|
| 749 | | - sd.total_len - left); |
|---|
| 685 | + iov_iter_bvec(&from, WRITE, array, n, sd.total_len - left); |
|---|
| 750 | 686 | ret = vfs_iter_write(out, &from, &sd.pos, 0); |
|---|
| 751 | 687 | if (ret <= 0) |
|---|
| 752 | 688 | break; |
|---|
| .. | .. |
|---|
| 756 | 692 | *ppos = sd.pos; |
|---|
| 757 | 693 | |
|---|
| 758 | 694 | /* dismiss the fully eaten buffers, adjust the partial one */ |
|---|
| 695 | + tail = pipe->tail; |
|---|
| 759 | 696 | while (ret) { |
|---|
| 760 | | - struct pipe_buffer *buf = pipe->bufs + pipe->curbuf; |
|---|
| 697 | + struct pipe_buffer *buf = &pipe->bufs[tail & mask]; |
|---|
| 761 | 698 | if (ret >= buf->len) { |
|---|
| 762 | 699 | ret -= buf->len; |
|---|
| 763 | 700 | buf->len = 0; |
|---|
| 764 | 701 | pipe_buf_release(pipe, buf); |
|---|
| 765 | | - pipe->curbuf = (pipe->curbuf + 1) & (pipe->buffers - 1); |
|---|
| 766 | | - pipe->nrbufs--; |
|---|
| 702 | + tail++; |
|---|
| 703 | + pipe->tail = tail; |
|---|
| 767 | 704 | if (pipe->files) |
|---|
| 768 | 705 | sd.need_wakeup = true; |
|---|
| 769 | 706 | } else { |
|---|
| .. | .. |
|---|
| 785 | 722 | return ret; |
|---|
| 786 | 723 | } |
|---|
| 787 | 724 | |
|---|
| 788 | | -EXPORT_SYMBOL(iter_file_splice_write); |
|---|
| 789 | | - |
|---|
| 790 | | -static int write_pipe_buf(struct pipe_inode_info *pipe, struct pipe_buffer *buf, |
|---|
| 791 | | - struct splice_desc *sd) |
|---|
| 792 | | -{ |
|---|
| 793 | | - int ret; |
|---|
| 794 | | - void *data; |
|---|
| 795 | | - loff_t tmp = sd->pos; |
|---|
| 796 | | - |
|---|
| 797 | | - data = kmap(buf->page); |
|---|
| 798 | | - ret = __kernel_write(sd->u.file, data + buf->offset, sd->len, &tmp); |
|---|
| 799 | | - kunmap(buf->page); |
|---|
| 800 | | - |
|---|
| 801 | | - return ret; |
|---|
| 802 | | -} |
|---|
| 803 | | - |
|---|
| 804 | | -static ssize_t default_file_splice_write(struct pipe_inode_info *pipe, |
|---|
| 805 | | - struct file *out, loff_t *ppos, |
|---|
| 806 | | - size_t len, unsigned int flags) |
|---|
| 807 | | -{ |
|---|
| 808 | | - ssize_t ret; |
|---|
| 809 | | - |
|---|
| 810 | | - ret = splice_from_pipe(pipe, out, ppos, len, flags, write_pipe_buf); |
|---|
| 811 | | - if (ret > 0) |
|---|
| 812 | | - *ppos += ret; |
|---|
| 813 | | - |
|---|
| 814 | | - return ret; |
|---|
| 815 | | -} |
|---|
| 725 | +EXPORT_SYMBOL_NS(iter_file_splice_write, ANDROID_GKI_VFS_EXPORT_ONLY); |
|---|
| 816 | 726 | |
|---|
| 817 | 727 | /** |
|---|
| 818 | 728 | * generic_splice_sendpage - splice data from a pipe to a socket |
|---|
| .. | .. |
|---|
| 835 | 745 | |
|---|
| 836 | 746 | EXPORT_SYMBOL(generic_splice_sendpage); |
|---|
| 837 | 747 | |
|---|
| 748 | +static int warn_unsupported(struct file *file, const char *op) |
|---|
| 749 | +{ |
|---|
| 750 | + pr_debug_ratelimited( |
|---|
| 751 | + "splice %s not supported for file %pD4 (pid: %d comm: %.20s)\n", |
|---|
| 752 | + op, file, current->pid, current->comm); |
|---|
| 753 | + return -EINVAL; |
|---|
| 754 | +} |
|---|
| 755 | + |
|---|
| 838 | 756 | /* |
|---|
| 839 | 757 | * Attempt to initiate a splice from pipe to file. |
|---|
| 840 | 758 | */ |
|---|
| 841 | 759 | static long do_splice_from(struct pipe_inode_info *pipe, struct file *out, |
|---|
| 842 | 760 | loff_t *ppos, size_t len, unsigned int flags) |
|---|
| 843 | 761 | { |
|---|
| 844 | | - ssize_t (*splice_write)(struct pipe_inode_info *, struct file *, |
|---|
| 845 | | - loff_t *, size_t, unsigned int); |
|---|
| 846 | | - |
|---|
| 847 | | - if (out->f_op->splice_write) |
|---|
| 848 | | - splice_write = out->f_op->splice_write; |
|---|
| 849 | | - else |
|---|
| 850 | | - splice_write = default_file_splice_write; |
|---|
| 851 | | - |
|---|
| 852 | | - return splice_write(pipe, out, ppos, len, flags); |
|---|
| 762 | + if (unlikely(!out->f_op->splice_write)) |
|---|
| 763 | + return warn_unsupported(out, "write"); |
|---|
| 764 | + return out->f_op->splice_write(pipe, out, ppos, len, flags); |
|---|
| 853 | 765 | } |
|---|
| 854 | 766 | |
|---|
| 855 | 767 | /* |
|---|
| .. | .. |
|---|
| 859 | 771 | struct pipe_inode_info *pipe, size_t len, |
|---|
| 860 | 772 | unsigned int flags) |
|---|
| 861 | 773 | { |
|---|
| 862 | | - ssize_t (*splice_read)(struct file *, loff_t *, |
|---|
| 863 | | - struct pipe_inode_info *, size_t, unsigned int); |
|---|
| 864 | 774 | int ret; |
|---|
| 865 | 775 | |
|---|
| 866 | 776 | if (unlikely(!(in->f_mode & FMODE_READ))) |
|---|
| .. | .. |
|---|
| 873 | 783 | if (unlikely(len > MAX_RW_COUNT)) |
|---|
| 874 | 784 | len = MAX_RW_COUNT; |
|---|
| 875 | 785 | |
|---|
| 876 | | - if (in->f_op->splice_read) |
|---|
| 877 | | - splice_read = in->f_op->splice_read; |
|---|
| 878 | | - else |
|---|
| 879 | | - splice_read = default_file_splice_read; |
|---|
| 880 | | - |
|---|
| 881 | | - return splice_read(in, ppos, pipe, len, flags); |
|---|
| 786 | + if (unlikely(!in->f_op->splice_read)) |
|---|
| 787 | + return warn_unsupported(in, "read"); |
|---|
| 788 | + return in->f_op->splice_read(in, ppos, pipe, len, flags); |
|---|
| 882 | 789 | } |
|---|
| 883 | 790 | |
|---|
| 884 | 791 | /** |
|---|
| .. | .. |
|---|
| 946 | 853 | sd->flags &= ~SPLICE_F_NONBLOCK; |
|---|
| 947 | 854 | more = sd->flags & SPLICE_F_MORE; |
|---|
| 948 | 855 | |
|---|
| 949 | | - WARN_ON_ONCE(pipe->nrbufs != 0); |
|---|
| 856 | + WARN_ON_ONCE(!pipe_empty(pipe->head, pipe->tail)); |
|---|
| 950 | 857 | |
|---|
| 951 | 858 | while (len) { |
|---|
| 952 | | - unsigned int pipe_pages; |
|---|
| 859 | + unsigned int p_space; |
|---|
| 953 | 860 | size_t read_len; |
|---|
| 954 | 861 | loff_t pos = sd->pos, prev_pos = pos; |
|---|
| 955 | 862 | |
|---|
| 956 | 863 | /* Don't try to read more the pipe has space for. */ |
|---|
| 957 | | - pipe_pages = pipe->buffers - pipe->nrbufs; |
|---|
| 958 | | - read_len = min(len, (size_t)pipe_pages << PAGE_SHIFT); |
|---|
| 864 | + p_space = pipe->max_usage - |
|---|
| 865 | + pipe_occupancy(pipe->head, pipe->tail); |
|---|
| 866 | + read_len = min_t(size_t, len, p_space << PAGE_SHIFT); |
|---|
| 959 | 867 | ret = do_splice_to(in, &pos, pipe, read_len, flags); |
|---|
| 960 | 868 | if (unlikely(ret <= 0)) |
|---|
| 961 | 869 | goto out_release; |
|---|
| .. | .. |
|---|
| 994 | 902 | } |
|---|
| 995 | 903 | |
|---|
| 996 | 904 | done: |
|---|
| 997 | | - pipe->nrbufs = pipe->curbuf = 0; |
|---|
| 905 | + pipe->tail = pipe->head = 0; |
|---|
| 998 | 906 | file_accessed(in); |
|---|
| 999 | 907 | return bytes; |
|---|
| 1000 | 908 | |
|---|
| .. | .. |
|---|
| 1003 | 911 | * If we did an incomplete transfer we must release |
|---|
| 1004 | 912 | * the pipe buffers in question: |
|---|
| 1005 | 913 | */ |
|---|
| 1006 | | - for (i = 0; i < pipe->buffers; i++) { |
|---|
| 1007 | | - struct pipe_buffer *buf = pipe->bufs + i; |
|---|
| 914 | + for (i = 0; i < pipe->ring_size; i++) { |
|---|
| 915 | + struct pipe_buffer *buf = &pipe->bufs[i]; |
|---|
| 1008 | 916 | |
|---|
| 1009 | 917 | if (buf->ops) |
|---|
| 1010 | 918 | pipe_buf_release(pipe, buf); |
|---|
| .. | .. |
|---|
| 1080 | 988 | send_sig(SIGPIPE, current, 0); |
|---|
| 1081 | 989 | return -EPIPE; |
|---|
| 1082 | 990 | } |
|---|
| 1083 | | - if (pipe->nrbufs != pipe->buffers) |
|---|
| 991 | + if (!pipe_full(pipe->head, pipe->tail, pipe->max_usage)) |
|---|
| 1084 | 992 | return 0; |
|---|
| 1085 | 993 | if (flags & SPLICE_F_NONBLOCK) |
|---|
| 1086 | 994 | return -EAGAIN; |
|---|
| 1087 | 995 | if (signal_pending(current)) |
|---|
| 1088 | 996 | return -ERESTARTSYS; |
|---|
| 1089 | | - pipe->waiting_writers++; |
|---|
| 1090 | | - pipe_wait(pipe); |
|---|
| 1091 | | - pipe->waiting_writers--; |
|---|
| 997 | + pipe_wait_writable(pipe); |
|---|
| 1092 | 998 | } |
|---|
| 1093 | 999 | } |
|---|
| 1094 | 1000 | |
|---|
| .. | .. |
|---|
| 1099 | 1005 | /* |
|---|
| 1100 | 1006 | * Determine where to splice to/from. |
|---|
| 1101 | 1007 | */ |
|---|
| 1102 | | -static long do_splice(struct file *in, loff_t __user *off_in, |
|---|
| 1103 | | - struct file *out, loff_t __user *off_out, |
|---|
| 1104 | | - size_t len, unsigned int flags) |
|---|
| 1008 | +long do_splice(struct file *in, loff_t *off_in, struct file *out, |
|---|
| 1009 | + loff_t *off_out, size_t len, unsigned int flags) |
|---|
| 1105 | 1010 | { |
|---|
| 1106 | 1011 | struct pipe_inode_info *ipipe; |
|---|
| 1107 | 1012 | struct pipe_inode_info *opipe; |
|---|
| 1108 | 1013 | loff_t offset; |
|---|
| 1109 | 1014 | long ret; |
|---|
| 1110 | 1015 | |
|---|
| 1111 | | - ipipe = get_pipe_info(in); |
|---|
| 1112 | | - opipe = get_pipe_info(out); |
|---|
| 1016 | + if (unlikely(!(in->f_mode & FMODE_READ) || |
|---|
| 1017 | + !(out->f_mode & FMODE_WRITE))) |
|---|
| 1018 | + return -EBADF; |
|---|
| 1019 | + |
|---|
| 1020 | + ipipe = get_pipe_info(in, true); |
|---|
| 1021 | + opipe = get_pipe_info(out, true); |
|---|
| 1113 | 1022 | |
|---|
| 1114 | 1023 | if (ipipe && opipe) { |
|---|
| 1115 | 1024 | if (off_in || off_out) |
|---|
| 1116 | 1025 | return -ESPIPE; |
|---|
| 1117 | 1026 | |
|---|
| 1118 | | - if (!(in->f_mode & FMODE_READ)) |
|---|
| 1119 | | - return -EBADF; |
|---|
| 1120 | | - |
|---|
| 1121 | | - if (!(out->f_mode & FMODE_WRITE)) |
|---|
| 1122 | | - return -EBADF; |
|---|
| 1123 | | - |
|---|
| 1124 | 1027 | /* Splicing to self would be fun, but... */ |
|---|
| 1125 | 1028 | if (ipipe == opipe) |
|---|
| 1126 | 1029 | return -EINVAL; |
|---|
| 1030 | + |
|---|
| 1031 | + if ((in->f_flags | out->f_flags) & O_NONBLOCK) |
|---|
| 1032 | + flags |= SPLICE_F_NONBLOCK; |
|---|
| 1127 | 1033 | |
|---|
| 1128 | 1034 | return splice_pipe_to_pipe(ipipe, opipe, len, flags); |
|---|
| 1129 | 1035 | } |
|---|
| .. | .. |
|---|
| 1134 | 1040 | if (off_out) { |
|---|
| 1135 | 1041 | if (!(out->f_mode & FMODE_PWRITE)) |
|---|
| 1136 | 1042 | return -EINVAL; |
|---|
| 1137 | | - if (copy_from_user(&offset, off_out, sizeof(loff_t))) |
|---|
| 1138 | | - return -EFAULT; |
|---|
| 1043 | + offset = *off_out; |
|---|
| 1139 | 1044 | } else { |
|---|
| 1140 | 1045 | offset = out->f_pos; |
|---|
| 1141 | 1046 | } |
|---|
| 1142 | | - |
|---|
| 1143 | | - if (unlikely(!(out->f_mode & FMODE_WRITE))) |
|---|
| 1144 | | - return -EBADF; |
|---|
| 1145 | 1047 | |
|---|
| 1146 | 1048 | if (unlikely(out->f_flags & O_APPEND)) |
|---|
| 1147 | 1049 | return -EINVAL; |
|---|
| .. | .. |
|---|
| 1150 | 1052 | if (unlikely(ret < 0)) |
|---|
| 1151 | 1053 | return ret; |
|---|
| 1152 | 1054 | |
|---|
| 1055 | + if (in->f_flags & O_NONBLOCK) |
|---|
| 1056 | + flags |= SPLICE_F_NONBLOCK; |
|---|
| 1057 | + |
|---|
| 1153 | 1058 | file_start_write(out); |
|---|
| 1154 | 1059 | ret = do_splice_from(ipipe, out, &offset, len, flags); |
|---|
| 1155 | 1060 | file_end_write(out); |
|---|
| 1156 | 1061 | |
|---|
| 1157 | 1062 | if (!off_out) |
|---|
| 1158 | 1063 | out->f_pos = offset; |
|---|
| 1159 | | - else if (copy_to_user(off_out, &offset, sizeof(loff_t))) |
|---|
| 1160 | | - ret = -EFAULT; |
|---|
| 1064 | + else |
|---|
| 1065 | + *off_out = offset; |
|---|
| 1161 | 1066 | |
|---|
| 1162 | 1067 | return ret; |
|---|
| 1163 | 1068 | } |
|---|
| .. | .. |
|---|
| 1168 | 1073 | if (off_in) { |
|---|
| 1169 | 1074 | if (!(in->f_mode & FMODE_PREAD)) |
|---|
| 1170 | 1075 | return -EINVAL; |
|---|
| 1171 | | - if (copy_from_user(&offset, off_in, sizeof(loff_t))) |
|---|
| 1172 | | - return -EFAULT; |
|---|
| 1076 | + offset = *off_in; |
|---|
| 1173 | 1077 | } else { |
|---|
| 1174 | 1078 | offset = in->f_pos; |
|---|
| 1175 | 1079 | } |
|---|
| 1176 | 1080 | |
|---|
| 1081 | + if (out->f_flags & O_NONBLOCK) |
|---|
| 1082 | + flags |= SPLICE_F_NONBLOCK; |
|---|
| 1083 | + |
|---|
| 1177 | 1084 | pipe_lock(opipe); |
|---|
| 1178 | 1085 | ret = wait_for_space(opipe, flags); |
|---|
| 1179 | 1086 | if (!ret) { |
|---|
| 1180 | | - unsigned int pipe_pages; |
|---|
| 1087 | + unsigned int p_space; |
|---|
| 1181 | 1088 | |
|---|
| 1182 | 1089 | /* Don't try to read more the pipe has space for. */ |
|---|
| 1183 | | - pipe_pages = opipe->buffers - opipe->nrbufs; |
|---|
| 1184 | | - len = min(len, (size_t)pipe_pages << PAGE_SHIFT); |
|---|
| 1090 | + p_space = opipe->max_usage - pipe_occupancy(opipe->head, opipe->tail); |
|---|
| 1091 | + len = min_t(size_t, len, p_space << PAGE_SHIFT); |
|---|
| 1185 | 1092 | |
|---|
| 1186 | 1093 | ret = do_splice_to(in, &offset, opipe, len, flags); |
|---|
| 1187 | 1094 | } |
|---|
| .. | .. |
|---|
| 1190 | 1097 | wakeup_pipe_readers(opipe); |
|---|
| 1191 | 1098 | if (!off_in) |
|---|
| 1192 | 1099 | in->f_pos = offset; |
|---|
| 1193 | | - else if (copy_to_user(off_in, &offset, sizeof(loff_t))) |
|---|
| 1194 | | - ret = -EFAULT; |
|---|
| 1100 | + else |
|---|
| 1101 | + *off_in = offset; |
|---|
| 1195 | 1102 | |
|---|
| 1196 | 1103 | return ret; |
|---|
| 1197 | 1104 | } |
|---|
| 1198 | 1105 | |
|---|
| 1199 | 1106 | return -EINVAL; |
|---|
| 1107 | +} |
|---|
| 1108 | + |
|---|
| 1109 | +static long __do_splice(struct file *in, loff_t __user *off_in, |
|---|
| 1110 | + struct file *out, loff_t __user *off_out, |
|---|
| 1111 | + size_t len, unsigned int flags) |
|---|
| 1112 | +{ |
|---|
| 1113 | + struct pipe_inode_info *ipipe; |
|---|
| 1114 | + struct pipe_inode_info *opipe; |
|---|
| 1115 | + loff_t offset, *__off_in = NULL, *__off_out = NULL; |
|---|
| 1116 | + long ret; |
|---|
| 1117 | + |
|---|
| 1118 | + ipipe = get_pipe_info(in, true); |
|---|
| 1119 | + opipe = get_pipe_info(out, true); |
|---|
| 1120 | + |
|---|
| 1121 | + if (ipipe && off_in) |
|---|
| 1122 | + return -ESPIPE; |
|---|
| 1123 | + if (opipe && off_out) |
|---|
| 1124 | + return -ESPIPE; |
|---|
| 1125 | + |
|---|
| 1126 | + if (off_out) { |
|---|
| 1127 | + if (copy_from_user(&offset, off_out, sizeof(loff_t))) |
|---|
| 1128 | + return -EFAULT; |
|---|
| 1129 | + __off_out = &offset; |
|---|
| 1130 | + } |
|---|
| 1131 | + if (off_in) { |
|---|
| 1132 | + if (copy_from_user(&offset, off_in, sizeof(loff_t))) |
|---|
| 1133 | + return -EFAULT; |
|---|
| 1134 | + __off_in = &offset; |
|---|
| 1135 | + } |
|---|
| 1136 | + |
|---|
| 1137 | + ret = do_splice(in, __off_in, out, __off_out, len, flags); |
|---|
| 1138 | + if (ret < 0) |
|---|
| 1139 | + return ret; |
|---|
| 1140 | + |
|---|
| 1141 | + if (__off_out && copy_to_user(off_out, __off_out, sizeof(loff_t))) |
|---|
| 1142 | + return -EFAULT; |
|---|
| 1143 | + if (__off_in && copy_to_user(off_in, __off_in, sizeof(loff_t))) |
|---|
| 1144 | + return -EFAULT; |
|---|
| 1145 | + |
|---|
| 1146 | + return ret; |
|---|
| 1200 | 1147 | } |
|---|
| 1201 | 1148 | |
|---|
| 1202 | 1149 | static int iter_to_pipe(struct iov_iter *from, |
|---|
| .. | .. |
|---|
| 1259 | 1206 | static long vmsplice_to_user(struct file *file, struct iov_iter *iter, |
|---|
| 1260 | 1207 | unsigned int flags) |
|---|
| 1261 | 1208 | { |
|---|
| 1262 | | - struct pipe_inode_info *pipe = get_pipe_info(file); |
|---|
| 1209 | + struct pipe_inode_info *pipe = get_pipe_info(file, true); |
|---|
| 1263 | 1210 | struct splice_desc sd = { |
|---|
| 1264 | 1211 | .total_len = iov_iter_count(iter), |
|---|
| 1265 | 1212 | .flags = flags, |
|---|
| .. | .. |
|---|
| 1294 | 1241 | if (flags & SPLICE_F_GIFT) |
|---|
| 1295 | 1242 | buf_flag = PIPE_BUF_FLAG_GIFT; |
|---|
| 1296 | 1243 | |
|---|
| 1297 | | - pipe = get_pipe_info(file); |
|---|
| 1244 | + pipe = get_pipe_info(file, true); |
|---|
| 1298 | 1245 | if (!pipe) |
|---|
| 1299 | 1246 | return -EBADF; |
|---|
| 1300 | 1247 | |
|---|
| .. | .. |
|---|
| 1339 | 1286 | * Currently we punt and implement it as a normal copy, see pipe_to_user(). |
|---|
| 1340 | 1287 | * |
|---|
| 1341 | 1288 | */ |
|---|
| 1342 | | -static long do_vmsplice(struct file *f, struct iov_iter *iter, unsigned int flags) |
|---|
| 1343 | | -{ |
|---|
| 1344 | | - if (unlikely(flags & ~SPLICE_F_ALL)) |
|---|
| 1345 | | - return -EINVAL; |
|---|
| 1346 | | - |
|---|
| 1347 | | - if (!iov_iter_count(iter)) |
|---|
| 1348 | | - return 0; |
|---|
| 1349 | | - |
|---|
| 1350 | | - if (iov_iter_rw(iter) == WRITE) |
|---|
| 1351 | | - return vmsplice_to_pipe(f, iter, flags); |
|---|
| 1352 | | - else |
|---|
| 1353 | | - return vmsplice_to_user(f, iter, flags); |
|---|
| 1354 | | -} |
|---|
| 1355 | | - |
|---|
| 1356 | 1289 | SYSCALL_DEFINE4(vmsplice, int, fd, const struct iovec __user *, uiov, |
|---|
| 1357 | 1290 | unsigned long, nr_segs, unsigned int, flags) |
|---|
| 1358 | 1291 | { |
|---|
| 1359 | 1292 | struct iovec iovstack[UIO_FASTIOV]; |
|---|
| 1360 | 1293 | struct iovec *iov = iovstack; |
|---|
| 1361 | 1294 | struct iov_iter iter; |
|---|
| 1362 | | - long error; |
|---|
| 1295 | + ssize_t error; |
|---|
| 1363 | 1296 | struct fd f; |
|---|
| 1364 | 1297 | int type; |
|---|
| 1298 | + |
|---|
| 1299 | + if (unlikely(flags & ~SPLICE_F_ALL)) |
|---|
| 1300 | + return -EINVAL; |
|---|
| 1365 | 1301 | |
|---|
| 1366 | 1302 | f = fdget(fd); |
|---|
| 1367 | 1303 | error = vmsplice_type(f, &type); |
|---|
| .. | .. |
|---|
| 1370 | 1306 | |
|---|
| 1371 | 1307 | error = import_iovec(type, uiov, nr_segs, |
|---|
| 1372 | 1308 | ARRAY_SIZE(iovstack), &iov, &iter); |
|---|
| 1373 | | - if (!error) { |
|---|
| 1374 | | - error = do_vmsplice(f.file, &iter, flags); |
|---|
| 1375 | | - kfree(iov); |
|---|
| 1376 | | - } |
|---|
| 1309 | + if (error < 0) |
|---|
| 1310 | + goto out_fdput; |
|---|
| 1311 | + |
|---|
| 1312 | + if (!iov_iter_count(&iter)) |
|---|
| 1313 | + error = 0; |
|---|
| 1314 | + else if (iov_iter_rw(&iter) == WRITE) |
|---|
| 1315 | + error = vmsplice_to_pipe(f.file, &iter, flags); |
|---|
| 1316 | + else |
|---|
| 1317 | + error = vmsplice_to_user(f.file, &iter, flags); |
|---|
| 1318 | + |
|---|
| 1319 | + kfree(iov); |
|---|
| 1320 | +out_fdput: |
|---|
| 1377 | 1321 | fdput(f); |
|---|
| 1378 | 1322 | return error; |
|---|
| 1379 | 1323 | } |
|---|
| 1380 | | - |
|---|
| 1381 | | -#ifdef CONFIG_COMPAT |
|---|
| 1382 | | -COMPAT_SYSCALL_DEFINE4(vmsplice, int, fd, const struct compat_iovec __user *, iov32, |
|---|
| 1383 | | - unsigned int, nr_segs, unsigned int, flags) |
|---|
| 1384 | | -{ |
|---|
| 1385 | | - struct iovec iovstack[UIO_FASTIOV]; |
|---|
| 1386 | | - struct iovec *iov = iovstack; |
|---|
| 1387 | | - struct iov_iter iter; |
|---|
| 1388 | | - long error; |
|---|
| 1389 | | - struct fd f; |
|---|
| 1390 | | - int type; |
|---|
| 1391 | | - |
|---|
| 1392 | | - f = fdget(fd); |
|---|
| 1393 | | - error = vmsplice_type(f, &type); |
|---|
| 1394 | | - if (error) |
|---|
| 1395 | | - return error; |
|---|
| 1396 | | - |
|---|
| 1397 | | - error = compat_import_iovec(type, iov32, nr_segs, |
|---|
| 1398 | | - ARRAY_SIZE(iovstack), &iov, &iter); |
|---|
| 1399 | | - if (!error) { |
|---|
| 1400 | | - error = do_vmsplice(f.file, &iter, flags); |
|---|
| 1401 | | - kfree(iov); |
|---|
| 1402 | | - } |
|---|
| 1403 | | - fdput(f); |
|---|
| 1404 | | - return error; |
|---|
| 1405 | | -} |
|---|
| 1406 | | -#endif |
|---|
| 1407 | 1324 | |
|---|
| 1408 | 1325 | SYSCALL_DEFINE6(splice, int, fd_in, loff_t __user *, off_in, |
|---|
| 1409 | 1326 | int, fd_out, loff_t __user *, off_out, |
|---|
| .. | .. |
|---|
| 1421 | 1338 | error = -EBADF; |
|---|
| 1422 | 1339 | in = fdget(fd_in); |
|---|
| 1423 | 1340 | if (in.file) { |
|---|
| 1424 | | - if (in.file->f_mode & FMODE_READ) { |
|---|
| 1425 | | - out = fdget(fd_out); |
|---|
| 1426 | | - if (out.file) { |
|---|
| 1427 | | - if (out.file->f_mode & FMODE_WRITE) |
|---|
| 1428 | | - error = do_splice(in.file, off_in, |
|---|
| 1429 | | - out.file, off_out, |
|---|
| 1430 | | - len, flags); |
|---|
| 1431 | | - fdput(out); |
|---|
| 1432 | | - } |
|---|
| 1341 | + out = fdget(fd_out); |
|---|
| 1342 | + if (out.file) { |
|---|
| 1343 | + error = __do_splice(in.file, off_in, out.file, off_out, |
|---|
| 1344 | + len, flags); |
|---|
| 1345 | + fdput(out); |
|---|
| 1433 | 1346 | } |
|---|
| 1434 | 1347 | fdput(in); |
|---|
| 1435 | 1348 | } |
|---|
| .. | .. |
|---|
| 1445 | 1358 | int ret; |
|---|
| 1446 | 1359 | |
|---|
| 1447 | 1360 | /* |
|---|
| 1448 | | - * Check ->nrbufs without the inode lock first. This function |
|---|
| 1361 | + * Check the pipe occupancy without the inode lock first. This function |
|---|
| 1449 | 1362 | * is speculative anyways, so missing one is ok. |
|---|
| 1450 | 1363 | */ |
|---|
| 1451 | | - if (pipe->nrbufs) |
|---|
| 1364 | + if (!pipe_empty(pipe->head, pipe->tail)) |
|---|
| 1452 | 1365 | return 0; |
|---|
| 1453 | 1366 | |
|---|
| 1454 | 1367 | ret = 0; |
|---|
| 1455 | 1368 | pipe_lock(pipe); |
|---|
| 1456 | 1369 | |
|---|
| 1457 | | - while (!pipe->nrbufs) { |
|---|
| 1370 | + while (pipe_empty(pipe->head, pipe->tail)) { |
|---|
| 1458 | 1371 | if (signal_pending(current)) { |
|---|
| 1459 | 1372 | ret = -ERESTARTSYS; |
|---|
| 1460 | 1373 | break; |
|---|
| 1461 | 1374 | } |
|---|
| 1462 | 1375 | if (!pipe->writers) |
|---|
| 1463 | 1376 | break; |
|---|
| 1464 | | - if (!pipe->waiting_writers) { |
|---|
| 1465 | | - if (flags & SPLICE_F_NONBLOCK) { |
|---|
| 1466 | | - ret = -EAGAIN; |
|---|
| 1467 | | - break; |
|---|
| 1468 | | - } |
|---|
| 1377 | + if (flags & SPLICE_F_NONBLOCK) { |
|---|
| 1378 | + ret = -EAGAIN; |
|---|
| 1379 | + break; |
|---|
| 1469 | 1380 | } |
|---|
| 1470 | | - pipe_wait(pipe); |
|---|
| 1381 | + pipe_wait_readable(pipe); |
|---|
| 1471 | 1382 | } |
|---|
| 1472 | 1383 | |
|---|
| 1473 | 1384 | pipe_unlock(pipe); |
|---|
| .. | .. |
|---|
| 1483 | 1394 | int ret; |
|---|
| 1484 | 1395 | |
|---|
| 1485 | 1396 | /* |
|---|
| 1486 | | - * Check ->nrbufs without the inode lock first. This function |
|---|
| 1397 | + * Check pipe occupancy without the inode lock first. This function |
|---|
| 1487 | 1398 | * is speculative anyways, so missing one is ok. |
|---|
| 1488 | 1399 | */ |
|---|
| 1489 | | - if (pipe->nrbufs < pipe->buffers) |
|---|
| 1400 | + if (!pipe_full(pipe->head, pipe->tail, pipe->max_usage)) |
|---|
| 1490 | 1401 | return 0; |
|---|
| 1491 | 1402 | |
|---|
| 1492 | 1403 | ret = 0; |
|---|
| 1493 | 1404 | pipe_lock(pipe); |
|---|
| 1494 | 1405 | |
|---|
| 1495 | | - while (pipe->nrbufs >= pipe->buffers) { |
|---|
| 1406 | + while (pipe_full(pipe->head, pipe->tail, pipe->max_usage)) { |
|---|
| 1496 | 1407 | if (!pipe->readers) { |
|---|
| 1497 | 1408 | send_sig(SIGPIPE, current, 0); |
|---|
| 1498 | 1409 | ret = -EPIPE; |
|---|
| .. | .. |
|---|
| 1506 | 1417 | ret = -ERESTARTSYS; |
|---|
| 1507 | 1418 | break; |
|---|
| 1508 | 1419 | } |
|---|
| 1509 | | - pipe->waiting_writers++; |
|---|
| 1510 | | - pipe_wait(pipe); |
|---|
| 1511 | | - pipe->waiting_writers--; |
|---|
| 1420 | + pipe_wait_writable(pipe); |
|---|
| 1512 | 1421 | } |
|---|
| 1513 | 1422 | |
|---|
| 1514 | 1423 | pipe_unlock(pipe); |
|---|
| .. | .. |
|---|
| 1523 | 1432 | size_t len, unsigned int flags) |
|---|
| 1524 | 1433 | { |
|---|
| 1525 | 1434 | struct pipe_buffer *ibuf, *obuf; |
|---|
| 1526 | | - int ret = 0, nbuf; |
|---|
| 1435 | + unsigned int i_head, o_head; |
|---|
| 1436 | + unsigned int i_tail, o_tail; |
|---|
| 1437 | + unsigned int i_mask, o_mask; |
|---|
| 1438 | + int ret = 0; |
|---|
| 1527 | 1439 | bool input_wakeup = false; |
|---|
| 1528 | 1440 | |
|---|
| 1529 | 1441 | |
|---|
| .. | .. |
|---|
| 1543 | 1455 | */ |
|---|
| 1544 | 1456 | pipe_double_lock(ipipe, opipe); |
|---|
| 1545 | 1457 | |
|---|
| 1458 | + i_tail = ipipe->tail; |
|---|
| 1459 | + i_mask = ipipe->ring_size - 1; |
|---|
| 1460 | + o_head = opipe->head; |
|---|
| 1461 | + o_mask = opipe->ring_size - 1; |
|---|
| 1462 | + |
|---|
| 1546 | 1463 | do { |
|---|
| 1464 | + size_t o_len; |
|---|
| 1465 | + |
|---|
| 1547 | 1466 | if (!opipe->readers) { |
|---|
| 1548 | 1467 | send_sig(SIGPIPE, current, 0); |
|---|
| 1549 | 1468 | if (!ret) |
|---|
| .. | .. |
|---|
| 1551 | 1470 | break; |
|---|
| 1552 | 1471 | } |
|---|
| 1553 | 1472 | |
|---|
| 1554 | | - if (!ipipe->nrbufs && !ipipe->writers) |
|---|
| 1473 | + i_head = ipipe->head; |
|---|
| 1474 | + o_tail = opipe->tail; |
|---|
| 1475 | + |
|---|
| 1476 | + if (pipe_empty(i_head, i_tail) && !ipipe->writers) |
|---|
| 1555 | 1477 | break; |
|---|
| 1556 | 1478 | |
|---|
| 1557 | 1479 | /* |
|---|
| 1558 | 1480 | * Cannot make any progress, because either the input |
|---|
| 1559 | 1481 | * pipe is empty or the output pipe is full. |
|---|
| 1560 | 1482 | */ |
|---|
| 1561 | | - if (!ipipe->nrbufs || opipe->nrbufs >= opipe->buffers) { |
|---|
| 1483 | + if (pipe_empty(i_head, i_tail) || |
|---|
| 1484 | + pipe_full(o_head, o_tail, opipe->max_usage)) { |
|---|
| 1562 | 1485 | /* Already processed some buffers, break */ |
|---|
| 1563 | 1486 | if (ret) |
|---|
| 1564 | 1487 | break; |
|---|
| .. | .. |
|---|
| 1578 | 1501 | goto retry; |
|---|
| 1579 | 1502 | } |
|---|
| 1580 | 1503 | |
|---|
| 1581 | | - ibuf = ipipe->bufs + ipipe->curbuf; |
|---|
| 1582 | | - nbuf = (opipe->curbuf + opipe->nrbufs) & (opipe->buffers - 1); |
|---|
| 1583 | | - obuf = opipe->bufs + nbuf; |
|---|
| 1504 | + ibuf = &ipipe->bufs[i_tail & i_mask]; |
|---|
| 1505 | + obuf = &opipe->bufs[o_head & o_mask]; |
|---|
| 1584 | 1506 | |
|---|
| 1585 | 1507 | if (len >= ibuf->len) { |
|---|
| 1586 | 1508 | /* |
|---|
| .. | .. |
|---|
| 1588 | 1510 | */ |
|---|
| 1589 | 1511 | *obuf = *ibuf; |
|---|
| 1590 | 1512 | ibuf->ops = NULL; |
|---|
| 1591 | | - opipe->nrbufs++; |
|---|
| 1592 | | - ipipe->curbuf = (ipipe->curbuf + 1) & (ipipe->buffers - 1); |
|---|
| 1593 | | - ipipe->nrbufs--; |
|---|
| 1513 | + i_tail++; |
|---|
| 1514 | + ipipe->tail = i_tail; |
|---|
| 1594 | 1515 | input_wakeup = true; |
|---|
| 1516 | + o_len = obuf->len; |
|---|
| 1517 | + o_head++; |
|---|
| 1518 | + opipe->head = o_head; |
|---|
| 1595 | 1519 | } else { |
|---|
| 1596 | 1520 | /* |
|---|
| 1597 | 1521 | * Get a reference to this pipe buffer, |
|---|
| .. | .. |
|---|
| 1605 | 1529 | *obuf = *ibuf; |
|---|
| 1606 | 1530 | |
|---|
| 1607 | 1531 | /* |
|---|
| 1608 | | - * Don't inherit the gift flag, we need to |
|---|
| 1532 | + * Don't inherit the gift and merge flags, we need to |
|---|
| 1609 | 1533 | * prevent multiple steals of this page. |
|---|
| 1610 | 1534 | */ |
|---|
| 1611 | 1535 | obuf->flags &= ~PIPE_BUF_FLAG_GIFT; |
|---|
| 1612 | | - |
|---|
| 1613 | | - pipe_buf_mark_unmergeable(obuf); |
|---|
| 1536 | + obuf->flags &= ~PIPE_BUF_FLAG_CAN_MERGE; |
|---|
| 1614 | 1537 | |
|---|
| 1615 | 1538 | obuf->len = len; |
|---|
| 1616 | | - opipe->nrbufs++; |
|---|
| 1617 | | - ibuf->offset += obuf->len; |
|---|
| 1618 | | - ibuf->len -= obuf->len; |
|---|
| 1539 | + ibuf->offset += len; |
|---|
| 1540 | + ibuf->len -= len; |
|---|
| 1541 | + o_len = len; |
|---|
| 1542 | + o_head++; |
|---|
| 1543 | + opipe->head = o_head; |
|---|
| 1619 | 1544 | } |
|---|
| 1620 | | - ret += obuf->len; |
|---|
| 1621 | | - len -= obuf->len; |
|---|
| 1545 | + ret += o_len; |
|---|
| 1546 | + len -= o_len; |
|---|
| 1622 | 1547 | } while (len); |
|---|
| 1623 | 1548 | |
|---|
| 1624 | 1549 | pipe_unlock(ipipe); |
|---|
| .. | .. |
|---|
| 1644 | 1569 | size_t len, unsigned int flags) |
|---|
| 1645 | 1570 | { |
|---|
| 1646 | 1571 | struct pipe_buffer *ibuf, *obuf; |
|---|
| 1647 | | - int ret = 0, i = 0, nbuf; |
|---|
| 1572 | + unsigned int i_head, o_head; |
|---|
| 1573 | + unsigned int i_tail, o_tail; |
|---|
| 1574 | + unsigned int i_mask, o_mask; |
|---|
| 1575 | + int ret = 0; |
|---|
| 1648 | 1576 | |
|---|
| 1649 | 1577 | /* |
|---|
| 1650 | 1578 | * Potential ABBA deadlock, work around it by ordering lock |
|---|
| .. | .. |
|---|
| 1652 | 1580 | * could deadlock (one doing tee from A -> B, the other from B -> A). |
|---|
| 1653 | 1581 | */ |
|---|
| 1654 | 1582 | pipe_double_lock(ipipe, opipe); |
|---|
| 1583 | + |
|---|
| 1584 | + i_tail = ipipe->tail; |
|---|
| 1585 | + i_mask = ipipe->ring_size - 1; |
|---|
| 1586 | + o_head = opipe->head; |
|---|
| 1587 | + o_mask = opipe->ring_size - 1; |
|---|
| 1655 | 1588 | |
|---|
| 1656 | 1589 | do { |
|---|
| 1657 | 1590 | if (!opipe->readers) { |
|---|
| .. | .. |
|---|
| 1661 | 1594 | break; |
|---|
| 1662 | 1595 | } |
|---|
| 1663 | 1596 | |
|---|
| 1597 | + i_head = ipipe->head; |
|---|
| 1598 | + o_tail = opipe->tail; |
|---|
| 1599 | + |
|---|
| 1664 | 1600 | /* |
|---|
| 1665 | | - * If we have iterated all input buffers or ran out of |
|---|
| 1601 | + * If we have iterated all input buffers or run out of |
|---|
| 1666 | 1602 | * output room, break. |
|---|
| 1667 | 1603 | */ |
|---|
| 1668 | | - if (i >= ipipe->nrbufs || opipe->nrbufs >= opipe->buffers) |
|---|
| 1604 | + if (pipe_empty(i_head, i_tail) || |
|---|
| 1605 | + pipe_full(o_head, o_tail, opipe->max_usage)) |
|---|
| 1669 | 1606 | break; |
|---|
| 1670 | 1607 | |
|---|
| 1671 | | - ibuf = ipipe->bufs + ((ipipe->curbuf + i) & (ipipe->buffers-1)); |
|---|
| 1672 | | - nbuf = (opipe->curbuf + opipe->nrbufs) & (opipe->buffers - 1); |
|---|
| 1608 | + ibuf = &ipipe->bufs[i_tail & i_mask]; |
|---|
| 1609 | + obuf = &opipe->bufs[o_head & o_mask]; |
|---|
| 1673 | 1610 | |
|---|
| 1674 | 1611 | /* |
|---|
| 1675 | 1612 | * Get a reference to this pipe buffer, |
|---|
| .. | .. |
|---|
| 1681 | 1618 | break; |
|---|
| 1682 | 1619 | } |
|---|
| 1683 | 1620 | |
|---|
| 1684 | | - obuf = opipe->bufs + nbuf; |
|---|
| 1685 | 1621 | *obuf = *ibuf; |
|---|
| 1686 | 1622 | |
|---|
| 1687 | 1623 | /* |
|---|
| 1688 | | - * Don't inherit the gift flag, we need to |
|---|
| 1689 | | - * prevent multiple steals of this page. |
|---|
| 1624 | + * Don't inherit the gift and merge flag, we need to prevent |
|---|
| 1625 | + * multiple steals of this page. |
|---|
| 1690 | 1626 | */ |
|---|
| 1691 | 1627 | obuf->flags &= ~PIPE_BUF_FLAG_GIFT; |
|---|
| 1692 | | - |
|---|
| 1693 | | - pipe_buf_mark_unmergeable(obuf); |
|---|
| 1628 | + obuf->flags &= ~PIPE_BUF_FLAG_CAN_MERGE; |
|---|
| 1694 | 1629 | |
|---|
| 1695 | 1630 | if (obuf->len > len) |
|---|
| 1696 | 1631 | obuf->len = len; |
|---|
| 1697 | | - |
|---|
| 1698 | | - opipe->nrbufs++; |
|---|
| 1699 | 1632 | ret += obuf->len; |
|---|
| 1700 | 1633 | len -= obuf->len; |
|---|
| 1701 | | - i++; |
|---|
| 1702 | | - } while (len); |
|---|
| 1703 | 1634 | |
|---|
| 1704 | | - /* |
|---|
| 1705 | | - * return EAGAIN if we have the potential of some data in the |
|---|
| 1706 | | - * future, otherwise just return 0 |
|---|
| 1707 | | - */ |
|---|
| 1708 | | - if (!ret && ipipe->waiting_writers && (flags & SPLICE_F_NONBLOCK)) |
|---|
| 1709 | | - ret = -EAGAIN; |
|---|
| 1635 | + o_head++; |
|---|
| 1636 | + opipe->head = o_head; |
|---|
| 1637 | + i_tail++; |
|---|
| 1638 | + } while (len); |
|---|
| 1710 | 1639 | |
|---|
| 1711 | 1640 | pipe_unlock(ipipe); |
|---|
| 1712 | 1641 | pipe_unlock(opipe); |
|---|
| .. | .. |
|---|
| 1726 | 1655 | * The 'flags' used are the SPLICE_F_* variants, currently the only |
|---|
| 1727 | 1656 | * applicable one is SPLICE_F_NONBLOCK. |
|---|
| 1728 | 1657 | */ |
|---|
| 1729 | | -static long do_tee(struct file *in, struct file *out, size_t len, |
|---|
| 1730 | | - unsigned int flags) |
|---|
| 1658 | +long do_tee(struct file *in, struct file *out, size_t len, unsigned int flags) |
|---|
| 1731 | 1659 | { |
|---|
| 1732 | | - struct pipe_inode_info *ipipe = get_pipe_info(in); |
|---|
| 1733 | | - struct pipe_inode_info *opipe = get_pipe_info(out); |
|---|
| 1660 | + struct pipe_inode_info *ipipe = get_pipe_info(in, true); |
|---|
| 1661 | + struct pipe_inode_info *opipe = get_pipe_info(out, true); |
|---|
| 1734 | 1662 | int ret = -EINVAL; |
|---|
| 1663 | + |
|---|
| 1664 | + if (unlikely(!(in->f_mode & FMODE_READ) || |
|---|
| 1665 | + !(out->f_mode & FMODE_WRITE))) |
|---|
| 1666 | + return -EBADF; |
|---|
| 1735 | 1667 | |
|---|
| 1736 | 1668 | /* |
|---|
| 1737 | 1669 | * Duplicate the contents of ipipe to opipe without actually |
|---|
| 1738 | 1670 | * copying the data. |
|---|
| 1739 | 1671 | */ |
|---|
| 1740 | 1672 | if (ipipe && opipe && ipipe != opipe) { |
|---|
| 1673 | + if ((in->f_flags | out->f_flags) & O_NONBLOCK) |
|---|
| 1674 | + flags |= SPLICE_F_NONBLOCK; |
|---|
| 1675 | + |
|---|
| 1741 | 1676 | /* |
|---|
| 1742 | 1677 | * Keep going, unless we encounter an error. The ipipe/opipe |
|---|
| 1743 | 1678 | * ordering doesn't really matter. |
|---|
| .. | .. |
|---|
| 1755 | 1690 | |
|---|
| 1756 | 1691 | SYSCALL_DEFINE4(tee, int, fdin, int, fdout, size_t, len, unsigned int, flags) |
|---|
| 1757 | 1692 | { |
|---|
| 1758 | | - struct fd in; |
|---|
| 1693 | + struct fd in, out; |
|---|
| 1759 | 1694 | int error; |
|---|
| 1760 | 1695 | |
|---|
| 1761 | 1696 | if (unlikely(flags & ~SPLICE_F_ALL)) |
|---|
| .. | .. |
|---|
| 1767 | 1702 | error = -EBADF; |
|---|
| 1768 | 1703 | in = fdget(fdin); |
|---|
| 1769 | 1704 | if (in.file) { |
|---|
| 1770 | | - if (in.file->f_mode & FMODE_READ) { |
|---|
| 1771 | | - struct fd out = fdget(fdout); |
|---|
| 1772 | | - if (out.file) { |
|---|
| 1773 | | - if (out.file->f_mode & FMODE_WRITE) |
|---|
| 1774 | | - error = do_tee(in.file, out.file, |
|---|
| 1775 | | - len, flags); |
|---|
| 1776 | | - fdput(out); |
|---|
| 1777 | | - } |
|---|
| 1705 | + out = fdget(fdout); |
|---|
| 1706 | + if (out.file) { |
|---|
| 1707 | + error = do_tee(in.file, out.file, len, flags); |
|---|
| 1708 | + fdput(out); |
|---|
| 1778 | 1709 | } |
|---|
| 1779 | 1710 | fdput(in); |
|---|
| 1780 | 1711 | } |
|---|