Received: by 2002:a05:6358:3188:b0:123:57c1:9b43 with SMTP id q8csp2260461rwd; Sun, 21 May 2023 17:08:22 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ4zzE0nvnihSiJ704fAHwXegeemyqtOJuTVjKsgttO+A9eYBXtz8W8luPWjK36yVP7lkiHc X-Received: by 2002:a17:903:24c:b0:1ac:3ddf:2299 with SMTP id j12-20020a170903024c00b001ac3ddf2299mr10470639plh.44.1684714102063; Sun, 21 May 2023 17:08:22 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1684714102; cv=none; d=google.com; s=arc-20160816; b=PHB8pIOZrFHy9oQYRq+cRU8UhK1RHU240uR29m5jeg3nPTX04X52PMgEFwkgDk1h5x q14HeqSUmO9dbPLIZLjQ6b3wQslYLlPV9Xjrn0PExLHEY3NVLHTnWCH/pgZXnmgjuYuE etifT8KCe2hLUjT95wgmLAoLHErgNafEqtagz+LXYRSlDdvJ3HcwTuxRhsXUrczwPzft CCs+cHCz1vnT7ooshHsGUGlgyCq67D/W2mtfl3yHCQzbg/4RcRQY5Xw2NglFLqDGs+51 xmbTeu6v5CxbbQz1IeVxuVa5azGeYUHLGOPAPveJZqjQ+MnWRL1liST4F1Q24s091hc3 GUyw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to :organization:from:references:cc:to:content-language:subject :user-agent:mime-version:date:message-id:dkim-signature; bh=vMyxo8JcHGinlxUdMcuInfNb7vz2Ov9+urCxYD/rlUA=; b=dfZI9k0DCPZHhCT8yumOF+zAqo6jmjlM7k+9GHbC8uXkk3NA9Vstq0eKw0ntXEOvTf fuqaKfx+nIrIGo4oIAAFGbxUuW/UaAVwJdOsuMwybDOI3JTmTa0vTPOLGFF2nf4udK1r KTyyKwanSf5DEgvdrMAe5bn2q1x0F4fwHyFPF4Vkjde6LG5FZL0yVruuRUtjNa4rH5th jL6HcQi/XTG0sZyxElpPEFhhQ0FcJJvtK5nIUaP1o1JIsdH74fDJZUiNiqU7Oe2oxV44 xm4fGlR8YKPgt2gKpIdAGLIjGZBtkjQgHEgLl78uJBRrhW+QV1PZMH9UZuixh55BphfV O3Sw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=tg9VRhD3; spf=pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-ext4-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id n13-20020a170902e54d00b001a9baadecc0si2131174plf.160.2023.05.21.17.08.08; Sun, 21 May 2023 17:08:22 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=tg9VRhD3; spf=pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-ext4-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229689AbjEVAFG (ORCPT + 99 others); Sun, 21 May 2023 20:05:06 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59762 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229481AbjEVAFG (ORCPT ); Sun, 21 May 2023 20:05:06 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1C40F9D; Sun, 21 May 2023 17:05:05 -0700 (PDT) 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 A46CB612A5; 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-Spam-Status: No, score=-4.5 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A, RCVD_IN_DNSWL_MED,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-ext4@vger.kernel.org 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