Re: [PATCH v3 2/2] xfs/068: new fsstress operation breaks xfsrestore output

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]



On Sat, Jan 19, 2019 at 10:42:58AM +0800, Zorro Lang wrote:
> After fsstress has new operation, xfs/068 has different dump output,
> so change the expected number of files and directories.
> 
> Signed-off-by: Zorro Lang <zlang@xxxxxxxxxx>
> ---
>  tests/xfs/068.out | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/xfs/068.out b/tests/xfs/068.out
> index fa3a5523..f9ec37b8 100644
> --- a/tests/xfs/068.out
> +++ b/tests/xfs/068.out
> @@ -22,7 +22,7 @@ xfsrestore: session id: ID
>  xfsrestore: media ID: ID
>  xfsrestore: searching media for directory dump
>  xfsrestore: reading directories
> -xfsrestore: 383 directories and 1335 entries processed
> +xfsrestore: 416 directories and 1373 entries processed

Shouldn't you tell fsstress to avoid this new operation so none of
the tests that rely on the number of files created by a specific
fstress seed break?

i.e. changing FSSTRESS_AVOID in common/dump as is done already to
turn off dedupe/clone/copy_file_range for xfsdump/restore tests?

Cheers,

dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx



[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux