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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 77673C4332F for ; Thu, 17 Nov 2022 06:32:10 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239248AbiKQGcI (ORCPT ); Thu, 17 Nov 2022 01:32:08 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36894 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232809AbiKQGcG (ORCPT ); Thu, 17 Nov 2022 01:32:06 -0500 Received: from bombadil.infradead.org (bombadil.infradead.org [IPv6:2607:7c80:54:3::133]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C9D5C3136A; Wed, 16 Nov 2022 22:32:05 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=In-Reply-To:Content-Type:MIME-Version :References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=sPTNEKYS/tLrxLQ4oI8+Sww3RL9H9VBsThKBpc9p/Gk=; b=K5/Kn6cFMb9LpqpOmmVr3AL3ai 0vZiISKwXqKUiJyr5NDOKZcFSpggWGsQkawGPm3P7UqdnGkVIGS0HN54pOlcTrJ8rePonZkJs3Cil 8bG5OKkn5iawEo8woWwszoVV1jztfvn8yDDH4UUHHaZgV+LkaxPvvIKtxh7sEHllokJHoVwbKvpOR ENvaQDOMDREuKSfPLrgrnsEd9eK2C9hvWh0vih31N633ucRXxLWAkuGXNyd7W0qzrIZ0pV3+UyCxA ivXc9kNZLeJEgwy4smO9WMtwJHqptMT8zrurd6/ISPrEggAEdLgDwvS6/Hx+XBRJGJhsBKQH7y+wz 1FJWbVoA==; Received: from hch by bombadil.infradead.org with local (Exim 4.94.2 #2 (Red Hat Linux)) id 1ovYR9-00Aqki-1y; Thu, 17 Nov 2022 06:31:47 +0000 Date: Wed, 16 Nov 2022 22:31:47 -0800 From: Christoph Hellwig To: Jan Kara Cc: Christoph Hellwig , Christoph Hellwig , Namjae Jeon , Sungjong Seo , Jan Kara , OGAWA Hirofumi , Mikulas Patocka , Dave Kleikamp , Bob Copeland , linux-fsdevel@vger.kernel.org, linux-ext4@vger.kernel.org, jfs-discussion@lists.sourceforge.net, linux-karma-devel@lists.sourceforge.net, linux-mm@kvack.org Subject: Re: [PATCH 2/9] ext2: remove ->writepageo Message-ID: References: <20221113162902.883850-1-hch@lst.de> <20221113162902.883850-3-hch@lst.de> <20221114104927.k5x4i4uanxskfs6m@quack3> <20221116182040.tecis3dqejsdqnum@quack3> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20221116182040.tecis3dqejsdqnum@quack3> X-SRS-Rewrite: SMTP reverse-path rewritten from by bombadil.infradead.org. See http://www.infradead.org/rpr.html Precedence: bulk List-ID: X-Mailing-List: linux-ext4@vger.kernel.org On Wed, Nov 16, 2022 at 07:20:40PM +0100, Jan Kara wrote: > Looking at the code, IMO the write_one_page() looks somewhat premature > anyway in that place. AFAICS we could handle the writeout using > filemap_write_and_wait() if we moved it to somewhat later moment. So > something like attached patch (only basic testing only so far)? Yes, this looks sensible. Do you want to queue this one and the ext2 and udf patches from this series if the testing works fine? The same transformation should also be done for minix, sysfs and ufs. And a bunch of the others are probaby similar as well.