On Thu, May 19, 2022 at 09:53:23AM +0800, Shiyang Ruan wrote:
>
>
> 在 2022/5/19 2:38, Zorro Lang 写道:
> > On Fri, May 13, 2022 at 11:14:09AM +0800, Shiyang Ruan wrote:
> > > Make sure memory failure mechanism works when filesystem is mounted with
> > > dax option.
> > >
> > > Signed-off-by: Shiyang Ruan <ruansy.fnst@xxxxxxxxxxx>
> > > ---
> > > .gitignore | 1 +
> > > src/Makefile | 3 +-
> > > src/t_mmap_cow_memory_failure.c | 157 ++++++++++++++++++++++++++++++++
> > > tests/xfs/900 | 48 ++++++++++
> > > tests/xfs/900.out | 9 ++
> > > 5 files changed, 217 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
> > >
> ...
> > > diff --git a/tests/xfs/900.out b/tests/xfs/900.out
> > > new file mode 100644
> > > index 00000000..d861bf1f
> > > --- /dev/null
> > > +++ b/tests/xfs/900.out
> > > @@ -0,0 +1,9 @@
> > > +QA output created by 900
> > > +Format and mount
> > > +Create the original files
> > > +Inject memory failure (1 page)
> > > +Inject posion...
> >
> > As your program is: printf("Inject poison...\n");
> >
> > So I think this place should be "poison".
>
> My bad. I forgot to fix this typo in .out file.
Never mind. I'd like to merge these tests after your kernel patchset about
"Add reflink&dedupe support for fsdax" be merged at first. To avoid you
might add/change something. And I need to see the test works on offical
kernel at least, before merging it. So feel free to ping me, if I forget
this patchset at that time :)
Thanks,
Zorro
>
> >
> > > +Process is killed by signal: 7
> > > +Inject memory failure (2 pages)
> > > +Inject posion...
> >
> > same as above. And other 2 patches might have same issue.
>
> Thanks! Will fix them.
>
>
> --
> Ruan.
>
> >
> > Thanks,
> > Zorro
> >
> > > +Process is killed by signal: 7
> > > --
> > > 2.35.1
> > >
> > >
> > >
> >
>
>
