mbox series

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

Message ID 20220311151816.2174870-1-ruansy.fnst@fujitsu.com (mailing list archive)
Headers show
Series xfs: add memory failure tests for dax mode | expand

Message

Shiyang Ruan March 11, 2022, 3:18 p.m. UTC
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

Comments

Shiyang Ruan March 21, 2022, 3 p.m. UTC | #1
ping...  any comments?

在 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
>
Eryu Guan April 10, 2022, 4:08 p.m. UTC | #2
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
> > 
>
Zorro Lang April 27, 2022, 6:22 a.m. UTC | #3
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
> 
> 
>