- 23 Nov, 2015 8 commits
-
-
Andrew Vagin authored
Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
v2: Add --report for report generation Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Acked-by:
Andrew Vagin <avagin@virtuozzo.com>
-
Tycho Andersen authored
Closes #87 Signed-off-by:
Tycho Andersen <tycho.andersen@canonical.com> CC: Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
When we restore ipv6 addresses, they go through a “tentative” phase and sockets could not be bound to them in this moment. v2: add more comments in code Reported-by:
Ross Boucher <boucher@gmail.com> Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
v2: set has_freebind Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Tycho Andersen authored
This include is actually unused (probably leftover from an old version of the kernel API when I was porting the patchsets); we can delete it. Signed-off-by:
Tycho Andersen <tycho.andersen@canonical.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Tycho Andersen authored
Also define some constants for people who don't have them in their headers. Signed-off-by:
Tycho Andersen <tycho.andersen@canonical.com> Acked-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 20 Nov, 2015 8 commits
-
-
Pavel Emelyanov authored
This teat crashes the ptrace-based freezer, need freeze-cgroup Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Tycho Andersen authored
Signed-off-by:
Tycho Andersen <tycho.andersen@canonical.com> CC: Andrew Vagin <avagin@virtuozzo.com> Acked-by:
Andrew Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Tycho Andersen authored
Signed-off-by:
Tycho Andersen <tycho.andersen@canonical.com> Acked-by:
Andrew Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
Cc: Stanislav Kinsbursky <skinsbursky@openvz.org> Reported-by:
Stanislav Kinsbursky <skinsbursky@openvz.org> Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
In current implementation of thread bomb effectively only one thread runs, the rest sit in the read from pipe. Fix this by making threads constantly creating new ones and just exiting. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Acked-by:
Andrew Vagin <avagin@virtuozzo.com>
-
Andrew Vagin authored
SIGSTOP can't be blocked, so we need to wait when it will be handled. v2: don't duplicate the same code twice Reported-by: Mr Jenkins Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
It's required to check the SIGSTOP signal, which can't be blocked. Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 19 Nov, 2015 9 commits
-
-
Andrew Vagin authored
Let's imagine that we have two local interconnected sockets. Whe we are restoring tcp sockets, we need to disable the repair mode to restore data in sending queues. If traffic isn't blocked, a socket starts operating, but in this time another end can be not restored yet. $ test/zdtm.sh -r ns/static/socket-tcpbuf-local ... (00.274632) 5: Error (sk-tcp.c:485): Can't restore 2 queue data (-1), want (1780919:1780919): Connection reset by peer We create a separate chain to avoid conflicts with other rules. https://bugs.openvz.org/browse/CRIU-96 v2: use iptables-restore to apply whole configuration for one call v3: add a comment why we use iptables-restore instread of iptables. v4: remove unused headers Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
So we keep it and dont close inside close_old_fds() helper but pass into veth creation so the kernel can fetch the net namespace of the veth peer. v2 (by avagin@): - don't forget to close opened descriptor Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
This patch suppresses these fake errors: ./socket-tcp6-local --pidfile=socket-tcp6-local.pid --outfile=socket-tcp6-local.out Dump 5921 (00.041988) Error (parasite-syscall.c:388): si_code=1 si_pid=5936 si_status=0 (00.051210) Error (parasite-syscall.c:388): si_code=1 si_pid=5937 si_status=0 (00.070800) Error (parasite-syscall.c:388): si_code=1 si_pid=5938 si_status=0 (00.077582) Error (parasite-syscall.c:388): si_code=1 si_pid=5939 si_status=0 (00.091313) Error (action-scripts.c:59): One of more action scripts failed Check results 5921 18:39:14.451: 5921: PASS Test: zdtm/live/static/socket-tcp6-local, Result: PASS ZDTM tests PASS. Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Acked-by:
Tycho Andersen <tycho.andersen@canonical.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
Reported-by:
Stanislav Kinsbursky <skinsbursky@openvz.org> Cc: Stanislav Kinsbursky <skinsbursky@openvz.org> Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
v2: don't dump iptables if ipv6 isn't supported Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
v2: use a cached value to dump ipv6 interface addesses call get_ipv6() from kerndat_init_rst too Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
It returns EINTR, so we need to handle it. $ bash test/zdtm.sh --restore-sibling ns/static/env00 ... futex(0x7fc20ec92010, FUTEX_WAIT, 1, {120, 0}) = ? ERESTART_RESTARTBLOCK (Interrupted by signal) Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 17 Nov, 2015 15 commits
-
-
Pavel Emelyanov authored
There's a sat.py script in criu-scripts repo that analyzes the strace outputs. It's useful to check the amount of system calls criu takes to do dump or restore of certain apps. So add the --sat key to zdtm.py that would generate sat-able logs for individual tests. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Right now report looks like generic python exception about inability to copy some file. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
It's safer than alloca() calls. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Better to obtain error if there is no free memory than smashing the stack. A rule of thumb for alloca() based functions is to use them with predefined small sizes (such as we do in swapping builtin sizes for parasite engine). Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Tycho Andersen authored
v2: * add another filter to the chain to test the inheritance chaining check * include zdtm.py desc file Signed-off-by:
Tycho Andersen <tycho.andersen@canonical.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Tycho Andersen authored
v2: include zdtm.py desc file Signed-off-by:
Tycho Andersen <tycho.andersen@canonical.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Tycho Andersen authored
v2: include zdtm.py desc file Signed-off-by:
Tycho Andersen <tycho.andersen@canonical.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Tycho Andersen authored
v2: use a non-racy version of fork_and_ptrace_attach Signed-off-by:
Tycho Andersen <tycho.andersen@canonical.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Tycho Andersen authored
This commit adds basic support for dumping and restoring seccomp filters via the new ptrace interface. There are two current known limitations with this approach: 1. This approach doesn't support restoring tasks who first do a seccomp() and then a setuid(); the test elaborates on this and I don't think it is tough to do, but it is not done yet. 2. Filters are compared via memcmp(), so two tasks which have the same parent task and install identical (via memory) filters will have those filters considered to be the "same". Since we force all tasks to have the same creds (including seccomp filters) right now, this isn't a problem. The approach used here is very similar to the cgroup approach: the actual filters are stored in a seccomp.img, and each task has an id that points to the part of the filter tree it needs to restore. This keeps us from dumping the same filter multiple times, since filters are inherited on fork. v2: * remove unused seccomp_filters field from struct rst_info * rework memory layout for passing filters to restorer blob * add a sanity check when finding inherited filters Signed-off-by:
Tycho Andersen <tycho.andersen@canonical.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
Otherwise the kernel can set it as a control terminal for the currect session and CRIU will not be able to set it as a control terminal for the target process. Reported-by:
Andrey Ryabinin <aryabinin@virtuozzo.com> Acked-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
v2: add comments and rename ns_created to ns_populated. Reported-by:
Andrey Ryabinin <aryabinin@virtuozzo.com> Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
It can be the same with a parent one and the helper will inherit all parent resources. Now we can dereference item->ids for all task. It's used in restore_task_mnt_ns, for example. Reported-by: Mr Jenkins Signed-off-by:
Andrew Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-