- 22 Jun, 2013 4 commits
-
-
Alexander Kartashov authored
Signed-off-by:
Alexander Kartashov <alekskartashov@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
Otherwise it contains trash and BUG at page-read.c:98 may be triggered. https://bugzilla.openvz.org/show_bug.cgi?id=2633Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
All other states are unexpected. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
PARASITE_CMD_DUMPPAGES is called many times and the parasite args contains an array with vmas at this time, so VMAs can be unprotected in error case Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 13 Jun, 2013 1 commit
-
-
Pavel Emelyanov authored
Since *all* of them just call do_dump_gen_file with proper ops, just call one directly. Compacts the code. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 14 Jun, 2013 8 commits
-
-
Andrey Vagin authored
This code was written, when all threads were daemonized. Now only a thread leader is daemonized. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
For some reasons a task can go out from a parasite code. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
The controll socket is shut down for only writing, because it will wait FINI command. https://bugzilla.openvz.org/show_bug.cgi?id=2622Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
A negative value must be resturned in error case Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
ret contains a successful code of readlink Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 13 Jun, 2013 3 commits
-
-
Cyrill Gorcunov authored
In commit 2f5df09d we included asm/type.h for bool definition. This is fine in terms of CRIU but makes cpt2 converter to carry the whole asm/type.h, sometimes causing error due to confilicting definitions. So lets be simplier and include <stdbool.h> instead. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Just to protect ourself from unpredicted changes. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Chanho Park authored
A x64 only converts PRIx64 to %lx. However, ARM will convert it to %llx. Thus, we have to peek one more character and skip it if next is 'l'. Signed-off-by:
Chanho Park <chanho61.park@samsung.com> Signed-off-by:
Myungjoo Ham <myungjoo.ham@samsung.com> Acked-by:
Alexander Kartashov <alekskartashov@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 11 Jun, 2013 5 commits
-
-
Pavel Emelyanov authored
The -v option is declared as "optional argument" and thus only -v=N or -vN are valid. Currently used -v N syntax no longer works as expected, this will be treated as LOG_ERR level (single -v) with dangling (unparsed) argument. Sorry for that :( but it's better to this earlier. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Kernel has more and more links with rtnl-ops, which report a string kind of the device, which is handy for debugging. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Chanho Park authored
This patch adds LDFLAGS directive to build with -static. You can build in static mode like below: $ make ARCH=arm CROSS_COMPILE=arm-linux-gnueabi LDFLAGS=-static Signed-off-by:
Chanho Park <chanho61.park@samsung.com> Signed-off-by:
Myungjoo Ham <myungjoo.ham@samsung.com> Acked-by:
Alexander Kartashov <alekskartashov@parallels.com> Acked-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Chanho Park authored
Parasite blobs should be compiled in arm mode. Signed-off-by:
Chanho Park <chanho61.park@samsung.com> Signed-off-by:
Myungjoo Ham <myungjoo.ham@samsung.com> Acked-by:
Alexander Kartashov <alekskartashov@parallels.com> Acked-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Chanho Park authored
Sys_mmap returns 'unsigned long' value. Contrary to x64, ARM has 32bit size of 'unsigned long'. If sys_mmap returns 32bit value, higher 32bit will be filled 0xffffffff. Signed-off-by:
Chanho Park <chanho61.park@samsung.com> Signed-off-by:
Myungjoo Ham <myungjoo.ham@samsung.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 07 Jun, 2013 4 commits
-
-
Pavel Emelyanov authored
Reminder: the '&<string>' arg turns out into an address in memory containing <string>. The '@<size>' arg turns out into an address to uninitilized memory of size <size>, which contents it printed after syscall returns. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
If we meet shared mount point without share master belonging to us -- it means we might fail on restore, thus require both master/slave mount peers to be collected on dump. In other words, the output will be like | (00.077025) Error (mount.c:421): Mount 49 (master_id: 2 shared_id: 0) has unreachable sharing | (00.077123) Error (mount.c:472): Can't proceed 4237's mountinfo | (00.077865) Error (namespaces.c:442): Namespaces dumping finished with error 65280 https://bugzilla.openvz.org/show_bug.cgi?id=2608Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Allocate it with xzalloc instead of massive NULL assignment. Moreover, don't forget to initialize @siblings. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 03 Jun, 2013 3 commits
-
-
Cyrill Gorcunov authored
The jenkins framework might need some lifting up (crtools.$commit changed to criu.$commit). Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> 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>
-
Cyrill Gorcunov authored
Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 01 Jun, 2013 2 commits
-
-
Alexander Kartashov authored
This patch adds the test static/arm-neon00 modelled after the test static/fpu00: it initializes ARM NEON registers before a checkpoint, carries out a simple computation on ARM NEON after a restore and compares the result against the result of the same computation carried out before a checkpoint. Signed-off-by:
Alexander Kartashov <alekskartashov@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Alexander Kartashov authored
A struct sigframe* instead of fpu_state_t should be passed to the routine restore_fpu() since FPU registers are stored in different fields of the sigframe in different architectures. An architecture-specific implementation of the routine restore_fpu() should know details of this layout instead of construct_sigframe(). This change makes it possible to move ARM FPU restoration from sigreturn_prep_fpu_frame() (where it caused a segfault since the pointer fpu_stat has become invalid in the dumper address space) to restore_fpu() Signed-off-by:
Alexander Kartashov <alekskartashov@parallels.com> Acked-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 31 May, 2013 10 commits
-
-
Andrey Vagin authored
After creating namespaces we may need to apply some configuration. For example uid and gid maps should be applied in this moment. This script should be executed from crtools, but namespaces are created in a root process, so we need to add a new stage for synchronizing. v2: use a separate stage instead of socketpair. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Alexander Kartashov authored
The names of these functions should be swapped to avoid confusion. Signed-off-by:
Alexander Kartashov <alekskartashov@parallels.com> Acked-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Alexander Kartashov authored
The macro REG_SYSCALL_NR is designated to get the value of the CPU register storing syscall number in a machine-independent way. Signed-off-by:
Alexander Kartashov <alekskartashov@parallels.com> Acked-by:
Andrew Vagin <avagin@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Alexander Kartashov authored
The field user_regs_struct_t::ARM_ORIG_r0 isn't used to prevent a syscall restart on ARM unlike x86. Signed-off-by:
Alexander Kartashov <alekskartashov@parallels.com> Reported-by:
Chanho Park <chanho61.park@samsusng.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Tested-by:
Alexander Kartashov <alekskartashov@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
parasite_fixup_vdso is not worth calling on arm at all, moreover it might try to read pfn for addresses greater than TASK_SIZE, leading to dump interruption. Lets make it arch dependent. [alekskartashov@: Fix compiling on arm] Reported-by:
Alexander Kartashov <alekskartashov@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Tested-by:
Alexander Kartashov <alekskartashov@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Tested-by:
Alexander Kartashov <alekskartashov@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
- gather structs at top - add externs - align arguments in declarations Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Tested-by:
Alexander Kartashov <alekskartashov@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
It will be needed in x86/ code. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Tested-by:
Alexander Kartashov <alekskartashov@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> CC: Chanho Park <chanho61.park@samsung.com> CC: Myungjoo Ham <myungjoo.ham@samsung.com> Acked-by:
Chanho Park <chanho61.park@samsung.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-