From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3CE41C43603 for ; Wed, 11 Dec 2019 00:28:22 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 06F502073D for ; Wed, 11 Dec 2019 00:28:21 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 06F502073D Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=fromorbit.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 9F5196B2F18; Tue, 10 Dec 2019 19:28:21 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 9A5D86B2F19; Tue, 10 Dec 2019 19:28:21 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 8E3326B2F1A; Tue, 10 Dec 2019 19:28:21 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0245.hostedemail.com [216.40.44.245]) by kanga.kvack.org (Postfix) with ESMTP id 7454D6B2F18 for ; Tue, 10 Dec 2019 19:28:21 -0500 (EST) Received: from smtpin25.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with SMTP id 1644D8249980 for ; Wed, 11 Dec 2019 00:28:21 +0000 (UTC) X-FDA: 76250974002.25.dog80_ea364d1cde62 X-HE-Tag: dog80_ea364d1cde62 X-Filterd-Recvd-Size: 2510 Received: from mail104.syd.optusnet.com.au (mail104.syd.optusnet.com.au [211.29.132.246]) by imf09.hostedemail.com (Postfix) with ESMTP for ; Wed, 11 Dec 2019 00:28:20 +0000 (UTC) Received: from dread.disaster.area (pa49-195-139-249.pa.nsw.optusnet.com.au [49.195.139.249]) by mail104.syd.optusnet.com.au (Postfix) with ESMTPS id 459837EA491; Wed, 11 Dec 2019 11:28:19 +1100 (AEDT) Received: from dave by dread.disaster.area with local (Exim 4.92.3) (envelope-from ) id 1ieprT-0006Z1-IC; Wed, 11 Dec 2019 11:28:15 +1100 Date: Wed, 11 Dec 2019 11:28:15 +1100 From: Dave Chinner To: Jens Axboe Cc: linux-mm@kvack.org, linux-fsdevel@vger.kernel.org, linux-block@vger.kernel.org Subject: Re: [PATCH 3/5] mm: make buffered writes work with RWF_UNCACHED Message-ID: <20191211002815.GD19213@dread.disaster.area> References: <20191210162454.8608-1-axboe@kernel.dk> <20191210162454.8608-4-axboe@kernel.dk> <20191211002349.GC19213@dread.disaster.area> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20191211002349.GC19213@dread.disaster.area> User-Agent: Mutt/1.10.1 (2018-07-13) X-Optus-CM-Score: 0 X-Optus-CM-Analysis: v=2.3 cv=X6os11be c=1 sm=1 tr=0 a=KoypXv6BqLCQNZUs2nCMWg==:117 a=KoypXv6BqLCQNZUs2nCMWg==:17 a=jpOVt7BSZ2e4Z31A5e1TngXxSK0=:19 a=kj9zAlcOel0A:10 a=pxVhFHJ0LMsA:10 a=7-415B0cAAAA:8 a=YySyZI0ml3ss4FvGJbkA:9 a=CjuIK1q_8ugA:10 a=biEYGPWJfzWAr4FL6Ov7:22 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Wed, Dec 11, 2019 at 11:23:49AM +1100, Dave Chinner wrote: > On Tue, Dec 10, 2019 at 09:24:52AM -0700, Jens Axboe wrote: > > If RWF_UNCACHED is set for io_uring (or pwritev2(2)), we'll drop the > > cache instantiated for buffered writes. If new pages aren't > > instantiated, we leave them alone. This provides similar semantics to > > reads with RWF_UNCACHED set. > > So what about filesystems that don't use generic_perform_write()? > i.e. Anything that uses the iomap infrastructure (i.e. > iomap_file_buffered_write()) instead of generic_file_write_iter()) > will currently ignore RWF_UNCACHED. That's XFS and gfs2 right now, > but there are likely to be more in the near future as more > filesystems are ported to the iomap infrastructure. Hmmm - I'm missing part of this patchset, and I see a second version has been posted that has iomap stuff in it. I'll go look at that now... Cheers, Dave. -- Dave Chinner david@fromorbit.com