- 19 Dec, 2014 10 commits
-
-
Ruslan Kuprieiev authored
Currently zdtm doesn't set additional groups when running tests. It sets uids and gids to some non-root values, but additional groups remain untouched, which allows tests to inherit groups list, which leads to different behaviour depending on a way you run tests. Signed-off-by:
Ruslan Kuprieiev <kupruser@gmail.com> Acked-by:
Andrew Vagin <avagin@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Ruslan Kuprieiev authored
Signed-off-by:
Ruslan Kuprieiev <kupruser@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Ruslan Kuprieiev authored
This is a very common error when using criu. The problem here is that we need to somehow transfer cr_errno from one process to another. I suggest using pipe to give one end to children and read cr_errno on other after restore is finished. v2, Pavel suggested putting errno into shared task_entries. v3. and he also suggested using cmpxchg Signed-off-by:
Ruslan Kuprieiev <kupruser@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Ruslan Kuprieiev authored
Signed-off-by:
Ruslan Kuprieiev <kupruser@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Ruslan Kuprieiev authored
Signed-off-by:
Ruslan Kuprieiev <kupruser@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Ruslan Kuprieiev authored
Signed-off-by:
Ruslan Kuprieiev <kupruser@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Nikolay Plotnik authored
Some packages monitor the FS /. (00.573051) irmap: Scanning /. hint (00.573053) irmap: Stat for 110025: INO - 2891453, DEV - 64768 (00.573055) irmap: Refresh stat for /. (00.573063) irmap: MNTNS_ROOT: 110025, PATH: /., INO: 2891453, DEV: 64768 Signed-off-by:
Nikolay Plotnik <nikolay.plotnik@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
In case if we're migrating to another node where @total_sleep_time has a big value we might fail here because we don't support tracking of @total_sleep_time at all, neither we can somehow restore it without kernel patching (and it's hardly if ever be allowed in vanilla kernel). Thus simply remember @total_sleep_time value before and after c/r stage removing it from the computations. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Saied Kazemi authored
Add comment to inherit_fd_reused() explaining how an inherit fd may be closed or reused outside the inherit fd logic. Signed-off-by:
Saied Kazemi <saied@google.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
(00.312396) 4: Error (pipes.c:301): Not enough room for pipe 562178190 identifier string Reported-by: Mr Jenkinks Cc: Saied Kazemi <saied@google.com> Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 17 Dec, 2014 3 commits
-
-
Pavel Emelyanov authored
It's for more natural adding of new else-if branch for aio. 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>
-
- 11 Dec, 2014 1 commit
-
-
Ruslan Kuprieiev authored
Reported-by:
Paschalis Mpeis <paschalis.mpeis@ed.ac.uk> Signed-off-by:
Ruslan Kuprieiev <kupruser@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 10 Dec, 2014 6 commits
-
-
Saied Kazemi authored
Hi Saied, This patch adds your test in the criu test system. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
We may not have permissions for this. Reported-by:
Ruslan Kuprieiev <kupruser@gmail.com> Cc: Ruslan Kuprieiev <kupruser@gmail.com> Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Ruslan Kuprieiev authored
Signed-off-by:
Ruslan Kuprieiev <kupruser@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Saied Kazemi authored
There are cases where a process's file descriptor cannot be restored from the checkpoint images. For example, a pipe file descriptor with one end in the checkpointed process and the other end in a separate process (that was not part of the checkpointed process tree) cannot be restored because after checkpoint the pipe will be broken. There are also cases where the user wants to use a new file during restore instead of the original file at checkpoint time. For example, the user wants to change the log file of a process from /path/to/oldlog to /path/to/newlog. In these cases, criu's caller should set up a new file descriptor to be inherited by the restored process and specify the file descriptor with the --inherit-fd command line option. The argument of --inherit-fd has the format fd[%d]:%s, where %d tells criu which of its own file descriptors to use for restoring the file identified by %s. As a debugging aid, if the argument has the format debug[%d]:%s, it tells criu to write out the string after colon to the file descriptor %d. This can be used, for example, as an easy way to leave a "restore marker" in the output stream of the process. It's important to note that inherit fd support breaks applications that depend on the state of the file descriptor being inherited. So, consider inherit fd only for specific use cases that you know for sure won't break the application. For examples please visit http://criu.org/Category:HOWTO. v2: Added a check in send_fd_to_self() to avoid closing an inherit fd. Also, as an extra measure of caution, added checks in the inherit fd look up functions to make sure that the inherit fd hasn't been reused. The patch also includes minor cosmetic changes. Signed-off-by:
Saied Kazemi <saied@google.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Oleg Nesterov authored
Signed-off-by:
Oleg Nesterov <oleg@redhat.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Oleg Nesterov authored
PS: never ever use PTRACE_KILL. Signed-off-by:
Oleg Nesterov <oleg@redhat.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 08 Dec, 2014 7 commits
-
-
Ruslan Kuprieiev authored
Signed-off-by:
Ruslan Kuprieiev <kupruser@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Ruslan Kuprieiev authored
Signed-off-by:
Ruslan Kuprieiev <kupruser@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Ruslan Kuprieiev authored
Signed-off-by:
Ruslan Kuprieiev <kupruser@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
The kernel can do it better. The problem exists only for recv queues. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Ruslan Kuprieiev authored
Signed-off-by:
Ruslan Kuprieiev <kupruser@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Vladimir Davydov authored
We may close the master before reading the slave if the child runs first when the parent wakes it up using task_waiter_complete. In this case the subsequent read from the slave will return 0 resulting in a false positive test failure: 13:23:42.194: 30914: ERR: pty02.c:89: read(slave) failed (errno = 11 (Resource temporarily unavailable)) Fix this by waking the child, which closes the master, only after the read has been completed. Also note errno = EAGAIN in the failure report above, which looks really confusing. Let's print the actual number of bytes read/written along with the errno while reporting a read/write failure to make the error reporting clearer in this test. Signed-off-by:
Vladimir Davydov <vdavydov@parallels.com> Cc: Cyrill Gorcunov <gorcunov@parallels.com> Acked-by:
Andrew Vagin <avagin@parallels.com> Acked-by:
Cyrill Gorcunov <gorcunov@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
maps007 segfaults on i386, because "size" is calculated bigger than allowed. This occurs when the result of lrand48() * PAGE_SIZE is negative. In this case the % operation returns a negative value too, what is unexpected. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 01 Dec, 2014 4 commits
-
-
Pavel Emelyanov authored
So, according to the recently accepted plan we have 1.4. Biggest achievement here is that we've finally started with user namespaces, fixed CRIU to work smoothly with the p.haul project and optimized images and /proc files IO. And a bunch of bugs, of course. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
When we generate a path to the previous images, we take a directory name and the previous iteration number. Reported-by: Mr Jenkins Signed-off-by:
Andrew Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
We see big load-avarage for the CRIU-iter job, because the host is writing images for the maps04 test. If we will remove images after retore, they will live in a page cache. Curretly we collect images for three iterations and their size is too big to live in a cache. [root@jenkins workspace]# time bash test/zdtm.sh -i 3 ns/static/maps04 real 2m23.765s user 0m0.719s sys 0m4.006s [root@jenkins workspace]# time bash test/zdtm.sh -C -i 3 ns/static/maps04 real 1m34.884s user 0m0.200s sys 0m4.191s Signed-off-by:
Andrew Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrew Vagin authored
Currently ns/test and test may have the same test directory, especially if zdtm.sh is executed with --ct. Reported-by: Mr Jenkins Signed-off-by:
Andrew Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 29 Nov, 2014 3 commits
-
-
Andrey Vagin authored
32 is too many for our test server Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
CID 83243 (#1 of 1): Resource leak (RESOURCE_LEAK) 6. leaked_handle: Handle variable fd going out of scope leaks the handle. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
When we are doing pre-dump, we splice pages in pipes and only then open images and dump pages. But when we are splicing pages, we need to know about existence of parent images. This patch adds a new call to determin existence of parent images. In addition this patch fixes a following issue: CID 83244 (#1 of 1): Uninitialized pointer read (UNINIT) 14. uninit_use: Using uninitialized value xfer.parent. v2: initialize unused field of struct page_server_iov, because it sends in network. CID 83451 (#1 of 1): Uninitialized scalar variable (UNINIT) 2. uninit_use_in_call: Using uninitialized value pi. Field pi.nr_pages is uninitialized when calling write. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 26 Nov, 2014 2 commits
-
-
Andrey Vagin authored
For stream sockets it's possiable and we should not fail in this case. 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> Acked-by:
Andrew Vagin <avagin@parallels.com>
-
- 19 Nov, 2014 3 commits
-
-
vkonyashkin authored
Need to print a mask of events for UNKNOWN event in inotify_system_nodel.c and inotify_system.c tests. The current error message in this case is too vague: inotify_system_nodel.c:266: got unexpected event UNKNOWN Signed-off-by:
vkonyashkin <vkonyashkin@parallels.com> Acked-by:
Andrew Vagin <avagin@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
It is in mainline and required for proper CRIU work. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
And move the 2nd piece lower to avoid fwd declaration and keep similar calls close to each other. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 14 Nov, 2014 1 commit
-
-
Cyrill Gorcunov authored
Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-