> > Hi, Dan > > Do you have any comments on this? Ping > > > -- > Thanks, > Ruan Shiyang. > > > -----Original Message----- > > From: Shiyang Ruan <ruansy.fnst@xxxxxxxxxxx> > > 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@xxxxxxxxxxxxxx> > > > > 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@xxxxxxxxxxx/ > > > > 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 > >
