Commits · 1edb5b01f3838a9c361102788ea39a49ee97ec29 · zhul / criu

01 Jul, 2013 25 commits

mem: Don't ignore memory tracking reset errors · 1edb5b01

Pavel Emelyanov authored Jul 01, 2013

Otherwise on non-soft-dirty kernel dump passes, but
produces broken image.
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

1edb5b01

timers: Unmap posix timers info mem after restoring them · 2030f2df
Pavel Emelyanov authored Jul 01, 2013
```
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
2030f2df

timers: Split posix timers restore into two stages · f8464fda

Pavel Emelyanov authored Jul 01, 2013

1st stage is -- creating the timers. It may fail if kernel
allocated IDs in a manner we don't expect or runs out of
memory.

2nd stage is -- arm the timers. It cannot fail, since we've
validated the timespecs in advance and should happen after
we've waited for all the other tasks to complete the restore.
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

f8464fda

test: app-emu -- Fix typo in command line · b62df5c9

Cyrill Gorcunov authored Jul 01, 2013

$pid variable get chopped by editor
Signed-off-by: Cyrill Gorcunov <gorcunov@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

b62df5c9

test: app-emu -- Use new -v syntax · 47faeca6

Cyrill Gorcunov authored Jul 01, 2013

We've slightly updated -v option argument parsing,
thus update cmdline used in a few app-emu tests.
Signed-off-by: Cyrill Gorcunov <gorcunov@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

47faeca6

zdtm: don't take into account cpt/rst time in posix_timer · 1b2c3e69

Andrey Vagin authored Jul 01, 2013

Add cpt/rst timeout to inaccuracy.

All timers are stopped during cpt/rst. We don't know in which moment
they are stopped and run, so this timeout is added to acceptable
inaccuracy.
Signed-off-by: Andrey Vagin <avagin@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

1b2c3e69

parasite: minor clean up · 553c0480

Andrey Vagin authored Jul 01, 2013

Signed-off-by: Andrey Vagin <avagin@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

553c0480

check: Netlink diag and peek-siginfo is in 3.10 already · ca6c026a
Pavel Emelyanov authored Jul 01, 2013
```
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
ca6c026a

files-reg: Show ghost file name on error · 00be48d5

Cyrill Gorcunov authored Jul 01, 2013

| Can't open ghost file /tmp/file_aio.cRgOVJ.cr.169.ghost: File exists
Signed-off-by: Cyrill Gorcunov <gorcunov@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

00be48d5

zdtm: maps00 -- Add more detailed output · 781cf5d3

Cyrill Gorcunov authored Jul 01, 2013

For easier detection where exactly test fails.

| 665: map: ptr 0x7fa96bbe4000 flag        2 prot        0
| 665: map: ptr 0x7fa96bbe2000 flag        1 prot        0
| 665: map: ptr 0x7fa96bbe0000 flag       22 prot        0
| 665: map: ptr 0x7fa96bbdb000 flag       21 prot        0
...
| 665: Read: 0, Expected: 553e7b06
| 665: Read: 0, Expected: 553e7b06
| 665: Read: 0, Expected: 553e7b06
| 665: FAIL: maps00.c:145: CRC mismatch: ptr 0x7fa96bbcb000 flag 21 prot        3
Signed-off-by: Cyrill Gorcunov <gorcunov@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

781cf5d3

zdtm: Wire posix-timers test in · 55b51d8f
Pavel Emelyanov authored Jul 01, 2013
```
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
55b51d8f
proc: Use open_proc helper to open timers file · db77402a
Pavel Emelyanov authored Jul 01, 2013
```
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
db77402a

check: Check that /proc/PID/timers file is here · 1f5acc02

Pavel Emelyanov authored Jul 01, 2013

It's required to get support for posix cpu timers and
it has appeared in v3.10
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

1f5acc02

posix-timer: Add macros to exclude periodic · 162866fe

Pavel Tikhomirov authored Jun 27, 2013

Explanation of exclude periodic test is that we can't set overrun for
timer, so all checks will fail in case of big overrun before dump.
Signed-off-by: Pavel Tikhomirov <snorcht@gmail.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

162866fe

posix-timer: Add test non consequent timer ids · f8bb9c36

Pavel Tikhomirov authored Jun 27, 2013

Add fake timers to inshure that then timer ids don't go one after
another c/r go well.
Signed-off-by: Pavel Tikhomirov <snorcht@gmail.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

f8bb9c36

posix-timer: Restore timers, need create and delete 'fake' timers · b782b982
Pavel Tikhomirov authored Jun 27, 2013
```
Signed-off-by: Pavel Tikhomirov <snorcht@gmail.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
b782b982
posix-timer: Add reading image and put it to shared memory · 5c8db84f
Pavel Tikhomirov authored Jun 27, 2013
```
Signed-off-by: Pavel Tikhomirov <snorcht@gmail.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
5c8db84f

posix-timer: Restore and fix one timer · 2940109a

Pavel Tikhomirov authored Jun 27, 2013

