Hi,
> On Fri, Jun 10, 2022 at 10:35:53AM +0800, Wang Yugui wrote:
> > stat -c '%Y' report seconds as int, so the delta 2.01s may result as 3s.
> >
> > Signed-off-by: Wang Yugui <wangyugui@xxxxxxxxxxxx>
> > ---
> > tests/generic/297 | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/tests/generic/297 b/tests/generic/297
> > index 6bdc3e1c..e3082202 100755
> > --- a/tests/generic/297
> > +++ b/tests/generic/297
> > @@ -51,7 +51,7 @@ for i in $(seq 0 $fnr); do
> > touch $TEST_DIR/after
> > before=$(stat -c '%Y' $TEST_DIR/before)
> > after=$(stat -c '%Y' $TEST_DIR/after)
> > - delta=$((after - before))
> > + delta=$((after - before -1)) # 2.01s may result as 3s; so -1
>
> What issue is this change trying to fix? "timeout=8"s is not long enough?
for the command
$TIMEOUT_PROG -s INT ${kill_after}s
delta=$((after - before )) may report 'kill_after+1' in some case.
so no relationship to "timeout=8" or "timeout=2".
'$XFS_IO_PROG -f -c reflink' without '$TIMEOUT_PROG -s INT ${kill_after}s'
may take 20s because this is a very complex reflink.
Best Regards
Wang Yugui (wangyugui@xxxxxxxxxxxx)
2022/06/10
