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 284DBC77B75 for ; Tue, 23 May 2023 01:06:32 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id B462D900004; Mon, 22 May 2023 21:06:31 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id AF6F0900003; Mon, 22 May 2023 21:06:31 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 99851900004; Mon, 22 May 2023 21:06:31 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id 8C4FB900003 for ; Mon, 22 May 2023 21:06:31 -0400 (EDT) Received: from smtpin08.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay08.hostedemail.com (Postfix) with ESMTP id 5AAA114055A for ; Tue, 23 May 2023 01:06:31 +0000 (UTC) X-FDA: 80819729382.08.7D4C2D5 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf16.hostedemail.com (Postfix) with ESMTP id A3F1E180006 for ; Tue, 23 May 2023 01:06:29 +0000 (UTC) Authentication-Results: imf16.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=B7HpXX0w; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf16.hostedemail.com: domain of djwong@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=djwong@kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1684803989; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=2ej59/T/mMH9enC8u09fLhXt1y6kmoqAgjeImlgOrUg=; b=bIGudLImDX+RkXyy4fnIQdDPgzNMB1BhbMuugPqRCVrmsGpRDC6W4M6HyMFFVafBBN0rN9 yTF4GpYWmY+8hRePuQLAahWk30KIYEKE45mWrbKM92C95ggSJXhOZkFqnQqK/7+3TVj7w5 SkzkB7+HqRhf3cK0ZYX+isafESfO8vE= ARC-Authentication-Results: i=1; imf16.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=B7HpXX0w; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf16.hostedemail.com: domain of djwong@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=djwong@kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1684803989; a=rsa-sha256; cv=none; b=y3bGgUjkCEvJChhyOG1MA64BfDzYzmNModKQ0yEKERYs+04rJNS758F6AMl71ge69fN/Le vbkpmt1NHQSWWgp3O4tpDypysW0CthSbqc8a20EeWw+pogibtpqXBxaH4Iqn2+vi0JazYB XBAy7UbUsk6v7c7AM5hf4gRs61u6LFg= Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 8A5E662CFF; Tue, 23 May 2023 01:06:28 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id E5A8CC433EF; Tue, 23 May 2023 01:06:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1684803988; bh=h4E1J1AKDKMVuSfgh/Fow3jhm+kdCps73if0IpfmP5M=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=B7HpXX0w8KazRHI9JNKr78xlTtBd/1Nh3mIbBpnJMweSFzrOqUOt7vzqrdBkuR6da VpRkQ0B9OD3Lv3dmDJT2f5qM2ZiJ/Kyn1Lq2WhZw69nlhNmf+f3buLVFiU8DbZvv7F UZtgkITdSnN+xy5B/gLJIpuTpnarir62FwLnGDW1sBWKc+tVEtQAX97dY0y9SoCq5l zRaQqJlwCLVdQFoTj4OZUDBxuXLoYm1YXms74/n+fosCJ8AqmV5X1QCSlaFFWigY5U 89PsbK7iFzdJ2y1/246qlBtpGb/jm+WXxujyUQIlwqewLNfDNz7C/tJF/7+ceO+zUq U9+tx48Oj/N1Q== Date: Mon, 22 May 2023 18:06:27 -0700 From: "Darrick J. Wong" To: Christoph Hellwig Cc: Matthew Wilcox , Jens Axboe , Xiubo Li , Ilya Dryomov , Alexander Viro , Christian Brauner , Theodore Ts'o , Jaegeuk Kim , Chao Yu , Miklos Szeredi , Andreas Gruenbacher , Trond Myklebust , Anna Schumaker , Damien Le Moal , Andrew Morton , linux-block@vger.kernel.org, ceph-devel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-ext4@vger.kernel.org, "open list:F2FS FILE SYSTEM" , cluster-devel@redhat.com, linux-xfs@vger.kernel.org, linux-nfs@vger.kernel.org, linux-mm@kvack.org Subject: Re: [PATCH 08/13] iomap: assign current->backing_dev_info in iomap_file_buffered_write Message-ID: <20230523010627.GD11598@frogsfrogsfrogs> References: <20230519093521.133226-1-hch@lst.de> <20230519093521.133226-9-hch@lst.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230519093521.133226-9-hch@lst.de> X-Rspamd-Queue-Id: A3F1E180006 X-Rspam-User: X-Rspamd-Server: rspam05 X-Stat-Signature: z8kfdpaywmzc87kstwf5ihn9qe1owu7s X-HE-Tag: 1684803989-198950 X-HE-Meta: U2FsdGVkX196QRLWV52zW6BBWPo6vTcRZ8H4UTG3gZcIMoKp7aea24cmF1E2Fru6hgbAp7g0zALNJrkDVbKie7yY9sUKtR9K0oYoN/LzMu4MyQ6N4SI7DzzQXW6kP4UtNKX/4840PGG5yIGkCn+81BXGYhQ+mvLSvCo30QS0TBmK+oRH6WjYbs2WNPr1NWjdvVjt9HHatzbKKQfgHPEA6SNYG/OmH1bE62Bx82KbMXm6XnxNihNpcq0meURvhVROwoAMRcQpApMRxscjeonMedQfQzeZoDdiOtlscoaYI5csmjE+PhTA/YU+itYs+kBiLBXuX58jI1ldCWglyOOMancL/mgR0tctnBjZG8anWcmTonhPzps9ENhVTqB7DSdYjkYYF8FRbW/4c0GfmE+ECGIyQp3pxNPZeZ51kRsFqf40bUHSpr4yKpKX29Rc6Xh5FQ/LOekry/Y9OXyCBrpvA2Vdo7UkE7BcC5EX7Yr2hYHYIPQ7cwvIXeCsQj2yW+0WBKTcgrHkXnj6VQLaxSrbew6+tkyMIeOTQP1UbKOjs5QTSROVu2pSjUT8BNU6LA+QO2ZD1OXE73hA7iSiIoHkSAhFaeJnpKivzszqgznLiOdE6y9OrBHeBcofMqBpwg+2pekCp3KfmcZgDUW9n931L2fQWsd5S3Y6NcEwPUnHNPTp/fvQn9GwTgWhF2vkjb1mgZzp5gMX7Ds2t2/OtZkuVp0r11kuqBWhk+mygw9ZpgrCrG7oA10R8P/WOloem+7REOmvl709oMWtJINhaP19f8QJF9TYek0X2GOvq8UC6AZDyWHa4qXO2dmLIbKg4OgscTr+SVjKQxn1eAQpuT805qKvfRTuxHcU9LILZubf6XFfvj2q2WK0TYmSJHkfa6WLXzjfc1GWwap405klWtKUbUz/u8xZsrH2uk3l/3r4fH2S59Tkl450B/nR012ae0CE/6CUm1wwmxWhOZg0KSx 17yXA3EY rRNDf+6qjG0Q0N2aIof7jN7I9SIikmgiGhDa+M6s5wfmA1NEU67nwgZz70OGXYXdMboA3M+5468SmdN5XmWXKv0XOxUJN9Bg+DuzX0CrMl0nqFn++rOJHFvaAJgHkmjXG380TTC+V0oJFds8Sz2lfcn/xNpV0JnX2ZL+9PtMInElYPOBmVVyvt7Q9h1Un+MuPPnJzaO+li47wujBZG67bk73wJ2pLAilcngUlMGVliepKPYWQXRh2y5JJMVbtVnAHDQKyhB4pu7wpBDTzjvCn8xawA0hcBiYCiszbihCWMQGLdMFNBwpEjJRKBByrpHs7f4zu+445EJRJmxZB6MrT6cZbkw== 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 Fri, May 19, 2023 at 11:35:16AM +0200, Christoph Hellwig wrote: > Move the assignment to current->backing_dev_info from the callers into > iomap_file_buffered_write to reduce boiler plate code and reduce the > scope to just around the page dirtying loop. > > Note that zonefs was missing this assignment before. I'm still wondering (a) what the hell current->backing_dev_info is for, and (b) if we need it around the iomap_unshare operation. $ git grep current..backing_dev_info fs/btrfs/file.c:1148: current->backing_dev_info = inode_to_bdi(inode); fs/btrfs/file.c:1169: current->backing_dev_info = NULL; fs/btrfs/file.c:1692: current->backing_dev_info = NULL; fs/ceph/file.c:1795: current->backing_dev_info = inode_to_bdi(inode); fs/ceph/file.c:1943: current->backing_dev_info = NULL; fs/ext4/file.c:288: current->backing_dev_info = inode_to_bdi(inode); fs/ext4/file.c:290: current->backing_dev_info = NULL; fs/f2fs/file.c:4520: current->backing_dev_info = inode_to_bdi(inode); fs/f2fs/file.c:4522: current->backing_dev_info = NULL; fs/fuse/file.c:1366: current->backing_dev_info = inode_to_bdi(inode); fs/fuse/file.c:1412: current->backing_dev_info = NULL; fs/gfs2/file.c:1044: current->backing_dev_info = inode_to_bdi(inode); fs/gfs2/file.c:1048: current->backing_dev_info = NULL; fs/nfs/file.c:652: current->backing_dev_info = inode_to_bdi(inode); fs/nfs/file.c:654: current->backing_dev_info = NULL; fs/ntfs/file.c:1914: current->backing_dev_info = inode_to_bdi(vi); fs/ntfs/file.c:1918: current->backing_dev_info = NULL; fs/ntfs3/file.c:823: current->backing_dev_info = inode_to_bdi(inode); fs/ntfs3/file.c:996: current->backing_dev_info = NULL; fs/xfs/xfs_file.c:721: current->backing_dev_info = inode_to_bdi(inode); fs/xfs/xfs_file.c:756: current->backing_dev_info = NULL; mm/filemap.c:3995: current->backing_dev_info = inode_to_bdi(inode); mm/filemap.c:4056: current->backing_dev_info = NULL; AFAICT nobody uses it at all? Unless there's some bizarre user that isn't extracting it from @current? Oh, hey, new question (c) isn't this set incorrectly for xfs realtime files? --D > Signed-off-by: Christoph Hellwig > --- > fs/gfs2/file.c | 3 --- > fs/iomap/buffered-io.c | 3 +++ > fs/xfs/xfs_file.c | 5 ----- > 3 files changed, 3 insertions(+), 8 deletions(-) > > diff --git a/fs/gfs2/file.c b/fs/gfs2/file.c > index 499ef174dec138..261897fcfbc495 100644 > --- a/fs/gfs2/file.c > +++ b/fs/gfs2/file.c > @@ -25,7 +25,6 @@ > #include > #include > #include > -#include > #include > > #include "gfs2.h" > @@ -1041,11 +1040,9 @@ static ssize_t gfs2_file_buffered_write(struct kiocb *iocb, > goto out_unlock; > } > > - current->backing_dev_info = inode_to_bdi(inode); > pagefault_disable(); > ret = iomap_file_buffered_write(iocb, from, &gfs2_iomap_ops); > pagefault_enable(); > - current->backing_dev_info = NULL; > if (ret > 0) > written += ret; > > diff --git a/fs/iomap/buffered-io.c b/fs/iomap/buffered-io.c > index 550525a525c45c..b2779bd1f10611 100644 > --- a/fs/iomap/buffered-io.c > +++ b/fs/iomap/buffered-io.c > @@ -3,6 +3,7 @@ > * Copyright (C) 2010 Red Hat, Inc. > * Copyright (C) 2016-2019 Christoph Hellwig. > */ > +#include > #include > #include > #include > @@ -869,8 +870,10 @@ iomap_file_buffered_write(struct kiocb *iocb, struct iov_iter *i, > if (iocb->ki_flags & IOCB_NOWAIT) > iter.flags |= IOMAP_NOWAIT; > > + current->backing_dev_info = inode_to_bdi(iter.inode); > while ((ret = iomap_iter(&iter, ops)) > 0) > iter.processed = iomap_write_iter(&iter, i); > + current->backing_dev_info = NULL; > > if (unlikely(ret < 0)) > return ret; > diff --git a/fs/xfs/xfs_file.c b/fs/xfs/xfs_file.c > index bfba10e0b0f3c2..98d763cc3b114c 100644 > --- a/fs/xfs/xfs_file.c > +++ b/fs/xfs/xfs_file.c > @@ -27,7 +27,6 @@ > > #include > #include > -#include > #include > #include > #include > @@ -717,9 +716,6 @@ xfs_file_buffered_write( > if (ret) > goto out; > > - /* We can write back this queue in page reclaim */ > - current->backing_dev_info = inode_to_bdi(inode); > - > trace_xfs_file_buffered_write(iocb, from); > ret = iomap_file_buffered_write(iocb, from, > &xfs_buffered_write_iomap_ops); > @@ -751,7 +747,6 @@ xfs_file_buffered_write( > goto write_retry; > } > > - current->backing_dev_info = NULL; > out: > if (iolock) > xfs_iunlock(ip, iolock); > -- > 2.39.2 >