- 09 Dec, 2016 17 commits
-
-
Andrei Vagin authored
page_in_parent() returns true for all not dirty pages, probably it should not be executed for zero pages. ------------------------ grep Error ------------------------ (00.115327) Error (criu/pagemap.c:191): Missing 8000 in parent pagemap, current iov: base=9000,len=32768 (00.115329) Error (criu/page-xfer.c:237): Hole 0x8000/4096 not found in parent (00.160222) Error (criu/cr-dump.c:1636): Dumping FAILED. ------------------------ ERROR OVER ------------------------ Signed-off-by:
Andrei Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Andrei Vagin authored
https://github.com/xemul/criu/issues/253 travis-ci: success for test: don't execute cgroup03 with other cgroup tests concurrently Signed-off-by:
-
Dmitry Safonov authored
Before: =[log]=> dump/zdtm/transition/shmem/142/1/dump.log ------------------------ grep Error ------------------------ pie: 1: Error (/home/travis/build/0x7f454c46/criu/criu/arch/x86/includ> pie: 1: Error (/home/travis/build/0x7f454c46/criu/criu/arch/x86/includ> pie: 5: Error (/home/travis/build/0x7f454c46/criu/criu/arch/x86/includ> pie: 5: Error (/home/travis/build/0x7f454c46/criu/criu/arch/x86/includ> pie: 6: Error (/home/travis/build/0x7f454c46/criu/criu/arch/x86/includ> pie: 6: Error (/home/travis/build/0x7f454c46/criu/criu/arch/x86/includ> ------------------------ ERROR OVER ------------------------ After: =[log]=> dump/zdtm/transition/shmem/62/1/dump.log ------------------------ grep Error ------------------------ pie: 1: Error (/home/travis/build/0x7f454c46/criu/criu/arch/x86/includ> pie: 1: e/asm/parasite.h:45): Failed to dump TLS descriptor #13: -4 pie: 1: Error (/home/travis/build/0x7f454c46/criu/criu/arch/x86/includ> pie: 1: e/asm/parasite.h:45): Failed to dump TLS descriptor #14: -4 pie: 5: Error (/home/travis/build/0x7f454c46/criu/criu/arch/x86/includ> pie: 5: e/asm/parasite.h:45): Failed to dump TLS descriptor #13: -4 pie: 5: Error (/home/travis/build/0x7f454c46/criu/criu/arch/x86/includ> pie: 5: e/asm/parasite.h:45): Failed to dump TLS descriptor #14: -4 pie: 6: Error (/home/travis/build/0x7f454c46/criu/criu/arch/x86/includ> pie: 6: e/asm/parasite.h:45): Failed to dump TLS descriptor #13: -4 pie: 6: Error (/home/travis/build/0x7f454c46/criu/criu/arch/x86/includ> pie: 6: e/asm/parasite.h:45): Failed to dump TLS descriptor #14: -4 ------------------------ ERROR OVER ------------------------ Signed-off-by:
Dmitry Safonov <dsafonov@virtuozzo.com> Signed-off-by:
-
Andrei Vagin authored
Signed-off-by:
-
Andrei Vagin authored
If a mount overmounts something else, we can try to restore it separetly and then move it to the right places after restoring all mounts. In this patch if we see that a mount is overmounts something, we create a new directory in the root yard and restore this mount and its sub-tree in this directory. https://bugs.openvz.org/browse/OVZ-6778 v2: add more comments rename roots_mp into roots_yard_mp v3: rename functions Signed-off-by:
-
Andrei Vagin authored
It is preparation to handle over-mounts. v2: don't set the same name for global and local variables Signed-off-by:
-
Andrei Vagin authored
It will be used to handle overmounts Signed-off-by:
-
Andrei Vagin authored
Signed-off-by:
-
Yixun Lan authored
Since glibc is moving away from implicitly including sys/sysmacros.h all the time via sys/types.h, include the header directly in more places. This seems to cover most makedev/major/minor usage. Signed-off-by:
Yixun Lan <dlan@gentoo.org> Signed-off-by:
Mike Frysinger <vapier@gentoo.org> Reviewed-by:
-
Cyrill Gorcunov authored
travis-ci: success for restore: Report error number if PR_SET_MM failed Reported-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
-
Adrian Reber authored
To run the test zdtm/static/sched_policy00 /sys/fs/cgroup/cpu/user.slice/cpu.cfs_period_us needs to be non zero. This changes it to 100 during the test and resets it to the original value once the test finishes. travis-ci: success for zdtm: set cpu.cfs_period_us to 100 during sched_policy00 test Signed-off-by:
Adrian Reber <areber@redhat.com> Signed-off-by:
-
Andrei Vagin authored
Currently when we have a shared group, we set a source mount for all of them and then we need to update a source mount when we mount a first mount from a shared group, because a shared id can be only inherited. Signed-off-by:
-
Andrei Vagin authored
This function is going to be used more than once. v2: don't change logic in this patch Signed-off-by:
-
Cyrill Gorcunov authored
One level always been handled properly but two levels may cause problems. travis-ci: success for Multiple nested deleted dentries fix Signed-off-by:
-
Cyrill Gorcunov authored
In case if several levels of dentries are missed we can't restore them otherwise. travis-ci: success for Multiple nested deleted dentries fix Signed-off-by:
-
Cyrill Gorcunov authored
- take @mode into account - return system error codes on error travis-ci: success for Multiple nested deleted dentries fix Signed-off-by:
-
Andrei Vagin authored
https://github.com/xemul/criu/issues/260 https://travis-ci.org/avagin/criu/builds/182462796 https://ci.openvz.org/job/CRIU/job/criu-patchwork/941//console v2: don't call page_pipe_add_hole for zero pages Signed-off-by:
-
- 05 Dec, 2016 17 commits
-
-
Raghavendra Prabhu authored
In some distros (such as Arch), libxtables_standard is part of iptables as `/usr/lib/iptables/libxt_standard.so`. With this change, the test passes, otherwise netns-nf test fails. Tested with make test. Signed-off-by:
Raghavendra D Prabhu <me@rdprabhu.com> Signed-off-by:
-
Kirill Tkhai authored
Negative retval is error, while 0 is OK. v2: More places travis-ci: success for criu: Fix open() retval analysing Signed-off-by:
Kirill Tkhai <ktkhai@virtuozzo.com> Acked-by:
-
Andrei Vagin authored
CRIU uses iptables rules to block tcp connections and if rules are added at the tail of the chain, other rules can accept packets which have to be blocked. travis-ci: success for series starting with [01/21] build: install libnet-dev Signed-off-by:
-
Mike Rapoport authored
Currently it's possible to continue testing despite failures only when running zdtm.py with --all option. Let's make --keep-going option also valid when more than one test is requested with --tests or --from. Signed-off-by:
Mike Rapoport <rppt@linux.vnet.ibm.com> Signed-off-by:
-
Pavel Emelyanov authored
This makes crit show the type, which is handy. Also this will let us deprecate the old format where remap type was encoded into remap ID. travis-ci: success for files: Explicitly set linked remap type Signed-off-by: -
Kir Kolyshkin authored
This is an addition to commit 6273a5c ("Add ability to restore with missing sysctls on the target node"). travis-ci: success for Document --weak-syscalls Cc: Pavel Tikhomirov <ptikhomirov@virtuozzo.com> Signed-off-by:Kir Kolyshkin <kir@openvz.org> Acked-by:
Pavel Tikhomirov <ptikhomirov@virtuozzo.com> Signed-off-by:
-
Kir Kolyshkin authored
Option --help should provide a brief information, like a list of commands and options and their meaning, ideally no more than 1 line per option. There is no need to describe nuances and details here, since we have documentation (wiki and man pages) for that. travis-ci: success for Document --weak-syscalls Cc: Pavel Tikhomirov <ptikhomirov@virtuozzo.com> Signed-off-by:
-
Pavel Tikhomirov authored
Signed-off-by:
-
Pavel Tikhomirov authored
Do it through new --weak-sysctls option to "criu restore". When we migrate from source with newer kernel with some modern sysctl to destination with older kernel without it or just with a kernel configured without it (e.g. no CONFIG_IPV6_ROUTER_PREF), we face a problem that not all of dumped sysctls can be successfuly restored on the destination. So to address these problem introduce weak sysctls migration, where we restore only available sysctls and skip others. https://github.com/xemul/criu/issues/248 While on it, fix workaround for stable_secret eio skip. travis-ci: success for Add ability to restore with missing sysctls on the target node Signed-off-by:
-
Kir Kolyshkin authored
With the recent patches, criu is finally compileable with clang for all architectures. To keep it that way, we need Travis to test that on a regular basis. The downside is, Travis build time is increased about 2x. travis-ci: success for travis: enable arm/arm64/ppc builds with clang Signed-off-by:
-
Adrian Reber authored
When using the option --keep-going to get a quick overview of the test results there is now a summary which tests have failed like: ############# 7 TEST(S) FAILED (TOTAL 268/SKIPPED 83) ############## * zdtm/static/sched_policy00(unknown) * zdtm/static/pthread00(unknown) * zdtm/static/pthread01(unknown) * zdtm/static/maps01(unknown) * zdtm/static/maps00(unknown) * zdtm/transition/maps008(unknown) * zdtm/transition/shmem(unknown) ############################### FAIL ############################### Signed-off-by:
-
Vitaly Ostrosablin authored
Those two tests use same socket name and if both of them are run simultaneously, they would fight for socket and one will get EADDRINUSE. This patch gives socket_dgram_data an unique name, so that everyone will be happy. travis-ci: success for test: fix same socket names in socket-{dgram_data, snd_addr} Acked-by: -
Raghavendra Prabhu authored
The installation needs to use python2, but it uses python in the Makefile which fails later with syntax error (since it is written for python2). As per PEP-394 - https://www.python.org/dev/peps/pep-0394/ -, 'all distributions should ensure that python refers to the same target as python2 .', hence this change. Therefore, python2 is used by default in lib/Makefile, though this can be overriden with PYTHON_BIN. Tested with: a) PYTHON_BIN=python2 make install b) make install c) export PYTHON_BIN=python2 make test Signed-off-by: Raghavendra Prabhu <me at rdprabhu dot com> Signed-off-by:
-
Kirill Tkhai authored
Do not dump external sockets, that are not peers for some processes sockets. We do not use them on restore ==> we do not need to dump them. Signed-off-by:
-
Venkateswara Rao Mandela authored
Build in tests/others/rpc folder was failing due to the use of LDFLAGS variable instead of LDLIBS variable to link libprotobuf-c. The makefile uses implicit build rules. $(CC) $(LDFLAGS) n.o $(LOADLIBES) $(LDLIBS) Using LDFLAGS places -llibprotobuf-c before the object files, preventing the library from being searched for required functions. This commit changes the variable to LDLIBS instead to fix the issue. travis-ci: success for test/others/rpc: fix build error Signed-off-by:
Venkateswara Rao Mandela <venkat.mandela@ti.com> Reviewed-by:
-
Pavel Emelyanov authored
https://travis-ci.org/criupatchwork/criu/builds/177945936Signed-off-by:
-
Pavel Emelyanov authored
We can MSG_PEEK those and can dump the files easily. The problem is in restore -- we need to first pass through all the fdinfo openings before sending the descriptors out, but by that time some sending sockets may have already closed :( So for now just detect SCM-s in queue and abort the dump, otherwise we'd just have them lost and the state spoiled. https://travis-ci.org/criupatchwork/criu/builds/177945936Signed-off-by:
-
- 22 Nov, 2016 6 commits
-
-
Dmitry Safonov authored
1. Dropped check for (fd < 0) - there is check after open() in the test, just two lines upper or so. 2. Combined two test to share common code: they differ only by proc path to open in a dead process. travis-ci: success for pstree: rename lookup_create_{item <=> pid} (rev2) Cc: Tycho Andersen <tycho.andersen@canonical.com> Acked-by:Tycho Andersen <tycho.andersen@canonical.com> Signed-off-by:
-
Pavel Emelyanov authored
Currently each page read request results in pread() call. But we can do better -- if there's no urgent need in data (which is likely the case for non-cow pagemaps) we can save where the date should go to and then read it in in one preadv syscall. For lazy pages we get an ability to do async read_pages call that would send the request to server and return w/o data in the buffer. The explicit sync() method is added since .close is void and nobody checks its return code. v2: Lost limits.h in pagemap.c for IOV_MAX (compilation). v3: split added code into smaller chunks https://travis-ci.org/criupatchwork/criu/builds/175451746Signed-off-by:
-
Pavel Emelyanov authored
Flag PR_ASYNC means, that the caller is OK if the routine returns w/o data read into the buffer provided. https://travis-ci.org/criupatchwork/criu/builds/175451746Signed-off-by:
-
Pavel Emelyanov authored
It's already written into page_read->pi_off, so just use one. https://travis-ci.org/criupatchwork/criu/builds/175451746Signed-off-by:
-
Mike Rapoport authored
travis-ci: success for page-xfer: spelling: s/heppen/happen Signed-off-by:
-
Mike Rapoport authored
With in-memory pagemap put_pagemap is not required because it does not frees any resources. Minor modifications to get_pagemap and init_pagemaps allow proper bookkeeping without put_pagemap. Signed-off-by:
-
