- 20 Mar, 2014 1 commit
-
-
Andrey Vagin authored
The current code think that /vz/lxc/centos-6-x86_64-root is in /vz/lxc/centos-6-x86_64. If the path is not equal to mountpoint, we need to check, that path contains a slash after mountpoint. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Acked-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 19 Mar, 2014 6 commits
-
-
Alexander Kartashov authored
This patch splits the file arch/x86/vdso-pie.c into machine-dependent and machine-independent parts by moving the routines vdso_fill_symtable(), vdso_proxify(), and vdso_remap() to the file pie/vdso.c. The ARM version of the routines is moved to the source pie/vdso-stub.c to provide the vDSO proxy stub implementation for architectures that don't provide the vDSO. Signed-off-by:
Alexander Kartashov <alekskartashov@parallels.com> Looks-good-to: Cyrill Gorcunov <gorcunov@openvz.org> Reviewed-by:
Christopher Covington <cov@codeaurora.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Alexander Kartashov authored
These variables are architecture-specific so they shouldn't appear in the upcoming generic version of the routine vdso_fill_symtable(). At first glance it seems it's better to make these variables global and reference them as external in the generic version of the routine. However experiments with the vDSO restore routine for AArch64 showed that the AArch64 compiler uses the GOT to access such variables rendering our blobs unrelocatable. Signed-off-by:
Alexander Kartashov <alekskartashov@parallels.com> Looks-good-to: Cyrill Gorcunov <gorcunov@openvz.org> Reviewed-by:
Christopher Covington <cov@codeaurora.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Alexander Kartashov authored
This patch makes the name of the vDSO symbol name table a bit more appropriate for the generic version of the routine vdso_fill_symtable(). Signed-off-by:
Alexander Kartashov <alekskartashov@parallels.com> Looks-good-to: Cyrill Gorcunov <gorcunov@openvz.org> Reviewed-by:
Christopher Covington <cov@codeaurora.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Alexander Kartashov authored
This patch moves the enum VDSO_SYMBOL_* and macros VDSO_SYMBOL_*_NAME to the x86 specific header since different architectures export different symbols from their vDSOs. Signed-off-by:
Alexander Kartashov <alekskartashov@parallels.com> Looks-good-to: Cyrill Gorcunov <gorcunov@openvz.org> Reviewed-by:
Christopher Covington <cov@codeaurora.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Alexander Kartashov authored
This patch splits the file arch/x86/vdso.c into machine-independent and machine-dependent parts by moving the routine vdso_init() from the file vdso.c. The routine seems to be suitable for all architectures supporting the vDSO. The ARM version of the routine is moved to the source vdso-stub.c that is supposed to be the vDSO proxy stub implementation for architectures that don't provide the vDSO. The build scripts are adjusted as well to enable selection between the full-fledged and stub vDSO proxy implementations. Signed-off-by:
Alexander Kartashov <alekskartashov@parallels.com> Looks-good-to: Cyrill Gorcunov <gorcunov@openvz.org> Reviewed-by:
Christopher Covington <cov@codeaurora.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Alexander Kartashov authored
Supported machine architectures provide TLS stogares of different sizes: the size of the TLS storage in x86-64 is 24 bytes, ARM --- 4 bytes and upcoming AArch64 --- 8 bytes. This means every supported architecture needs a specific type to store the value of the TLS register. This patch reworks the insterface of the routines arch_get_tls() and restore_tls() passing them the TLS storage by pointer rather than by value to simplify the TLS stub for x86. Signed-off-by:
Alexander Kartashov <alekskartashov@parallels.com> Reviewed-by:
Cyrill Gorcunov <gorcunov@openvz.org> Reviewed-by:
Christopher Covington <cov@codeaurora.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 18 Mar, 2014 5 commits
-
-
Andrey Vagin authored
For example restore_shmem_content allocates the page_read structure on stack. Cc: Pavel Emelyanov <xemul@parallels.com> Reported-by: Jenkins Criuovich Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
bers stads for berserker which should eat computer resources emulating "load" for CRIU performance testing. It's still far from being complete but can do trivial things: - generate mmap's with memory dirtified - open files Nothing serious. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Tikhomirov Pavel authored
if final size of all pages images is zero than dedup works use : bash test/zdtm.sh --auto-dedup bash test/zdtm.sh --auto-dedup static/maps04 bash test/zdtm.sh -P -i 3 --auto-dedup -t transition/maps007 changes: aplicable for all tests, -ad changed to --auto-dedup, simplify, check all private pages images, check shmem images and go with shmem patch set. Acked-by:
Andrew Vagin <avagin@parallels.com> Signed-off-by:
Tikhomirov Pavel <snorcht@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Tikhomirov Pavel authored
in restore_shmem_content() use open_page_read() to open images 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>
-
- 17 Mar, 2014 1 commit
-
-
Cyrill Gorcunov authored
During the time some files become obsolete and might be missing in checkpoint image set, but to keep backward compatibility we still trying to open them, which might print out error like | Unable to open 'path-to-file' and confuse a reader why criu prints error but continue working. To eliminate this problem O_OPT flag has been introduced in commit 16b56920, which suppress error message priting if the flag is set. Now start using O_OPT in the following functions - open_irmap_cache: irmap cache is relatively new optional feature - prepare_rlimits, open_signal_image, restore_file_locks, prepare_fd_pid, prepare_mm_pid, collect_image: all these helpers are trying to open image files which can be missing. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 14 Mar, 2014 14 commits
-
-
Cyrill Gorcunov authored
On PI machine we've got | CC protobuf.o | pstree.c: In function ‘core_entry_alloc’: | pstree.c:36:10: error: ‘RLIM_NLIMITS’ undeclared (first use in this function) due to old kernel headers. Note I've dropped off BUG_ON here to localize all things in pstree code, no need to sprinkle constants. Reported-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
No all distros (Rpi) provide O_PATH definition, so include fcntl.h here thus we don't hit compilation problem like | CC image.o | image.c: In function ‘open_image_at’: | image.c:187:29: error: ‘O_PATH’ undeclared (first use in this function) Reported-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
The array element is RlimitEntry properly initialized, no need in additional memcpy-s and size-checks. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
On Thu, Mar 13, 2014 at 02:30:50PM +0400, Cyrill Gorcunov wrote: > > This image is deprecated now so move it out of > _CR_FD_TASK thus we won't be even generating it > on the dump. > > Signed-off-by: Cyrill Gorcunov <gorcunov@openvz.org> Updated >From cb9c3953beac7d42de80635e7a6e537cc867c479 Mon Sep 17 00:00:00 2001 From: Cyrill Gorcunov <gorcunov@openvz.org> Date: Thu, 13 Mar 2014 14:24:50 +0400 Subject: [PATCH 7/7] rlimit: Move CR_FD_RLIMIT out of _CR_FD_TASK This image is deprecated now so move it out of _CR_FD_TASK thus we won't be even generating it on the dump. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
We're using new image format, but old image file is still generated. This will be addressed in next patch. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
To save backward compatibility try to read data from old image if Core entry doesn't has rlimits bound. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Note the restore remains as is for a while, it'll be addressed later. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
On Thu, Mar 13, 2014 at 04:20:37PM +0400, Pavel Emelyanov wrote: > > Would you rework this patch on top of my recent > "allocate Core in on xmalloc call" one? Attached. >From c2233d4fafce30c4e7214a1a7ab3677824a30d75 Mon Sep 17 00:00:00 2001 From: Cyrill Gorcunov <gorcunov@openvz.org> Date: Thu, 13 Mar 2014 16:57:14 +0400 Subject: [PATCH] rlimit: Allocate and free appropriate Core entry Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
We will carry rlimits inside Core itself instead of separate image. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
This allows us to distinguish the situation where image to be opened is missing but optional, thus no error message should be printed. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
arch/x86/crtools.c: In function ‘arch_alloc_thread_info’: arch/x86/crtools.c:271:6: error: ‘with_xsave’ may be used uninitialized in this function [-Werror=uninitialized] Actually the with_xsave is with_fpu dependant, but some gccs can't guess that fact :\ 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> Acked-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
- 13 Mar, 2014 1 commit
-
-
Andrey Vagin authored
For example opening the FIFO blocks until the other end is opened also. We can use O_PATH to avoid sleeping in the open() call. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 11 Mar, 2014 9 commits
-
-
Andrey Vagin authored
It's going to be used for restoring namespaces. For example we need to enumirate the ns_ids list for restoring mount namespaces. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Alexander Kartashov authored
The implementation of bit operations for ARM isn't actually architecture-specific so it would rather be shared with the upcoming port for AArch64 that won't provide optimized implementation of bit operations. Signed-off-by:
Alexander Kartashov <alekskartashov@parallels.com> Reviewed-by:
Christopher Covington <cov@codeaurora.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Alexander Kartashov authored
Documentation: remove the linker command line switch --static from the final step described in the cross-compilation guide Static linking of the binary criu isn't supported any more. Reported-by:
riya khanna <riyakhanna1983@gmail.com> Signed-off-by:
Alexander Kartashov <alekskartashov@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Alexander Kartashov authored
Signed-off-by: Alexander Kartashov <alekskartashov@parallels.com Reviewed-by:
Christopher Covington <cov@codeaurora.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Alexander Kartashov authored
This patch should simplify the routine interface change in the following patch. Signed-off-by:
Alexander Kartashov <alekskartashov@parallels.com> Reviewed-by:
Christopher Covington <cov@codeaurora.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
pre_dump_one_fanotify calls for parse_fdinfo_pid_s where fsn_params mut not be NULL, otherwise we get nil dereference. Fix it by passing a real variable instead. Reported-by:
Pavel Tikhomirov <snorcht@gmail.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
Here is a regresion test cae for commit 1934e1a9 Author: Andrey Vagin <avagin@openvz.org> Date: Tue Feb 18 16:53:00 2014 +0400 posix-timer: take into account that sival_ptr can be NULL (v3) Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Saied Kazemi authored
The pie.lds.S.in needs two minor changes to work with the gold (/usr/bin/gold) linker. These changes are compatible with /usr/bin/ld and make linker script more portable. The first change is adding a comma before /DISCARD/ so that the grammar won't be ambiguous. A Otherwise, gold treats it as a part of the assignment and would generate a syntax error about the "unexpected ':'". The second change is moving initialization of __export_parasite_args to inside the SECTIONS command because it references the dot symbol. Otherwise, gold would generate the error "invalid reference to dot symbol outside of SECTIONS clause. Signed-off-by:
Saied Kazemi <saied@google.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 07 Mar, 2014 3 commits
-
-
Pavel Emelyanov authored
Fix for the previous patch :\ 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>
-