This also add function to check if time represented by two numbers is
valid. I.e. for timespec(sec, nsec), sec and nsec must be > 0 and nsec
must be less when NSEC_PER_SEC.
Signed-off-by: Pavel Tikhomirov <snorcht@gmail.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

2940109a

posix-timer: Add restore structures · d8be68f3

Pavel Tikhomirov authored Jun 27, 2013

Signed-off-by: Pavel Tikhomirov <snorcht@gmail.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

d8be68f3

posix-timer: Add dump functionality · 9a9424ee

Pavel Tikhomirov authored Jun 27, 2013

Signed-off-by: Pavel Tikhomirov <snorcht@gmail.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

9a9424ee

posix-timer: Get info in parasite, and structures · 335a158e

Pavel Tikhomirov authored Jun 27, 2013

Signed-off-by: Pavel Tikhomirov <snorcht@gmail.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

335a158e

posix-timer: Parse proc /proc/<pid>/timers and save info in list · d992960f
Pavel Tikhomirov authored Jun 27, 2013
```
Signed-off-by: Pavel Tikhomirov <snorcht@gmail.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
d992960f

posix-timer: Add fd, magic and show functionality. · 2752cf9b

Pavel Tikhomirov authored Jun 27, 2013

Signed-off-by: Pavel Tikhomirov <snorcht@gmail.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

2752cf9b

posix-timer: Add protobuf message and rename to protobuf/timer.proto · 2f98253c
Pavel Tikhomirov authored Jun 27, 2013
```
Signed-off-by: Pavel Tikhomirov <snorcht@gmail.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
2f98253c

syscalls: Add proper syscalls definition. · 652ce02e

Pavel Tikhomirov authored Jun 27, 2013

Signed-off-by: Pavel Tikhomirov <snorcht@gmail.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

652ce02e

27 Jun, 2013 2 commits

mm: handle new processes which created between snapshots (v2) · dd38ae16

Andrey Vagin authored Jun 25, 2013

These processes don't have image files in a parent snapshot and crtools
should not fail in this case.

https://bugzilla.openvz.org/show_bug.cgi?id=2636

v2: return NULL from mem_snap_init, if a parent image is absent.
Signed-off-by: Andrey Vagin <avagin@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

dd38ae16

zdtm: add a new test case, which create processes in a loop · 538e87ca

Andrey Vagin authored Jun 25, 2013

and this processes live a short period of time. This test case is needed
for testing snapshots.

https://bugzilla.openvz.org/show_bug.cgi?id=2636Signed-off-by: Andrey Vagin <avagin@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

538e87ca

22 Jun, 2013 7 commits

zdtm.sh: check snapshot functionality · ce988e44

Andrey Vagin authored Jun 22, 2013

Only last iteration stops processes, all other iterations make
snapshots. Processes are restored from the last snapshot.

bash -x test/zdtm.sh -s -i 3
Signed-off-by: Andrey Vagin <avagin@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

ce988e44

zdtm.sh: fix indentations · 13a809a9

Andrey Vagin authored Jun 22, 2013

Signed-off-by: Andrey Vagin <avagin@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

13a809a9

mem: don't screen page dump failure · 5bf83eaf

Alexander Kartashov authored Jun 22, 2013

This patch prevents page dump failure screening
by the PARASITE_CMD_MPROTECT_VMAS command success.
Signed-off-by: Alexander Kartashov <alekskartashov@parallels.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

5bf83eaf

arm: bitops: fixed bitops to access more than 32 bits · 1ef1c80f

Alexander Kartashov authored Jun 22, 2013

Signed-off-by: Alexander Kartashov <alekskartashov@parallels.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

1ef1c80f

mm: initialize PageEntry pe to NULL in PagemapEntry · c97911a5

Andrey Vagin authored Jun 22, 2013

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>

c97911a5

ptrace: check that process is stopped by SIGTRAP · 93884266

Andrey Vagin authored Jun 22, 2013

All other states are unexpected.
Signed-off-by: Andrey Vagin <avagin@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

93884266

parasite: unprotect VMA-s if one of DUMPPAGES operations failed · b43c29bb

Andrey Vagin authored Jun 22, 2013

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>

b43c29bb

13 Jun, 2013 1 commit

files: Compact the code by removing per-file dump helpers · d020ebb3

Pavel Emelyanov authored Jun 13, 2013

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>

d020ebb3

14 Jun, 2013 5 commits

parasite: wait a specific pid · 42f77676

Andrey Vagin authored Jun 14, 2013

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>

42f77676

parasite: wait sigreturn only if task is in parasite code · 799182a1

Andrey Vagin authored Jun 14, 2013

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>

799182a1

parasite: close control socket on finalization stage · 557f5935

Andrey Vagin authored Jun 14, 2013

Signed-off-by: Andrey Vagin <avagin@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

557f5935

parasite: handle errors of PTRACE_INTERRUPT · a45f07e2

Andrey Vagin authored Jun 14, 2013

Signed-off-by: Andrey Vagin <avagin@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

a45f07e2

parasite: exit from parasite code if something failed · 842dbf6e

Andrey Vagin authored Jun 14, 2013

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>

842dbf6e