- 08 May, 2015 3 commits
-
-
Tycho Andersen authored
Note that we should only do the ns specific mounting when we are actually in a ns test. Signed-off-by:
Tycho Andersen <tycho.andersen@canonical.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Tycho Andersen authored
Signed-off-by:
-
Andrey Vagin authored
Cc: Tycho Andersen <tycho.andersen@canonical.com> Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
-
- 07 May, 2015 3 commits
-
-
Cyrill Gorcunov authored
In case if there is no .gitid file we might endup with build error. Make sure it exist. Reported-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
-
Kir Kolyshkin authored
In case opts.ps_socket is set (see commit 7058714f), we don't call accept() and so the peer address (caddr) is left uninitialized, but we try to print it. Fix by moving the printing code to right after accept(). Reported by Coverity, CID 51645. Signed-off-by:
Kir Kolyshkin <kir@openvz.org> Signed-off-by:
-
Ruslan Kuprieiev authored
Signed-off-by:
Ruslan Kuprieiev <rkuprieiev@cloudlinux.com> Signed-off-by:
-
- 05 May, 2015 17 commits
-
-
Laurent Dufour authored
Page size may change, so page size should be read through sysconf. Suggested-by:
Christopher Covington <cov@codeaurora.org> Signed-off-by:
Laurent Dufour <ldufour@linux.vnet.ibm.com> Signed-off-by:
-
Laurent Dufour authored
Fixing comment leading to think that 32bit tasks are supported, which is not the case. For the record, ppc64le is not supporting 32bit task, while ppc64 (the Big Endian architecture) has an option to support 32bit task but CRIU doesn't yet run on ppc64. Reported-by:
-
Cyrill Gorcunov authored
We might miss entry in "ri ? ri - 1" expression when ri = 1. Lets use known array size instead. For some reason it didn't trigger on my tests earlier. Reported-by:
Andrew Vagin <avagin@odin.com> Signed-off-by:
-
Andrey Vagin authored
Locks created by flock() are associated with an open file description This means that duplicate file descriptors (created by, for example, fork or dup) refer to the same lock. Reported-by: Mr Jenkins Signed-off-by:
-
Andrew Vagin authored
This variable is used from Makefile.zdtm Signed-off-by:
-
Cyrill Gorcunov authored
It is not used outside of net.c Signed-off-by:
-
Cyrill Gorcunov authored
This saves memory. Signed-off-by:
-
Cyrill Gorcunov authored
Signed-off-by:
-
Laurent Dufour authored
When it is run, cscope complains about files it cannot find: cscope: cannot find file test/zdtm/live/streaming/socket-tcp.c cscope: cannot find file test/zdtm/live/streaming/socket-tcp6.c cscope: cannot find file test/zdtm/live/static/socket-tcpbuf-local.c cscope: cannot find file test/zdtm/live/static/mntns_shared_bind02.c cscope: cannot find file test/zdtm/live/static/socket_listen6.c cscope: cannot find file test/zdtm/live/static/socket-tcp6.c cscope: cannot find file test/zdtm/live/static/socket-tcpbuf6.c cscope: cannot find file test/zdtm/live/static/mntns_link_ghost.c cscope: cannot find file test/zdtm/live/transition/fork2.c These files are relative symbolic links, and since cscope is run at the top directory, the ../* links are not valid. This patch removes the symbolic links from the list of scanned files. This is not removing any tags since the real file are still scanned. In addition, the test directory is not taken in account. Signed-off-by:
-
Kir Kolyshkin authored
Fix the following error: > > LINK arch/x86/syscalls.built-in.o > > arch/x86/crtools.c:36:20: error: unused function '__check_code_syscall' > > [-Werror,-Wunused-function] > > static inline void __check_code_syscall(void) As the function consists of a few BUILD_BUG_ONs, it gets optimized out. Let's add __attribute__((__unused__)) so clang stops complaining. [v2: s/used/unused/, fix all the arches, whitespace cleanup] Signed-off-by:
-
Kir Kolyshkin authored
I use Makefile.local to add my own rules for submitting a build to Coverity. As those rules contain login credentials, they should be kept private. Including Makefile.local (if present) seems to be the best way to achieve that. Also, add Makefile.local to .gitignore to make sure it never gets commited into repo. Signed-off-by:
-
Andrey Vagin authored
This is unexpected behaviour. pb_write_one open_image_lazy do_open_image xfree(img) 255 if (pb_write_one(img, &cpu_info, PB_CPUINFO) < 0) { >>> >>> CID 92728: Memory - illegal accesses (USE_AFTER_FREE) >>> >>> Calling "close_image" dereferences freed pointer "img". 256 close_image(img); 257 return -1; 258 } Signed-off-by: -
Andrey Vagin authored
Signed-off-by:
-
Andrey Vagin authored
288 if (flock(fd, LOCK_SH)) { 289 pr_perror("Can't take a lock\n"); >>> >>> CID 92726: Resource leaks (RESOURCE_LEAK) >>> >>> Handle variable "fd" going out of scope leaks the handle. 290 return -1; 291 } Signed-off-by: -
Kir Kolyshkin authored
When open_image() was modified to return a pointer rather than an int in commit 295090c1, these two checks were overlooked and never fixed. Signed-off-by:
-
Kir Kolyshkin authored
When using pr_perror(), format string should not end with \n, as it is added by the macro itself. Signed-off-by:
-
Laurent Dufour authored
Some architectures like ppc64 requires a trampoline to be called prior to the standard restorer services. This patch introduces 3 trampolines which can be overwritten by architectures in arch/x/include/asm/restore.h: - arch_export_restore_thread - arch_export_restore_task - arch_export_unmap The architecture which doesn't need to overwrite them, has nothing to do. Signed-off-by:
-
- 30 Apr, 2015 1 commit
-
-
Laurent Dufour authored
This patch initiates the ppc64le architecture support in CRIU. Note that ppc64 (Big Endian) architecture is not yet supported since there are still several issues to address with this architecture. However, in the long term, the two architectures should be addressed using the almost the same code, so sharing the ppc64 directory. Major ppc64 issues: Loader is not involved when the parasite code is loaded. So no relocation is done for the parasite code. As a consequence r2 must be set manually when entering the parasite code, and GOT is not filled. Furthermore, the r2 fixup code at the services's global address which has not been fixed by the loader should not be run. Branching at local address, as the assembly code does is jumping over it. On the long term, relocation should be done when loading the parasite code. We are introducing 2 trampolines for the 2 entry points of the restorer blob. These entry points are dealing with r2. These ppc64 specific entry points are overwritting the standard one in sigreturn_restore() from cr-restore.c. Instead of using #ifdef, we may introduce a per arch wrapper here. CRIU needs 2 kernel patches to be run powerpc which are not yet upstream: - Tracking the vDSO remapping - Enabling the kcmp system call on powerpc Feature not yet supported: - Altivec registers C/R - VSX registers C/R - TM support - all lot of things I missed.. Signed-off-by:
-
- 29 Apr, 2015 2 commits
-
-
Andrew Vagin authored
Signed-off-by:
-
Andrew Vagin authored
00:03:27.746 (00.008815) Error (bfd.c:149): bfd: Error reading file: No such process Reported-by: Mr Jenkins Signed-off-by:
-
- 27 Apr, 2015 10 commits
-
-
Andrew Vagin authored
We want to execute this test in Jenkins. Signed-off-by:
-
Andrew Vagin authored
It's required to execute the test a few times Signed-off-by:
-
Andrew Vagin authored
It resuces I/O load. Signed-off-by:
-
Andrey Vagin authored
maps007 fails due to intensive load Signed-off-by:
-
Andrey Vagin authored
It isn't required here and Mr Jenkins can't execute this job. Signed-off-by:
-
Andrey Vagin authored
Signed-off-by:
-
Oleg Nesterov authored
Subject. Signed-off-by:
Oleg Nesterov <oleg@redhat.com> Signed-off-by:
-
Oleg Nesterov authored
CRIU always retores the mounts as MNT_RELATIME. This is because the kernel uses this mode by default, so we need to pass MS_STRICTATIME explicitely if we didn't see "noatime" or "MS_RELATIME". While at it, make mnt_opt2flag[] and sb_opt2flag "static", otherwise gcc actually creates these arrays on stack even if there are "const". Signed-off-by:
-
Andrey Vagin authored
/proc/locks can contain a wrong pid for a lock and we always need to check this fact. Starting with the 4.1 kernel, locks are reported in fdinfo. v2: rebase to the curret master skip note_file_lock() Signed-off-by: -
Andrey Vagin authored
Starting with the 4.1 kernel, fdinfo contains information about file locks. v2: s/has_lock/has_fdinfo_lock/ Signed-off-by:
-
- 24 Apr, 2015 4 commits
-
-
Andrew Vagin authored
- ext pipe test should build binar - criu no longer accepts "extra" empty arguments - adduser doesn't add more than one same user All this came unnoticed due to error in tests failure detection (36a13cbd test: don't ignore errors other other tests) Signed-off-by:
-
Pavel Emelyanov authored
Signed-off-by: -
Andrey Vagin authored
Signed-off-by:
-
Cyrill Gorcunov authored
We need a custom flags to build 32bit varian of criu on 64bit host system, lets pass @ldflags-y here for that. Signed-off-by:
-
