On 21/04/07 02:32PM, Shiyang Ruan wrote: > The core logic in the two dax page fault functions is similar. So, move > the logic into a common helper function. Also, to facilitate the > addition of new features, such as CoW, switch-case is no longer used to > handle different iomap types. > > Signed-off-by: Shiyang Ruan <ruansy.f...@fujitsu.com> > Reviewed-by: Christoph Hellwig <h...@lst.de> > --- > fs/dax.c | 294 ++++++++++++++++++++++++++++--------------------------- > 1 file changed, 148 insertions(+), 146 deletions(-)
Thanks for addressing comments. A msg in cover letter acknowledging that the review comments mentioned here[1] were addressed is helpful for everyone. Please feel free to add. Reviewed-by: Ritesh Harjani <rite...@linux.ibm.com> [1]: https://patchwork.kernel.org/project/linux-nvdimm/patch/20210319015237.993880-3-ruansy.f...@fujitsu.com/ -ritesh