Re: [PATCH v2 00/11] Convert (most of) ntfs3 to use folios

[Konstantin Komarov <almaz.alexandrovich@paragon-software.com>]

On 22.04.2024 22:31, Matthew Wilcox (Oracle) wrote:
> I'm not making any attempt here to support large folios.  This is just
> to remove uses of the page-based APIs.  There are still a number of
> places in ntfs3 which use a struct page, but this is a good start on
> the conversions.
>
> v2:
>   - Rebase on next-20240422
>   - Drop "Convert reading $AttrDef to use folios", "Use a folio to read
>     UpCase", "Remove inode_write_data()" and "Remove ntfs_map_page and
>     ntfs_unmap_page" due to changes.
>   - Add "Convert inode_read_data() to use folios", "Remove calls to
>     set/clear the error flag", "Convert attr_wof_frame_info() to use a
>     folio", "Convert ntfs_get_frame_pages() to use a folio", "Convert
>     ni_readpage_cmpr() to take a folio"
>
> Matthew Wilcox (Oracle) (11):
>    ntfs3: Convert ntfs_read_folio to use a folio
>    ntfs3: Convert ntfs_write_begin to use a folio
>    ntfs3: Convert attr_data_read_resident() to take a folio
>    ntfs3: Convert ntfs_write_end() to work on a folio
>    ntfs3: Convert attr_data_write_resident to use a folio
>    ntfs3: Convert attr_make_nonresident to use a folio
>    ntfs3: Convert inode_read_data() to use folios
>    ntfs3: Remove calls to set/clear the error flag
>    ntfs3: Convert attr_wof_frame_info() to use a folio
>    ntfs3: Convert ntfs_get_frame_pages() to use a folio
>    ntfs3: Convert ni_readpage_cmpr() to take a folio
>
>   fs/ntfs3/attrib.c  | 94 ++++++++++++++++++++--------------------------
>   fs/ntfs3/file.c    | 17 +++++----
>   fs/ntfs3/frecord.c | 29 +++++++-------
>   fs/ntfs3/inode.c   | 73 ++++++++++++++++++-----------------
>   fs/ntfs3/ntfs_fs.h |  8 ++--
>   5 files changed, 102 insertions(+), 119 deletions(-)
>
Hi Matthew,

We have started testing the switch to folio as you proposed in v1.
Some of our tests went down.

After adapting I will add your patches with some minor changes.