Re: [PATCH 2/3] fsdax: Factor helper: dax_fault_actor()

2021年4月07日 03:22:15 -0700

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 <[email protected]>
> Reviewed-by: Christoph Hellwig <[email protected]>
> ---
> 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 <[email protected]>
[1]: 
https://patchwork.kernel.org/project/linux-nvdimm/patch/[email protected]/
-ritesh

Reply via email to