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 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id EE4BBC43334 for ; Fri, 3 Jun 2022 02:43:35 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id F015F6B0071; Thu, 2 Jun 2022 22:43:34 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id EAC756B0073; Thu, 2 Jun 2022 22:43:34 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id D9CC66B0074; Thu, 2 Jun 2022 22:43:34 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id CA3326B0071 for ; Thu, 2 Jun 2022 22:43:34 -0400 (EDT) Received: from smtpin06.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id 9911160820 for ; Fri, 3 Jun 2022 02:43:34 +0000 (UTC) X-FDA: 79535378748.06.94909ED Received: from mail105.syd.optusnet.com.au (mail105.syd.optusnet.com.au [211.29.132.249]) by imf04.hostedemail.com (Postfix) with ESMTP id 265584006B for ; Fri, 3 Jun 2022 02:43:13 +0000 (UTC) Received: from dread.disaster.area (pa49-181-2-147.pa.nsw.optusnet.com.au [49.181.2.147]) by mail105.syd.optusnet.com.au (Postfix) with ESMTPS id AAE3710E6D17; Fri, 3 Jun 2022 12:43:31 +1000 (AEST) Received: from dave by dread.disaster.area with local (Exim 4.92.3) (envelope-from ) id 1nwxHd-0021or-An; Fri, 03 Jun 2022 12:43:29 +1000 Date: Fri, 3 Jun 2022 12:43:29 +1000 From: Dave Chinner To: Jens Axboe Cc: Stefan Roesch , io-uring@vger.kernel.org, kernel-team@fb.com, linux-mm@kvack.org, linux-xfs@vger.kernel.org, linux-fsdevel@vger.kernel.org, jack@suse.cz, hch@infradead.org Subject: Re: [PATCH v7 00/15] io-uring/xfs: support async buffered writes Message-ID: <20220603024329.GI1098723@dread.disaster.area> References: <20220601210141.3773402-1-shr@fb.com> <545ab14b-a95a-de2e-dbc6-f5688b09b47c@kernel.dk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <545ab14b-a95a-de2e-dbc6-f5688b09b47c@kernel.dk> X-Optus-CM-Score: 0 X-Optus-CM-Analysis: v=2.4 cv=e9dl9Yl/ c=1 sm=1 tr=0 ts=62997554 a=ivVLWpVy4j68lT4lJFbQgw==:117 a=ivVLWpVy4j68lT4lJFbQgw==:17 a=kj9zAlcOel0A:10 a=JPEYwPQDsx4A:10 a=7-415B0cAAAA:8 a=STQVsATjAyQOeSVB2IsA:9 a=CjuIK1q_8ugA:10 a=igBNqPyMv6gA:10 a=biEYGPWJfzWAr4FL6Ov7:22 X-Rspamd-Queue-Id: 265584006B X-Stat-Signature: bds6xcordn4gj4xoiuksp916qsuj7u9g X-Rspam-User: Authentication-Results: imf04.hostedemail.com; dkim=none; dmarc=none; spf=none (imf04.hostedemail.com: domain of david@fromorbit.com has no SPF policy when checking 211.29.132.249) smtp.mailfrom=david@fromorbit.com X-Rspamd-Server: rspam08 X-HE-Tag: 1654224193-8048 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 Thu, Jun 02, 2022 at 02:09:00AM -0600, Jens Axboe wrote: > On 6/1/22 3:01 PM, Stefan Roesch wrote: > > This patch series adds support for async buffered writes when using both > > xfs and io-uring. Currently io-uring only supports buffered writes in the > > slow path, by processing them in the io workers. With this patch series it is > > now possible to support buffered writes in the fast path. To be able to use > > the fast path the required pages must be in the page cache, the required locks > > in xfs can be granted immediately and no additional blocks need to be read > > form disk. > > This series looks good to me now, but will need some slight rebasing > since the 5.20 io_uring branch has split up the code a bit. Trivial to > do though, I suspect it'll apply directly if we just change > fs/io_uring.c to io_uring/rw.c instead. > > The bigger question is how to stage this, as it's touching a bit of fs, > mm, and io_uring... What data integrity testing has this had? Has it been run through a few billion fsx operations with w/ io_uring read/write enabled? Cheers, Dave. -- Dave Chinner david@fromorbit.com