From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756392AbcI0Dp0 (ORCPT ); Mon, 26 Sep 2016 23:45:26 -0400 Received: from zeniv.linux.org.uk ([195.92.253.2]:47096 "EHLO ZenIV.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753628AbcI0DpR (ORCPT ); Mon, 26 Sep 2016 23:45:17 -0400 Date: Tue, 27 Sep 2016 04:45:15 +0100 From: Al Viro To: Miklos Szeredi Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 09/11] splice: use get_page_for_read() Message-ID: <20160927034515.GI19539@ZenIV.linux.org.uk> References: <1473842236-28655-1-git-send-email-mszeredi@redhat.com> <1473842236-28655-10-git-send-email-mszeredi@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1473842236-28655-10-git-send-email-mszeredi@redhat.com> User-Agent: Mutt/1.7.0 (2016-08-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Sep 14, 2016 at 10:37:14AM +0200, Miklos Szeredi wrote: > What __generic_file_splice_read() does is get a series of uptodate pages > and put them into the pipe buffer. > > The get_page_for_read() helper can now be used to get the pages, > simplifying the code and making sure the splice(2) stays in sync with > read(2). > > For example get_page_for_read() can handle partially uptodate pages and now > splice can take advantage of these as well. ... or, better yet, kill __generic_file_splice_read() off.