[PATCH v2 5/6] generic/531: Check for O_TMPFILE
Zorro Lang
zlang at redhat.com
Wed Apr 26 08:32:36 PDT 2023
On Wed, Apr 26, 2023 at 07:54:50AM -0700, Darrick J. Wong wrote:
> On Wed, Apr 26, 2023 at 10:51:49PM +0800, Zorro Lang wrote:
> > On Wed, Apr 26, 2023 at 10:49:31PM +0800, Zorro Lang wrote:
> > > On Mon, Apr 24, 2023 at 03:10:41PM +0100, David Howells wrote:
> > > > Make generic/531 check that the filesystem under test supports O_TMPFILE
> > > > before attempting to test it.
> > > >
> > > > Signed-off-by: David Howells <dhowells at redhat.com>
> > > > cc: linux-afs at lists.infradead.org
> > > > ---
> > > > tests/generic/531 | 1 +
> > > > 1 file changed, 1 insertion(+)
> > > >
> > > > diff --git a/tests/generic/531 b/tests/generic/531
> > > > index 5e84ca97..e5f3dddd 100755
> > > > --- a/tests/generic/531
> > > > +++ b/tests/generic/531
> > > > @@ -20,6 +20,7 @@ testfile=$TEST_DIR/$seq.txt
> > > > # real QA test starts here
> > > > _supported_fs generic
> > > > _require_scratch
> > > > +_require_xfs_io_command "-T"
> > >
> > > Make sense,
> > >
> > > Reviewed-by: Zorro Lang <zlang at redhat.com>
> >
> > Oh, generic/530 is similar with 531, why only 531 need this one?
>
> AFS doesn't support shutdown, presumably.
Oh, 530 require shutdown :)
>
> (But yes, both tests should be _require-ing this.)
Yes, I'll add that to generic/530 too, when I merge it.
>
> --D
>
> > >
> > > > _require_test_program "t_open_tmpfiles"
> > > >
> > > > _scratch_mkfs >> $seqres.full 2>&1
> > > >
> >
>
More information about the linux-afs
mailing list