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 54BB0C77B7C for ; Mon, 22 May 2023 00:05:08 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id DA1E6900002; Sun, 21 May 2023 20:05:07 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id D52A26B0075; Sun, 21 May 2023 20:05:07 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id C1937900002; Sun, 21 May 2023 20:05:07 -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 AAB1F6B0074 for ; Sun, 21 May 2023 20:05:07 -0400 (EDT) Received: from smtpin11.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay06.hostedemail.com (Postfix) with ESMTP id 7AA2CADF3B for ; Mon, 22 May 2023 00:05:07 +0000 (UTC) X-FDA: 80815945854.11.FD64A58 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf16.hostedemail.com (Postfix) with ESMTP id A13D6180010 for ; Mon, 22 May 2023 00:05:05 +0000 (UTC) Authentication-Results: imf16.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=tg9VRhD3; spf=pass (imf16.hostedemail.com: domain of dlemoal@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=dlemoal@kernel.org; dmarc=pass (policy=none) header.from=kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1684713905; 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:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=vMyxo8JcHGinlxUdMcuInfNb7vz2Ov9+urCxYD/rlUA=; b=CQRkoc3y81BDA2yQfrVezCIwBgwct6qoA0ksa4xZhZ5sdwi2kmPWAC9DMLG4QpGP0Jiy0e tjP3fAzZtNGCAAMXvOmtYKYFq8ZIU7xZVsqBXsO8mSiqq80IiSXFXOkPoKrV3+BmJxKyiZ iVjCrkPL+5UDWozBkJthKkgNq5Jrgfg= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1684713905; a=rsa-sha256; cv=none; b=vzLkAso3D7tryBW3RBT+skJupEGFZmp1ORkGs0VjrSk/t2GrOaqTLqRREvoaQALzLINZhD y9qCKahio/E/SDbsN3CT89V7IyVLbcllau2n2JgXHHtU7sMn20oe4FA7whD2vg/+tTBfGz rE9VHbCiz7Ou6x0Sb1TiCc9Xrz9bSNM= ARC-Authentication-Results: i=1; imf16.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=tg9VRhD3; spf=pass (imf16.hostedemail.com: domain of dlemoal@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=dlemoal@kernel.org; dmarc=pass (policy=none) header.from=kernel.org 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 A77DF6174C; Mon, 22 May 2023 00:05:04 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 750D7C433EF; Mon, 22 May 2023 00:05:01 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1684713904; bh=E9HylG/tcZ9ypOKQIb0ALyQo4ZdadXJ2F5MM9xsQvIo=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=tg9VRhD3pU5E6h82uRSebiA2CoUfSFazrPXhAdkuMdw/YHz7TizGQVPxH7sf1bybJ 9pVjxyd9RD6Esi/udTHpfhazC7ni+xDnJ4tp7E6eb55JCV0Ys8k/lLJ4eUDSxRqvq9 LJERPMDd3prQur8f5Edsobj6QFKiaT4Fgpq94NE2coTNbZwqY1W+WFhGHfAI4sp6Z8 DlkD0MMz0RQOpRdIUO3g4Qdn9cn0kk3tJJQwapFPHbh7Lv+sp54Q+MR5U2VlJ16StD Hgj+jgofNL7e7t3WCATjtmEpaHL/w1gXK0aWokjb1cSD/3mfQKZv5EhJQmXTLNRAyQ ZG2tBsVHf7xCA== Message-ID: <88fe9652-3cf5-c601-08f1-64b75e367ca5@kernel.org> Date: Mon, 22 May 2023 09:05:00 +0900 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0 Subject: Re: [PATCH 08/13] iomap: assign current->backing_dev_info in iomap_file_buffered_write Content-Language: en-US To: Christoph Hellwig , Matthew Wilcox Cc: Jens Axboe , Xiubo Li , Ilya Dryomov , Alexander Viro , Christian Brauner , Theodore Ts'o , Jaegeuk Kim , Chao Yu , Miklos Szeredi , Andreas Gruenbacher , "Darrick J. Wong" , Trond Myklebust , Anna Schumaker , 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 References: <20230519093521.133226-1-hch@lst.de> <20230519093521.133226-9-hch@lst.de> From: Damien Le Moal Organization: Western Digital Research In-Reply-To: <20230519093521.133226-9-hch@lst.de> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Rspamd-Queue-Id: A13D6180010 X-Rspam-User: X-Stat-Signature: wz89qzsnsry1iwrzbjz59m7yorbd5hpp X-Rspamd-Server: rspam03 X-HE-Tag: 1684713905-578117 X-HE-Meta: U2FsdGVkX1/XnxEPVKhfR/Gz+YEnN4+lMK8oYitUpqerbIMf/hoCoNPNyNKkC/VcgAXThlBLPHhQ3Br12cubZtZ6nfMIpebLkpDmVuZ3eycYAE6Mbc7VA0MiFdbHauV+GHXxHAta9r9fAAqn4WMjfhX7uSFOcl+eghwQTu/kBkaHw3fqF2ji46QAKBBFZqudzfWAOpPqAFdhSKaD11Tx6pfUJsFdDmqsUVh0x0pjXhbBF525x0GcnQuozer6In1OWkkFbMLE3VmUx8GiVzmQtBHjgYXgRMzb9b2qTwT7coqHOBAdQUpGfSCO01BLSlycDsBhBikYjmM0GJ+DeZzAe47hQ/NYL1p7ZwNtXlv2acDUkUx5s3f+R90WSjMCSrJo+DnyXIMkdSRObXmpqTjbcy7k3wVv9KsoYZhpJYLT2J07Dqsqi/wHQHiK3yVcmRZ7we5wEmnNkbfX/cCwFvRhcfV0r7GD7h+dMr46muh91WsIAR+Wz3lIMiFEYggvsAZICYbC9iPg9I/Tm070BuQSQpx/tOA1s/6NZhl9sYIugwxJ7hL+l6HE/KJRiswZsOHVezdECxl4McL0nyt8ji0jziMzxspi+UN1WEGfPjh3iBPcYONlThAXHC6+JWqv2gOJEw/6NZ2JISBaBm8mD+TO77BW4C04VJznqKSB2Z6LwYrOMNMVhNBphxu6wzHhOkshF/wUEDxdNGnDb7C60qyJ6N2ScPsPwpC35q/SUvt2QFF1fbKBZBM+71EIcsaXoISKZvfU+KUqVU1Sz1AHYpgx9yGomvMGRCrMYZ2GQYyNsYY+BBwO1WoYcao/8BiFl2Chjo1EjIECVuKPUAosCk7iZvCLVCGubm+myh0L7FpRM4MqipIG5FmUPWjRfuRKoiGjv0PelTEKGOM81c5cyblPzOlwA+UfOtrXkVRucIZyPn/2qIxRvtkKZ0EJN9P4EBMfYJSCGYJfn1o6qSfZOoK MCHPY9t/ BQtB9jN57eh1qwYjMoUdTdgbOoZKKX3CyRMyucvYqdkB9+gmPYjX79t6yr11bpKx8Ukpux9Hiidl3nlhLIHBGP2FcrJts/cP7z3KwtGrCxndObBWcAEHyyO6Ionqi8yt9VZMleiGikcjHp7SU34JY548GDgSIyLTf8N59s7sSHVkzn7lg/Uk8UOAMNOQkRxaD3b2GWb3IWf2KppEyBsdlG7EUnKwQXdFG7xzydhUoPbPEx2k+r8j9X6Mtv/bP/pExTjBVu8X1VaWYCBCo7afxN4B/nGF7gu1q1hZUzsjdefXAYv1g8FjeG2l5ZYScLa4ll+1jD3StU+flct4NuIl9tvXr+0KzzBMA3Z7Ed0OTEUPDXT2LLj2Egdd0/rV1izywgdLUPwS+RCmSeK8o/1PIuJD0Q1UplkRHyjPNn74WnqhFW5yWceaw1AaVvg== 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 5/19/23 18:35, 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. Hu... Shouldn't this be fixed as a separate patch with a Fixes tag for this cycle ? I have never noticed any issues with this missing though. Not sure how an issue can be triggered with this assignment missing. Apart from that, this patch look good to me. Reviewed-by: Damien Le Moal -- Damien Le Moal Western Digital Research