- 22 Dec, 2016 4 commits
-
-
Mike Rapoport authored
The return value of seep_pagemap_page can be either 1 (page found) or 0 (no page found). There's no point in checking its return value for -1. Signed-off-by:
Mike Rapoport <rppt@linux.vnet.ibm.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Mike Rapoport authored
travis-ci: success for uffd: A new set of improvements Signed-off-by:
Mike Rapoport <rppt@linux.vnet.ibm.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Mike Rapoport authored
Since commit 95009c4c6d5 (criu: pagemap: introduce advance() helper for pagemap iteration) seek_pagemap_page traverses the entire pagemap, rather than starts each time from the last pagemap that was used. With large pagemaps (like, e.g. in maps05 case) the new behavior causes exponential increase in amount of calls to skip_pagemap_page for every new snapshot. Let's restore the original behavior. Fixes: #217 Reported-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Mike Rapoport <rppt@linux.vnet.ibm.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Mike Rapoport authored
Signed-off-by:
Mike Rapoport <rppt@linux.vnet.ibm.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
- 21 Dec, 2016 35 commits
-
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Andrei Vagin authored
travis-ci: success for series starting with [01/21] build: install libnet-dev Signed-off-by:
Andrei Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Vitaly Ostrosablin authored
By default, sockets03 doesn't set SIGPIPE handler to SIG_IGN. When run by zdtm.py, it doesn't cause problem, because it inherits signal handler. However, when run via vzt-cpt (which uses system() to launch tests), it receives default handler (which would terminate test). As result, test is guaranteed to fail, because it will attempt to write to shutdown receiver and get SIGPIPE in the face. travis-ci: success for test: Fix sockets03 default SIGPIPE handler not being set to IGN (rev2) Acked-by:
Andrei Vagin <avagin@virtuozzo.com> Signed-off-by:
Vitaly Ostrosablin <vostrosablin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Kirill Tkhai authored
A hook to cleanup test temporary files travis-ci: success for zdtm: Add binfmt_misc.hook Signed-off-by:
Kirill Tkhai <ktkhai@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Kirill Tkhai authored
For easier debugging. v2: Forward scripts messages to criu log using cr_system travis-ci: success for scripts: Print error value in case of action script fail (rev2) Signed-off-by:
Kirill Tkhai <ktkhai@virtuozzo.com> Acked-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Kirill Tkhai authored
Check that SO_PEERCRED socket options restored correctly. Now it's not supported, thus test is marked as "noauto". (It seems support of this feature will require "ghost tasks", when we introduce them). travis-ci: success for zdtm: Add sock_peercred test Signed-off-by:
Kirill Tkhai <ktkhai@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Andrei Vagin authored
Now we remove a link_remap file when all its files are restored. It is wrong, because something else may fail and we will not be able to repeat an attempt to restore processes again. travis-ci: success for files: remove link_remaps when everything has been restored Signed-off-by:
Andrei Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Andrei Vagin authored
link-remap files has to be deleted only if processes have been restored successfully. Otherwise we can want to repeat an attempt, but it is imposible if some link-remap files were deleted. Signed-off-by:
Andrei Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Andrei Vagin authored
Signed-off-by:
Andrei Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Andrei Vagin authored
Starting with 4.8 the kernel doesn't allow to create inodes with a uid or gid unknown to an user namespace where a file system was mounted. 036d523641c66 ("vfs: Don't create inodes with a uid or gid unknown to the vfs") Signed-off-by:
Andrei Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Kir Kolyshkin authored
DESTDIR is mostly used by packagers (such as from rpm spec or deb rules), or just to try installation, and normally should be left unset/empty. Setting it to / looks weird to me. Besides, by leaving it empty we are making sure everything works if DESTDIR is not set. Fix INSTALL.md accordingly. travis-ci: success for Makefile.install: DESTDIR should be empty CC: Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Kir Kolyshkin <kir@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Andrei Vagin authored
Now it is impossible to find when a test has been started, but sometimes we want to knwo which tests were running in a specified moment. travis-ci: success for zdtm.py: add a test name into a progress line Signed-off-by:
Andrei Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Andrei Vagin authored
Each test has a process which controls other processes and if it dies the other processes have to be killed. travis-ci: success for zdtm: exit a test if the "root" process died Signed-off-by:
Andrei Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Andrei Vagin authored
CID 173074 (#1 of 1): Resource leak (RESOURCE_LEAK) 9. leaked_handle: Handle variable sock going out of scope leaks the handle. Signed-off-by:
Andrei Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Dmitry Safonov authored
Just a little optimization (one syscall less). travis-ci: success for Rectify 32-bit compatible C/R on x86 Signed-off-by:
Dmitry Safonov <dsafonov@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Pavel Emelyanov authored
There's no such field in resp, only cr_errno travis-ci: success for py: Fix resp.errno access Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com> Acked-by:
Andrei Vagin <avagin@virtuozzo.com>
-
Kirill Tkhai authored
No functional changes; this just improves uniformity. Signed-off-by:
Kirill Tkhai <ktkhai@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Kirill Tkhai authored
This will be used instead of currently existing per-fd transport sockets, which are used on receive_fd stage. v2: Create the socket before CR_STATE_FORKING state. Signed-off-by:
Kirill Tkhai <ktkhai@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Cyrill Gorcunov authored
INCLUDEDIR gonna be used in compel. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Dmitrii Shcherbakov authored
Should be easier to debug if the variable is not passed by accident. travis-ci: success for series starting with [1/2] zdtm:mnt_ext_auto: avoid segfault if ZDTM_NEWNS is not set Signed-off-by:
Dmitrii Shcherbakov <dshcherbakov@virtuozzo.com> Acked-by:
Andrei Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Dmitrii Shcherbakov authored
If getenv returns a null pointer we get a segfault here. Let's exit in a more sane way. travis-ci: success for series starting with [1/2] zdtm:mnt_ext_auto: avoid segfault if ZDTM_NEWNS is not set Signed-off-by:
Dmitrii Shcherbakov <dshcherbakov@virtuozzo.com> Acked-by:
Andrei Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Mike Rapoport authored
Move error printouts to the call sites to keep "fast path" clearer. Signed-off-by:
Mike Rapoport <rppt@linux.vnet.ibm.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Mike Rapoport authored
It seems that logging amount of skipped pages was useful for early debugging of pre-dumps, but now it just creates a lot of noise for lazy pages and pre-dump tests. Signed-off-by:
Mike Rapoport <rppt@linux.vnet.ibm.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Kir Kolyshkin authored
Don't append \n as pr_perror() already does it for us. travis-ci: success for more pr_perror() usage fixes Signed-off-by:
Kir Kolyshkin <kir@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Kir Kolyshkin authored
In places where we are not supposed to print errno, or where we already print in explicitly, use pr_err(). travis-ci: success for more pr_perror() usage fixes Signed-off-by:
Kir Kolyshkin <kir@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Kir Kolyshkin authored
Use alias function attribute to provide memcpy(), memcmp() and memset() in case we're not linking with libc. This is needed as compiler can generate calls to those functions (clang actually does that). [v2: add comment, move aliases to under HAS_BUILTIN_*] https://travis-ci.org/kolyshkin/criu/builds/174634847Idea-by:
Andrei Vagin <avagin@openvz.org> Signed-off-by:
Kir Kolyshkin <kir@openvz.org> Reviewed-by:
Dmitry Safonov <dsafonov@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Kir Kolyshkin authored
1. memcpy() last argument is size_t 2. memset() returns void * Make our builtin_*() functions compatible. https://travis-ci.org/kolyshkin/criu/builds/174634847Signed-off-by:
Kir Kolyshkin <kir@openvz.org> Reviewed-by:
Dmitry Safonov <dsafonov@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Pavel Emelyanov authored
Done with python pprint module (indent = 4, width = 80) for files with more than one field in and with length exceeding 80 characters. Travised-by: https://travis-ci.org/xemul/criu/jobs/171350152 travis-ci: success for zdtm: Split long .desc lines Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Cyrill Gorcunov authored
For portability sake, we gonna share this header. travis-ci: success for Common headers Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Dmitry Safonov authored
Don't use compiler-provided memset in parasite. Fixes: commit 86434deba0b4 ("x86: dump TLS entries from GDT"). Cc: Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Dmitry Safonov <dsafonov@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Cyrill Gorcunov authored
Tycho reported that Debian uses custom LDFLAGS to build criu, this flag get propagated into built-in targets where we use $(LD) instead causing build procedure to interrupt | LINK arch/x86/syscalls.built-in.o | ld: unrecognized option '-Wl,-Bsymbolic-functions' | ld: use the --help option for usage information | /«PKGBUILDDIR»/scripts/nmk/scripts/build.mk:141: recipe for target 'arch/x86/syscalls.built-in.o' failed Lets make maintainers life easier and filter out the flags we know never accepted by $(LD). Reported-by:
Tycho Andersen <tycho.andersen@canonical.com> Tested-by:
Tycho Andersen <tycho.andersen@canonical.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Reviewed-by:
Dmitry Safonov <dsafonov@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
Dmitry Safonov authored
Sigframe pointer should be fixed size. Convert members of rt_sigframe_ia32 to fixed-size. After this patch dumping of compatible tasks works. Cc: Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Dmitry Safonov <dsafonov@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-
- 19 Dec, 2016 1 commit
-
-
Andrei Vagin authored
org.tap4j.parser.ParserException: Error parsing TAP Stream: Invalid position of TAP Header. It must be the first element (apart of Comments) in the TAP Stream. Reported-by: Mr Jenkins travis-ci: success for zdtm: move a tap header to a first line Signed-off-by:
Andrei Vagin <avagin@virtuozzo.com> Signed-off-by:
Pavel Emelyanov <xemul@virtuozzo.com>
-