[CRIU] [PATCH 5/5] zdtm: now tempfs_overmounted will pass so remove crfail
Andrew Vagin
avagin at virtuozzo.com
Sat Dec 2 08:20:05 MSK 2017
On Fri, Nov 24, 2017 at 05:22:21PM +0300, Pavel Tikhomirov wrote:
> Signed-off-by: Pavel Tikhomirov <ptikhomirov at virtuozzo.com>
> ---
> test/zdtm/static/tempfs_overmounted.desc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/test/zdtm/static/tempfs_overmounted.desc b/test/zdtm/static/tempfs_overmounted.desc
> index 0d8b7f2b4..7657ba45c 100644
> --- a/test/zdtm/static/tempfs_overmounted.desc
> +++ b/test/zdtm/static/tempfs_overmounted.desc
> @@ -1 +1 @@
> -{'flavor': 'ns uns', 'flags': 'suid crfail'}
> +{'flavor': 'ns uns', 'flags': 'suid'}
You need to improve this test, it checks nothing now.
For example, you can apply the attached patch and try to run it
=== Run 1/1 ================ zdtm/static/tempfs_overmounted
=================== Run zdtm/static/tempfs_overmounted in ns ===================
Start test
Test is SUID
./tempfs_overmounted --pidfile=tempfs_overmounted.pid --outfile=tempfs_overmounted.out --dirname=tempfs_overmounted.test
Run criu dump
Unable to kill 43: [Errno 3] No such process
Run criu restore
1: Old mounts lost: [('/', '/zdtm/static/tempfs_overmounted.test/a')]
1: New mounts appeared: []
4: Old mounts lost: [('/', '/zdtm/static/tempfs_overmounted.test/a')]
4: New mounts appeared: []
Send the 15 signal to 78
Wait for zdtm/static/tempfs_overmounted(78) to die for 0.100000
Unable to kill 78: [Errno 3] No such process
########### Test zdtm/static/tempfs_overmounted FAIL at result check ###########
Test output: ================================
05:19:50.137: 4: ERR: tempfs_overmounted.c:62: umount
<<< ================================
##################################### FAIL #####################################
> --
> 2.13.6
>
More information about the CRIU
mailing list