Re: [RFC PATCH 0/3] xfs: add memory failure tests for dax mode

Re: [RFC PATCH 0/3] xfs: add memory failure tests for dax mode

[Eryu Guan]

On Mon, Mar 21, 2022 at 11:00:59PM +0800, Shiyang Ruan wrote:
> ping...  any comments?

Sorry for the DELAY.. The tests look fine to me overall, but I'd like
xfs folks to take a look at them as well.

Thanks,
Eryu

> 
> 在 2022/3/11 23:18, Shiyang Ruan 写道:
> > This patchset is to verify whether memory failure mechanism still works
> > with the dax-rmap feature[1].  With this feature, dax and reflink can be
> > used together[2].  So, we also test it for reflinked files in filesystem
> > mounted with dax option.
> > 
> > [1] https://lore.kernel.org/linux-xfs/20220227120747.711169-1-ruansy.fnst@fujitsu.com/
> > [2] https://lore.kernel.org/linux-xfs/20210928062311.4012070-1-ruansy.fnst@fujitsu.com/
> > 
> > Shiyang Ruan (3):
> >    xfs: add memory failure test for dax mode
> >    xfs: add memory failure test for dax&reflink mode
> >    xfs: add memory failure test for partly-reflinked&dax file
> > 
> >   .gitignore                      |   1 +
> >   src/Makefile                    |   3 +-
> >   src/t_mmap_cow_memory_failure.c | 154 ++++++++++++++++++++++++++++++++
> >   tests/xfs/900                   |  48 ++++++++++
> >   tests/xfs/900.out               |   9 ++
> >   tests/xfs/901                   |  49 ++++++++++
> >   tests/xfs/901.out               |   9 ++
> >   tests/xfs/902                   |  52 +++++++++++
> >   tests/xfs/902.out               |   9 ++
> >   9 files changed, 333 insertions(+), 1 deletion(-)
> >   create mode 100644 src/t_mmap_cow_memory_failure.c
> >   create mode 100755 tests/xfs/900
> >   create mode 100644 tests/xfs/900.out
> >   create mode 100755 tests/xfs/901
> >   create mode 100644 tests/xfs/901.out
> >   create mode 100755 tests/xfs/902
> >   create mode 100644 tests/xfs/902.out
> > 
> 

Re: [RFC PATCH 0/3] xfs: add memory failure tests for dax mode

[Zorro Lang]

On Fri, Mar 11, 2022 at 11:18:13PM +0800, Shiyang Ruan wrote:
> This patchset is to verify whether memory failure mechanism still works
> with the dax-rmap feature[1].  With this feature, dax and reflink can be
> used together[2].  So, we also test it for reflinked files in filesystem
> mounted with dax option.
> 
> [1] https://lore.kernel.org/linux-xfs/20220227120747.711169-1-ruansy.fnst@fujitsu.com/
> [2] https://lore.kernel.org/linux-xfs/20210928062311.4012070-1-ruansy.fnst@fujitsu.com/
> 
> Shiyang Ruan (3):
>   xfs: add memory failure test for dax mode
>   xfs: add memory failure test for dax&reflink mode
>   xfs: add memory failure test for partly-reflinked&dax file

This patchset hang here for long time, the cases looks fine, can anyone familiar with
DAX+XFS help to give it a double checking/reviewing :)

Thanks,
Zorro

> 
>  .gitignore                      |   1 +
>  src/Makefile                    |   3 +-
>  src/t_mmap_cow_memory_failure.c | 154 ++++++++++++++++++++++++++++++++
>  tests/xfs/900                   |  48 ++++++++++
>  tests/xfs/900.out               |   9 ++
>  tests/xfs/901                   |  49 ++++++++++
>  tests/xfs/901.out               |   9 ++
>  tests/xfs/902                   |  52 +++++++++++
>  tests/xfs/902.out               |   9 ++
>  9 files changed, 333 insertions(+), 1 deletion(-)
>  create mode 100644 src/t_mmap_cow_memory_failure.c
>  create mode 100755 tests/xfs/900
>  create mode 100644 tests/xfs/900.out
>  create mode 100755 tests/xfs/901
>  create mode 100644 tests/xfs/901.out
>  create mode 100755 tests/xfs/902
>  create mode 100644 tests/xfs/902.out
> 
> -- 
> 2.35.1
> 
> 
>