Hi, Dan Do you have any comments on this?

-- Thanks, Ruan Shiyang. > -----Original Message----- > From: Shiyang Ruan <ruansy.fnst(a)fujitsu.com&gt; > Sent: Thursday, April 22, 2021 9:45 PM > Subject: [PATCH v3 0/3] fsdax: Factor helper functions to simplify the > code > > From: Shiyang Ruan <ruansy.fnst(a)cn.fujitsu.com&gt; > > The page fault part of fsdax code is little complex. In order to add > CoW feature and make it easy to understand, I was suggested to factor > some helper functions to simplify the current dax code. > > This is separated from the previous patchset called "V3 fsdax,xfs: Add > reflink&dedupe support for fsdax", and the previous comments are here[1]. > > [1]: > https://patchwork.kernel.org/project/linux-nvdimm/patch/20210319015237 > .99 > 3880-3-ruansy.fnst(a)fujitsu.com/ > > Changes from V2: > - fix the type of 'major' in patch 2 > - Rebased on v5.12-rc8 > > Changes from V1: > - fix Ritesh's email address > - simplify return logic in dax_fault_cow_page() > > (Rebased on v5.12-rc8) > == > > Shiyang Ruan (3): > fsdax: Factor helpers to simplify dax fault code > fsdax: Factor helper: dax_fault_actor() > fsdax: Output address in dax_iomap_pfn() and rename it > > fs/dax.c | 443 > +++++++++++++++++++++++++++++-------------------------- > 1 file changed, 234 insertions(+), 209 deletions(-) > > -- > 2.31.1