- 16 Aug, 2017 1 commit
-
-
Andrei Vagin authored
In a error case, task_entries->nr_in_progress is set to -1 and we have to handle this case. Signed-off-by:Andrei Vagin <avagin@virtuozzo.com>
-
- 15 Aug, 2017 18 commits
-
-
Pavel Emelyanov authored
Item's thread struct pid is not a pointer in master. Signed-off-by:Pavel Emelyanov <xemul@virtuozzo.com>
-
Andrei Vagin authored
It is good to know what command were executed. https://github.com/xemul/criu/issues/371Signed-off-by:
-
Pavel Emelyanov authored
Signed-off-by:
-
Pavel Emelyanov authored
It turned out (surprise surprise) that page server exchanges CR_FD_PAGEMAP and CR_FD_SHMEM_PAGEMAP values in ps_iov_msg. The problem is that these constants sit in enum and change their values from version to version %) So here's the fixed version of the protocol including the backward compat checks on all the values that could be met from older CRIUs (we're lucky and they didn't intersect). Signed-off-by:
-
Mike Rapoport authored
Signed-off-by:
Mike Rapoport <rppt@linux.vnet.ibm.com> Signed-off-by:
-
Mike Rapoport authored
Signed-off-by:
-
Vitaly Ostrosablin authored
There's two issues with this code: 1. Child task is killed by parent faster, than it could print the line. 2. Even if it had time to print it - there would always be 0, because it's called from child process. Obviously, this print was meant to be in parent process. So, let's move it there. Signed-off-by:
Vitaly Ostrosablin <vostrosablin@virtuozzo.com> Signed-off-by:
-
Francesco Giancane authored
As of manual page INSTALL.md, it is stated that those variables can be overridden by means of environmental variables. export BINDIR="somedir" export SBINDIR="somedir" export LIBDIR="somedir" export MANDIR="somedir" export INCLUDEDIR="somedir" export LIBEXECDIR="somedir" make install But those settings will not be honored, sticking to default Makefile values. This patch fixes the issue. Signed-off-by:
Francesco Giancane <francescogiancane8@gmail.com> Reviewed-by:
Dmitry Safonov <dsafonov@virtuozzo.com> Signed-off-by:
-
Francesco Giancane authored
I tried to build and install the criu package from source. All went well, unless for the install part. In particular, PREFIX=/ DESTDIR=~/build_criu PREFIX=${PREFIX} DESTDIR=${DESTDIR} make install resulted in installing criu packages in ~/build_criu/usr/local/, ignoring PREFIX, even if this variable should be configurable as of manual and GCS. Reviewed-by: -
Mike Rapoport authored
When files are added to the report shutil.copytree is unhappy with lazy-pages.socket. Tell shutil.copytree that it should ignore *.socket. Signed-off-by:
-
Kirill Tkhai authored
Signed-off-by:
Kirill Tkhai <ktkhai@virtuozzo.com> Signed-off-by:
-
Mike Rapoport authored
Acked-by:
-
Andrei Vagin authored
Otherwise someone can include kerndat.h before config.h and get another kerndat structure. For example, proc_parse doesn't include config.h and Cyrill met this problem. Cc: Dmitry Safonov <dsafonov@virtuozzo.com> Cc: Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
-
Pavel Emelyanov authored
There's potential NULL-derefernece in dump_netns_con() -- two xmalloc results are not checked. However, since there's a huge set of these xmallocs, I propose to relax the whole thing with one big xmalloc and xptr_pull() helper. Signed-off-by:
-
Kirill Tkhai authored
It's unused and it's rewritten in shmem_wait_and_open(), and it just confuses a reader. So, kill it. Signed-off-by: -
Alice Frosi authored
For dumping threads we execute parasite code before we collect the floating point registers with get_task_regs(). The following describes the code path were this is done: dump_task_threads() for (i = 0; i < item->nr_threads; i++) dump_task_thread() parasite_dump_thread_seized() compel_prepare_thread() prepare_thread() ### Get general purpose registers ### ptrace_get_regs(ctx->regs) ### parasite code is executed in thread ### compel_run_in_thread(tctl, PARASITE_CMD_DUMP_THREAD) compel_get_thread_regs() ### Get FP and VX registers ### get_task_regs() Since on s390 gcc uses floating point registers to cache general purpose without the -msoft-float option the floating point registers would have been clobbered after compel_run_in_thread(). With this patch we first save all thread registers with task_get_regs() and then call the parasite code. We introduce a new function arch_set_thread_regs() that restores the saved registers for all threads. This is necessary for criu dump --leave-running, --check-only, and error handling. Pre-dump does not require to save the registers for the threads because because only the main thread (task) is used for parsite code execution. The above changes allow us to use all register sets in the parasite code - therefore we can remove -msoft-float on s390. Reviewed-by:Michael Holzheu <holzheu@linux.vnet.ibm.com> Signed-off-by:
Alice Frosi <alice@linux.vnet.ibm.com> Reviewed-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
-
Michael Holzheu authored
This test can be used to verify FP and VX registers on s390: - Verify that "criu restore" sets the correct register sets from "criu dump": $ zdtmp.py run -t zdtm/static/s390x_regs_check - Verify that dumpee continues running with correct registers after parasite injection: $ zdtm.py run --norst -t zdtm/static/s390x_regs_check $ zdtm.py run --norst --pre 2 -t zdtm/static/s390x_regs_check $ zdtm.py run --check-only -t zdtm/static/s390x_regs_check Reviewed-by:
-
Michael Holzheu authored
We have to include Makefile.inc earlier in order to use SRCARCH. Signed-off-by:
-
- 09 Aug, 2017 21 commits
-
-
Adrian Reber authored
The upcoming glibc release renamed 'struct ucontext' to 'struct ucontext_t': https://sourceware.org/git/gitweb.cgi?p=glibc.git;a=commitdiff;h=251287734e89a52da3db682a8241eb6bccc050c9;hp=c86ed71d633c22d6f638576f7660c52a5f783d66 Instead of using 'struct ucontext' this patch changes it to the typedef ucontext_t which already exists in older and new versions of glibc. Signed-off-by:
Adrian Reber <areber@redhat.com> Reviewed-by:
-
Adrian Reber authored
The upcoming glibc release renamed 'struct ucontext' to 'struct ucontext_t': https://sourceware.org/git/gitweb.cgi?p=glibc.git;a=commitdiff;h=251287734e89a52da3db682a8241eb6bccc050c9;hp=c86ed71d633c22d6f638576f7660c52a5f783d66 Instead of using 'struct ucontext' this patch changes it to the typedef ucontext_t which already exists in older and new versions of glibc. Signed-off-by:
-
Adrian Reber authored
The upcoming glibc release renamed 'struct ucontext' to 'struct ucontext_t': https://sourceware.org/git/gitweb.cgi?p=glibc.git;a=commitdiff;h=251287734e89a52da3db682a8241eb6bccc050c9;hp=c86ed71d633c22d6f638576f7660c52a5f783d66 Instead of using 'struct ucontext' this patch changes it to the typedef ucontext_t which already exists in older and new versions of glibc. Signed-off-by:
-
Adrian Reber authored
Signed-off-by:
-
Gianfranco Costamagna authored
This fixes the ppc64el build failure CC criu/arch/ppc64/sigframe.o In file included from criu/arch/ppc64/sigframe.c:5:0: criu/arch/ppc64/include/asm/types.h:32:0: error: "AT_VECTOR_SIZE_ARCH" redefined [-Werror] #define AT_VECTOR_SIZE_ARCH 6 Signed-off-by: -
Michael Holzheu authored
The entries in /proc/<pid>/maps look like the following: 3fffffdf000-40000000000 rw-p 00000000 00:00 0 The upper address is the first address that is *not* included in the address range. Our function max_mapped_addr() should return the last valid address for a process, but currently returns the first invalid address. This can lead to the following error message on kernel that have kernel commit ee71d16d22bb: Error (criu/proc_parse.c:694): Can't dump high memory region 1ffffffffff000-20000000000000 of task 24 because kernel commit ee71d16d22bb is missing Fix this and make max_mapped_addr() the last valid address (first invalid address - 1). Reported-by:
-
Michael Holzheu authored
We currently define "CFLAGS += -msoft-float -fno-optimize-sibling-calls" in Makefile.compel. Makefile.compel is included into the toplevel Makefile and so changes CFLAGS which are exported to all criu and zdtm Makefiles. We must not use -msoft-float outside the compel files. E.g. otherwise for zdtm we get the following build error: uptime_grow.o: In function `main': /tmp/2/criu/test/zdtm/static/uptime_grow.c:36: undefined reference to `__floatdidf' /tmp/2/criu/test/zdtm/static/uptime_grow.c:36: undefined reference to `__muldf3' /tmp/2/criu/test/zdtm/static/uptime_grow.c:36: undefined reference to `__floatdidf' /tmp/2/criu/test/zdtm/static/uptime_grow.c:36: undefined reference to `__adddf3 Fix this and move the CFLAGS definition to the compel Makefiles. We do this by defining a new flag CFLAGS_PIE that is only used for pie code. Reported-by:
-
Michael Holzheu authored
We have to use "debian.jessie" instead of "ubuntu.xenial" because qemu-s390x-static can only run z900 instructions and Ubuntu is built with a more recent architecture level set. We first enable only gcc because there are still some issues with clang. Signed-off-by:
-
Michael Holzheu authored
Debian is compiled for z900 - therefore we can't use the "laa" instruction. Fix this and replace "laa" by the older "cs" (compare and swap) loop. Signed-off-by:
-
Michael Holzheu authored
The flogr instruction not supported by debian jessie (z900). So replace it by the gcc built-in. Signed-off-by:
-
Michael Holzheu authored
Use the "-fno-optimize-sibling-calls" gcc option to prevent GOT relocations caused by gcc versions without the commit "S/390: Fix 64 bit sibcall": * https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=233169 Otherwise we get compile errors like: GEN criu/pie/parasite-blob.h Error (compel/src/lib/handle-elf-host.c:572): Unsupported relocation of type 26 Those can be caused by the following relocation types: * R_390_GOTENT (e.g. RHEL7 based systems) * R_390_GOTPCDBL (e.g. Debian Jessie) * R_390_GOT12 (e.g. Debian Jessie) Signed-off-by:
-
Michael Holzheu authored
For kernels that don't have commit ee71d16d22 ("s390/mm: make TASK_SIZE independent from the number of page table levels") criu sets TASK_SIZE to 4 TB on s390 (see compel_task_size()). When dumping tasks >= 4 TB on such systems, we would silently loose memory >= 4TB. So add a check and refuse to dump the task in that case. When restoring tasks >= 4 TB on such systems, the remap() call for moving the vmas at the end of the restor process would fail and a very unclear error message is printed. So add a check and write a more speaking error message in that case. Reviewed-by: -
Michael Holzheu authored
On s390 we have special console drivers with minors 64 and 65. Add support for them in criu code and in zdtm testsuite. Reviewed-by:
-
Michael Holzheu authored
If the kernel contains patch ee71d16d22 ("s390/mm: make TASK_SIZE independent from the number of page table levels") we are able to checkpoint tasks > 4TB. Add a testcase to verify this. Signed-off-by: -
Michael Holzheu authored
Reviewed-by:
-
Michael Holzheu authored
Reviewed-by:
-
Michael Holzheu authored
Add s390 parts to common code files. Patch history ------------- v2->v3: * Add: Fix big endian byte order checks Reviewed-by:
-
Michael Holzheu authored
Reviewed-by:
-
Michael Holzheu authored
Reviewed-by:
-
Michael Holzheu authored
Add s390 parts to common code files. Patch history ------------- v2->v3: * Add: s390: Consolidate -msoft-float into Makefile.compel Reviewed-by:
-
Michael Holzheu authored
This patch only adds the support but does not enable it for building. Reviewed-by:
-
