From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S938615AbcKLDhA convert rfc822-to-8bit (ORCPT ); Fri, 11 Nov 2016 22:37:00 -0500 Received: from mx1.redhat.com ([209.132.183.28]:31951 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S935102AbcKLDg6 (ORCPT ); Fri, 11 Nov 2016 22:36:58 -0500 Content-Type: text/plain; charset=utf-8 Mime-Version: 1.0 (Mac OS X Mail 10.1 \(3251\)) Subject: Re: [git pull] vfs.git From: "Yan, Zheng" In-Reply-To: Date: Sat, 12 Nov 2016 11:36:49 +0800 Cc: Al Viro , Ilya Dryomov , Linux Kernel Mailing List , linux-fsdevel Content-Transfer-Encoding: 8BIT Message-Id: <8ED81D5A-F56B-418A-99A9-2C197D9D2745@redhat.com> References: <20161111060515.GQ19539@ZenIV.linux.org.uk> To: Linus Torvalds X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.25]); Sat, 12 Nov 2016 03:36:57 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > On 12 Nov 2016, at 01:25, Linus Torvalds wrote: > > On Thu, Nov 10, 2016 at 10:05 PM, Al Viro wrote: >> Christoph's and Jan's aio fixes, fixup for generic_file_splice_read >> (removal of pointless detritus that actually breaks it when used for gfs2 >> ->splice_read()) and fixup for generic_file_read_iter() interaction with >> ITER_PIPE destinations. > > Hmm. I also just pulled the Ceph update that has commit 8a8d56176635 > ("ceph: use default file splice read callback"). I _think_ this splice > fix makes that ceph change unnecessary. But testing is always good. The commit is still needed. Al only fixes ITER_PIPE interaction with direct_IO. (it’s a no-op) Cephfs case is special. Depending on what capabilities client has, client is allowed or disallowed to read data from page cache. MDS changes client’s capabilities dynamically. We don’t want to splice read fail when client is disallowed to get page from page cache. Regards Yan, Zheng > Ilya? Can you double-check the current -git tree (well, what I *will* > push out soon after it has passed my build tests)? > > Because I think Ceph can go back to using generic_file_splice_read again. > > Linus