- 10 Feb, 2014 8 commits
-
-
Christopher Covington authored
The current mechanism for determining ARM architecture version breaks down when cross-compiling. Default to ARMv7 if the user specifies ARCH=arm. Users wishing to cross-compile for ARMv6 platforms such as the Raspberry Pi should use ARCH=armv6 as previously required. Acked-by:
Alexander Kartashov <alekskartashov@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Christopher Covington authored
Without this a user may encounter the following error. arch/arm/include/asm/atomic.h:38:2: error: implicit declaration of function ‘smp_mb’ With the #error pragma a user is a least pointed to the specific section in the file that's causing the problem arch/arm/include/asm/atomic.h:21:2: error: #error ARM architecture version (CONFIG_ARMV*) not set or unsupported. Acked-by:
Alexander Kartashov <alekskartashov@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Christopher Covington authored
ARCH is a user-facing variable and setting it on the command line clobbers any manipulation attempted in the makefile. Following the example of the Linux kernel makefiles, use a SRCARCH variable instead for the internal, potentially fixed-up, architecture name. This fixes the following error. make ARCH=x86_64 Makefile:69: *** "The architecture x86_64 isn't supported". Stop. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Christopher Covington authored
The top-level makefile defines an ARCH_DIR variable. Always use it rather than referencing ARCH directly, since ARCH may be set by the user on the command line and that raw input may not be what needs to be used internally. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Christopher Covington authored
A user may have necessary linker directories specified in CFLAGS. Make sure to include them. Change-Id: I76c2a7bd6e34cc282bfcd5f83cc69ae7fa2c8cc2 Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Christopher Covington authored
The makefile includes need to be moved for everything to be defined properly when the configure tests run. The Ubuntu 12.04 x86_64 GCC and Linaro's 13.08 and newer AArch64 GCC's have the if_packet.h kernel header, but as of 13.12, the Linaro AArch32 GCC does not. Change-Id: I363c43fdb81b028f99aac77e15bff9462c87af4b Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Adrian Reber authored
The links to the versioned libcriu.so.* included $(DESTDIR) and $(LIBDIR) which will generate wrong links if building criu packages. Now the links points to the correct file without any path. For some reason criu.8 was installed with executable permission. Use 'install' with '-m 644' to make sure this does not happen. Signed-off-by:
Adrian Reber <adrian@lisas.de> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
No need to read it in cycle. Repored-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 07 Feb, 2014 18 commits
-
-
Cyrill Gorcunov authored
Repeated custom entries are presented as an array of pointers in the memory, so once we fetch the first entry we are to continue iterating pointers instead. Before | vmas: { | start: 0x00000000400000 | end: 0x00000000406000 | pgoff: 0000000000000000 | shmid: 0x0000000000000b | prot: 0x5 | flags: 0x2 | status: 0x41 | fd: 0xffffffffffffffff | } | :{ | start: 0x00000000462d00 | end: 0000000000000000 | pgoff: 0000000000000000 | shmid: 0x00000000605000 | prot: 0x606000 | flags: 0 | status: 0x5000 | fd: 0x0000000000000b | madv: 0x00000000000041 | } After | vmas: { | start: 0x00000000400000 | end: 0x00000000406000 | pgoff: 0000000000000000 | shmid: 0x0000000000000b | prot: 0x5 | flags: 0x2 | status: 0x41 | fd: 0xffffffffffffffff | } | :{ | start: 0x00000000605000 | end: 0x00000000606000 | pgoff: 0x00000000005000 | shmid: 0x0000000000000b | prot: 0x1 | flags: 0x2 | status: 0x41 | fd: 0xffffffffffffffff | } Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Tikhomirov Pavel authored
Signed-off-by:
Tikhomirov Pavel <snorcht@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Tikhomirov Pavel authored
Signed-off-by:
Tikhomirov Pavel <snorcht@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
We're aligning members and number in protobuf files from the very beginning for readability sake. So fix up newer ones which were not. Also use /* */ comments, not //. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Kir Kolyshkin authored
setup_opts_from_req() prints an error message, so there's no need for its caller to print another one. While at it, simplify/unify error checking, treating any non-zero value as an error. Signed-off-by:
Kir Kolyshkin <kir@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Adrian Reber authored
Using a soname makes it easy for applications to detect if the ABI of libcriu.so has changed. It still creates libcriu.so as before in lib. During 'make install' additional links are created: libcriu.so -> libcriu.so.1.0 libcriu.so.1 -> libcriu.so.1.0 Signed-off-by:
Adrian Reber <adrian@lisas.de> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Tikhomirov Pavel authored
Signed-off-by:
Tikhomirov Pavel <snorcht@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
CONFIG_HAS_PEEKSIGINFO_ARGS is used there. Cc: Christopher Covington <cov@codeaurora.org> Reported-by:
Christopher Covington <cov@codeaurora.org> Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Kir Kolyshkin authored
For x86_64, LIBDIR can be tricky to find out. Nevertheless, this works for at least RedHat/CentOS/Fedora, SLES/openSUSE, Debian/Ubuntu and Gentoo. So, while it can guess LIBDIR wrong, this is still a good approximation for those installing criu from source. Signed-off-by:
Kir Kolyshkin <kir@openvz.org> Acked-by:
Cyrill Gorcunov <gorcunov@openvz.org> Acked-by:
Adrian Reber <adrian@lisas.de> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
After previous fixes we can merge and move a couple of them for easier reading. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Same for previous patch with vmas -- we do it on collect and on real open. Just put the pointer on fifo_info structure. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
We do it first -- on collect, second -- on restore. The 2nd lookup is excessive, we can put fd pointer on vm_area at lookup and reuse one later. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
On restore we only need to know currnet task mappings' start and end to find where to put the restorer blob. And since the smaps file in /proc/pid is up to 3 times slower, than the maps one, it makes perfect sense just to parse the latter one. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Those opened for cwd, root, exe link and vma-s do not care about the pos value. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 05 Feb, 2014 11 commits
-
-
Pavel Emelyanov authored
Usefult to test restore time, for example. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
With new remap refcount users will get the counter eventually as they appear. Reported-by: Andrew Vagin <avagin@parallels.com> via Jenkins Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Adrian Reber authored
To link Open MPI against CRIU I need the header files installed. Attached is a patch which adds the necessary Makefile rules to install libcriu.so and the header file to compile against CRIU. Signed-off-by:
Adrian Reber <adrian@lisas.de> Acked-by:
Kir Kolyshkin <kir@openvz.org> Acked-by:
Andrew Vagin <avagin@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
With packed reg-files we have a complex fd - file - vma - remap interaction. I think this should be reflected in the code comment. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
If a file mmaped or pointed by exe link is unlinked, we will generate a ghost file for it. On restore the ghost file will be created with the users counter 1 and the very first open (e.g. for mmap) will unlink the file. Handle this by bumping up user counter for every mapping pointing on the file. This appeared after previous patches that packed the reg-files image. Before it each vma and exe link created separate entry in the reg-files image. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
We have tons of files with equal IDs, e.g. libraries mapped to different tasks. No need in writing all this duplicate stuff into images -- cache the IDs with dev:ino keys. Acked-by:
Andrew Vagin <avagin@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
For unlinked opened and mmaped files we'd need to care about remaps, for this the callback with both file_desc and fdinfo_list_entry will be required. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Acked-by:
Andrew Vagin <avagin@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Acked-by:
Andrew Vagin <avagin@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Acked-by:
Andrew Vagin <avagin@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 04 Feb, 2014 3 commits
-
-
Andrey Vagin authored
When a process is started in a new pid namespace, the init process must colleck zombies. But before this the init process must waits the root test process, which returns a status of preparation stage. Currently these two operation can race, because the both ones call waitpid(). Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Useful to test restore time -- just abort restore with this action and that's it. Acked-by:
Andrew Vagin <avagin@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
If we specify log level to none (0) the result is LOG_INFO (2). Acked-by:
Andrew Vagin <avagin@parallels.com> Acked-by:
Kir Kolyshkin <kir@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-