From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761348AbcINIhj (ORCPT ); Wed, 14 Sep 2016 04:37:39 -0400 Received: from mail-wm0-f45.google.com ([74.125.82.45]:38538 "EHLO mail-wm0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1761076AbcINIh2 (ORCPT ); Wed, 14 Sep 2016 04:37:28 -0400 From: Miklos Szeredi To: linux-fsdevel@vger.kernel.org Cc: linux-kernel@vger.kernel.org, Al Viro Subject: [PATCH 06/11] pipe: no need to confirm page cache buf Date: Wed, 14 Sep 2016 10:37:11 +0200 Message-Id: <1473842236-28655-7-git-send-email-mszeredi@redhat.com> X-Mailer: git-send-email 2.5.5 In-Reply-To: <1473842236-28655-1-git-send-email-mszeredi@redhat.com> References: <1473842236-28655-1-git-send-email-mszeredi@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org page_cache_pipe_buf_confirm() is used only in page_cache_pipe_buf_ops. page_cache_pipe_buf_ops is used in two places: 1) __generic_file_splice_read() This iterates all the pages, if not uptodate locks page, and if still not uptodate does ->readpage() which reads the page synchronously. 2) shmem_file_splice_read() This calls shmem_getpage() on individual pages. shmem_getpage() swaps in the page synchronously if not in memory, so it also seems to return an uptodate page. So all pages put into the buffer will be uptodate. Things could happen to that page that make it not uptodate while sitting in the pipe, but it's questionable whether we should care about that. Checking for being uptodate in the face of such page state change is always going to be racy. Signed-off-by: Miklos Szeredi --- fs/splice.c | 43 +------------------------------------------ 1 file changed, 1 insertion(+), 42 deletions(-) diff --git a/fs/splice.c b/fs/splice.c index ba7a2240d58c..0ecbe3011796 100644 --- a/fs/splice.c +++ b/fs/splice.c @@ -92,51 +92,10 @@ static void page_cache_pipe_buf_release(struct pipe_inode_info *pipe, buf->flags &= ~PIPE_BUF_FLAG_LRU; } -/* - * Check whether the contents of buf is OK to access. Since the content - * is a page cache page, IO may be in flight. - */ -static int page_cache_pipe_buf_confirm(struct pipe_inode_info *pipe, - struct pipe_buffer *buf) -{ - struct page *page = buf->page; - int err; - - if (!PageUptodate(page)) { - lock_page(page); - - /* - * Page got truncated/unhashed. This will cause a 0-byte - * splice, if this is the first page. - */ - if (!page->mapping) { - err = -ENODATA; - goto error; - } - - /* - * Uh oh, read-error from disk. - */ - if (!PageUptodate(page)) { - err = -EIO; - goto error; - } - - /* - * Page is ok afterall, we are done. - */ - unlock_page(page); - } - - return 0; -error: - unlock_page(page); - return err; -} const struct pipe_buf_operations page_cache_pipe_buf_ops = { .can_merge = 0, - .confirm = page_cache_pipe_buf_confirm, + .confirm = generic_pipe_buf_confirm, .release = page_cache_pipe_buf_release, .steal = page_cache_pipe_buf_steal, .get = generic_pipe_buf_get, -- 2.5.5