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 323F3ECAAD3 for ; Mon, 19 Sep 2022 17:35:00 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230266AbiISRe5 (ORCPT ); Mon, 19 Sep 2022 13:34:57 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44926 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230309AbiISRey (ORCPT ); Mon, 19 Sep 2022 13:34:54 -0400 Received: from mail-ot1-x336.google.com (mail-ot1-x336.google.com [IPv6:2607:f8b0:4864:20::336]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2764224BE1; Mon, 19 Sep 2022 10:34:53 -0700 (PDT) Received: by mail-ot1-x336.google.com with SMTP id r34-20020a05683044a200b0065a12392fd7so43977otv.3; Mon, 19 Sep 2022 10:34:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date; bh=kdS9QD9VodJFlDe/hzPt/fIHrfeguLlK063E4oft3xY=; b=RSsN1QZHs/TmzsGp1gwypXdKb/ST8jjoxnGgjyJpGV7sq5aaYponooE63RtJDRuoFk 4VXjc4NPaG7MQomjEcSt+y7p4ZVGuVPclZGjlL/vQiCR0Me8vanyEOYot/zi8Qsubu9j Ix69jBgynF7tCkr+xxKol7c7kNhnVS8KJgxgKk0lyolCfmfJb+3zoq8Vz25NtkyW/5RJ p2+8AeVSPil9bJICaFxtsaSaqUw5Kt5tKUWnYm/Zm3aB+7O1VaahZGVySmK2tZf55QM0 YghzgAsvx0sqLEfTTQQuMHFJo6+U/klj2pt5kixy16lXa6EmRGn5eHCZGXkCezFh7SPi kjXg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date; bh=kdS9QD9VodJFlDe/hzPt/fIHrfeguLlK063E4oft3xY=; b=SbguzxJyq7PJFCL1+mck1F+Cr3HG4z21EO4jQ6MNzua4D84s6vNsQ8jPCkaxg1nUPq V+nA0uP5DdD7UYCozggMhtOkkbP8663qRdb1RuVLfXhtyBkgJJzGhpzMJ6wq0TLYtEIY Lk8PXTmZG8uCDgzWaXT13AxAlF53Pf2j2T0AY9VQKGzY5ZCBQ0MRH0pIvfKeUgZUFnQL ShY3gNpPecpmbq89zksg9BDrLshSHVVi7M/L1OUVyKht0RoaGH574ToAnq2LC4CRRENI O8sVoTkzHT5ASlQ5JzhnBTt7FE/E4D8ROcae1oVn8HmJTBw2FKabkiaOJTKvC0MDDHqP elPw== X-Gm-Message-State: ACrzQf1bcT6uPi0zdPPgs7Vkgf1VhABB8iIo5F6OqZDNC18KqqZUkDlC aWxPVTwVc1msbXng6/Oowcjrj8eBZEkjFYChKa9ES6rf7p5xPQ== X-Google-Smtp-Source: AMsMyM5sQYgUol/jmTSw3stKiJsYQ02a9vmJcS7x25EvIWll4MwqWjrE0KfujNT4rW/qZrnmuFrh2hzLLcoYzG7yGfA= X-Received: by 2002:a05:6830:d8c:b0:639:6034:b3d7 with SMTP id bv12-20020a0568300d8c00b006396034b3d7mr8692869otb.125.1663608892048; Mon, 19 Sep 2022 10:34:52 -0700 (PDT) MIME-Version: 1.0 References: <20220912182224.514561-1-vishal.moola@gmail.com> In-Reply-To: <20220912182224.514561-1-vishal.moola@gmail.com> From: Vishal Moola Date: Mon, 19 Sep 2022 10:34:40 -0700 Message-ID: Subject: Re: [PATCH v2 00/23] Convert to filemap_get_folios_tag() To: linux-fsdevel@vger.kernel.org Cc: linux-afs@lists.infradead.org, linux-kernel@vger.kernel.org, linux-btrfs@vger.kernel.org, ceph-devel@vger.kernel.org, linux-cifs@vger.kernel.org, linux-ext4@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net, cluster-devel@redhat.com, linux-nilfs@vger.kernel.org, linux-mm@kvack.org Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-fsdevel@vger.kernel.org On Mon, Sep 12, 2022 at 11:25 AM Vishal Moola (Oracle) wrote: > > This patch series replaces find_get_pages_range_tag() with > filemap_get_folios_tag(). This also allows the removal of multiple > calls to compound_head() throughout. > It also makes a good chunk of the straightforward conversions to folios, > and takes the opportunity to introduce a function that grabs a folio > from the pagecache. > > F2fs and Ceph have quite alot of work to be done regarding folios, so > for now those patches only have the changes necessary for the removal of > find_get_pages_range_tag(), and only support folios of size 1 (which is > all they use right now anyways). > > I've run xfstests on btrfs, ext4, f2fs, and nilfs2, but more testing may be > beneficial. The page-writeback and filemap changes implicitly work. Testing > and review of the other changes (afs, ceph, cifs, gfs2) would be appreciated. > --- > v2: > Got Acked-By tags for nilfs and btrfs changes > Fixed an error arising in f2fs > - Reported-by: kernel test robot > > Vishal Moola (Oracle) (23): > pagemap: Add filemap_grab_folio() > filemap: Added filemap_get_folios_tag() > filemap: Convert __filemap_fdatawait_range() to use > filemap_get_folios_tag() > page-writeback: Convert write_cache_pages() to use > filemap_get_folios_tag() > afs: Convert afs_writepages_region() to use filemap_get_folios_tag() > btrfs: Convert btree_write_cache_pages() to use > filemap_get_folio_tag() > btrfs: Convert extent_write_cache_pages() to use > filemap_get_folios_tag() > ceph: Convert ceph_writepages_start() to use filemap_get_folios_tag() > cifs: Convert wdata_alloc_and_fillpages() to use > filemap_get_folios_tag() > ext4: Convert mpage_prepare_extent_to_map() to use > filemap_get_folios_tag() > f2fs: Convert f2fs_fsync_node_pages() to use filemap_get_folios_tag() > f2fs: Convert f2fs_flush_inline_data() to use filemap_get_folios_tag() > f2fs: Convert f2fs_sync_node_pages() to use filemap_get_folios_tag() > f2fs: Convert f2fs_write_cache_pages() to use filemap_get_folios_tag() > f2fs: Convert last_fsync_dnode() to use filemap_get_folios_tag() > f2fs: Convert f2fs_sync_meta_pages() to use filemap_get_folios_tag() > gfs2: Convert gfs2_write_cache_jdata() to use filemap_get_folios_tag() > nilfs2: Convert nilfs_lookup_dirty_data_buffers() to use > filemap_get_folios_tag() > nilfs2: Convert nilfs_lookup_dirty_node_buffers() to use > filemap_get_folios_tag() > nilfs2: Convert nilfs_btree_lookup_dirty_buffers() to use > filemap_get_folios_tag() > nilfs2: Convert nilfs_copy_dirty_pages() to use > filemap_get_folios_tag() > nilfs2: Convert nilfs_clear_dirty_pages() to use > filemap_get_folios_tag() > filemap: Remove find_get_pages_range_tag() > > fs/afs/write.c | 114 +++++++++++++++++---------------- > fs/btrfs/extent_io.c | 57 +++++++++-------- > fs/ceph/addr.c | 138 ++++++++++++++++++++-------------------- > fs/cifs/file.c | 33 +++++++++- > fs/ext4/inode.c | 55 ++++++++-------- > fs/f2fs/checkpoint.c | 49 +++++++------- > fs/f2fs/compress.c | 13 ++-- > fs/f2fs/data.c | 69 ++++++++++---------- > fs/f2fs/f2fs.h | 5 +- > fs/f2fs/node.c | 72 +++++++++++---------- > fs/gfs2/aops.c | 64 ++++++++++--------- > fs/nilfs2/btree.c | 14 ++-- > fs/nilfs2/page.c | 59 ++++++++--------- > fs/nilfs2/segment.c | 44 +++++++------ > include/linux/pagemap.h | 32 +++++++--- > include/linux/pagevec.h | 8 --- > mm/filemap.c | 87 ++++++++++++------------- > mm/page-writeback.c | 44 +++++++------ > mm/swap.c | 10 --- > 19 files changed, 507 insertions(+), 460 deletions(-) > > -- > 2.36.1 > Just following up on these patches. Many of the changes still need review. If anyone has time this week to look over any of the affected areas (pagecache, afs, ceph, ciph, ext4, f2fs, or gfs) feedback would be much appreciated. Also, Thanks to David for looking at btrfs and Ryusuke for looking at nilfs already.