Commits · 8796b18eba6a0aa040ea687d3fab471ec0f06b9c · zhul / criu

13 Nov, 2015 6 commits

files: linkat_hard() should return errno of linkat · 8796b18e

Andrew Vagin authored Nov 13, 2015

errno is used in open_path()
Signed-off-by: Andrew Vagin <avagin@virtuozzo.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

8796b18e

zdtm: check an unlinked file when a file with the save name exists · 0b4fecca
Andrew Vagin authored Nov 13, 2015
```
Signed-off-by: Andrew Vagin <avagin@virtuozzo.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
0b4fecca

zdtm: add a new test to check tmpfs in a non-root mntns (v2) · 138d2631

Andrew Vagin authored Nov 13, 2015

v2: add a file mapping from a test tmpfs mount
Signed-off-by: Andrew Vagin <avagin@virtuozzo.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

138d2631

mount: use correct mount namespace in open_mountpoint · 947dcc73

Andrey Ryabinin authored Nov 11, 2015

open_mountpoint() have to operate in target's mount namespace,
which may differ from root's namespace - root_item->pid.real.

So obtain the correct mount namespace from mount_info and use
it in switch_ns().
Signed-off-by: Andrey Ryabinin <aryabinin@virtuozzo.com>
Acked-by: Andrew Vagin <avagin@virtuozzo.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

947dcc73

zdtm.py: Fix race in root construction · bada2929

Pavel Emelyanov authored Nov 12, 2015

If two ns_flavor.init() are called in parallel then all the os.mkdir()
calls would end up with EEXISTS exception.
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

bada2929

zdtm.py: Do not run tests marked with excl in parallel (v2) · a9632060

Pavel Emelyanov authored Nov 12, 2015

There are tests (e.g. on locks) that can conflict with each other, so
wait for all the other jobs to finish before running such.

v2:
  Wait for the excl test to finish before launching more of them.
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

a9632060

12 Nov, 2015 27 commits

page-read: Fix compilation on arm (2) · 65408e92
Pavel Emelyanov authored Nov 12, 2015
```
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
65408e92
page-read: Fix compilation on arm · efb31988
Pavel Emelyanov authored Nov 12, 2015
```
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
efb31988

jenkins: fix path to report · a9e5d2b4

Andrey Vagin authored Nov 12, 2015

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

a9e5d2b4

jenkin: mark all scripts as executalbe · 25eb3d6d

Andrew Vagin authored Nov 12, 2015

Signed-off-by: Andrew Vagin <avagin@virtuozzo.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

25eb3d6d

jenkins: Do not test cgroup02 with --norst · a6cb3d78

Pavel Emelyanov authored Nov 12, 2015

This test expects that it _will_ be C/R-ed into new hierarchy
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

a6cb3d78

jenkins: Make criu-snap test use zdtm.py · 12715226
Pavel Emelyanov authored Nov 12, 2015
```
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
12715226
jenkins: Make criu-dump test use zdtm.py · cc378b90
Pavel Emelyanov authored Nov 12, 2015
```
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
cc378b90

net: Dont print error in rule save · dc00fea3

Pavel Emelyanov authored Nov 12, 2015

This thing is new and can be absent in ip tool, which is OK
and is handled by net.c code itself.
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

dc00fea3

net: Add flags to run_ip_tool · 336b9909
Pavel Emelyanov authored Nov 12, 2015
```
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
336b9909
util: Add flags to cr_system · 18d91708
Pavel Emelyanov authored Nov 12, 2015
```
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
18d91708

zdtm: Do not create dangling executables · e5a91658

Pavel Emelyanov authored Nov 12, 2015

Otherwise they are left after test and make zdtm.py list thing
these files are tests too.
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

e5a91658

zdtm: Make /dev/net/tun optional device in test ns · 9a366728
Pavel Emelyanov authored Nov 12, 2015
```
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
9a366728

zdtm: don't execute the bridge test from zdtm.sh · 79be3677

Andrew Vagin authored Nov 12, 2015

zdtm.sh doesn't suppport dependencies. This test uses the ip tool.
Signed-off-by: Andrew Vagin <avagin@virtuozzo.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

79be3677

jenkins: Move pre-dump test on .py launcher · 37ec9676
Pavel Emelyanov authored Nov 12, 2015
```
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
37ec9676

page-read: Properly handle overlapping pagemaps in stack · 7d72537d

Pavel Emelyanov authored Nov 12, 2015

If the pagemap we read at some point covers two (or more) pagemaps
in parent images, we cannot just call the parent page read for all
those pages. Instead, we must call parent page read with shorter
requests so that it handles its own pagemaps properly.
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

7d72537d

zdtm: execute the bridge test in a new netns only · 87c0376b

Andrew Vagin authored Nov 12, 2015

Signed-off-by: Andrew Vagin <avagin@virtuozzo.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

87c0376b

jenkins: Exclude maps04 from -sibling test · 446a8ba8

Pavel Emelyanov authored Nov 12, 2015

This guy runs on tmpfs for dumps and maps04 is too heavy for it
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

446a8ba8

restore: Use min_t helper for type casting · fa0587ed

Cyrill Gorcunov authored Nov 12, 2015

On arm

 | CC       crtools.o
 | In file included from arch/arm/include/asm/bitops.h:4:0,
 |                  from arch/arm/include/asm/types.h:9,
 |                  from include/proc_parse.h:5,
 |                  from include/ptrace.h:8,
 |                  from cr-restore.c:27:
 | cr-restore.c: In function 'restore_priv_vma_content':
 | include/compiler.h:60:17: error: comparison of distinct pointer types lacks a cast [-Werror]
 |   (void) (&_min1 == &_min2);  \
 |
Reported-by: Pavel Emelyanov <xemul@parallels.com>
Signed-off-by: Cyrill Gorcunov <gorcunov@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

fa0587ed

compiler: Grab min_t, max_t from the kernel · ee2409ec

Cyrill Gorcunov authored Nov 12, 2015

Signed-off-by: Cyrill Gorcunov <gorcunov@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

ee2409ec

test: use the ip tool instead of ifconfig and brctl · 23a2472c

Andrew Vagin authored Nov 12, 2015

I'm too lazy to install new tools on all test hosts.
Signed-off-by: Andrew Vagin <avagin@virtuozzo.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

23a2472c

restore: Use bitmap_set helper · c63a42ac

Cyrill Gorcunov authored Nov 09, 2015

Signed-off-by: Cyrill Gorcunov <gorcunov@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

c63a42ac

bitmap -- Add few helpers for bits manipulations · ba475b8d

Cyrill Gorcunov authored Nov 09, 2015

Grabbed from kernel. Probably worth to gather
all bits manipulators here in future.
Signed-off-by: Cyrill Gorcunov <gorcunov@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

ba475b8d

restore: Restore as many pages at once as possible · 03b217c0

Pavel Emelyanov authored Nov 09, 2015

When the VMA being restored is not COW-ed we read pages from images
one-by-one which results in suboptimal pages.img access. Fix this
by reading as many pages from iamge at once as possible withing the
active pagemap and VMA.
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

03b217c0

page-read: Teach page-read to read multiple pages at once · 780d6994

Pavel Emelyanov authored Nov 09, 2015

This is preparatory patch, the problem to solve is described in
the next one.
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

780d6994

gitignore: ignore a bunch of other generated stuff · e41f4dbc

Tycho Andersen authored Nov 11, 2015

Some test binaries weren't ignored, and some of the new zdtm.py files.
Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

e41f4dbc

tests: add a test for c/r of an empty bridge · 34f33bd7

Tycho Andersen authored Nov 11, 2015

v2: * add a zdtm.py .desc file
    * only look to make sure inet addresses match (in particular, don't
      match the state)
Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

34f33bd7

net: allow c/r of empty bridges in the container · 8a95be06

Tycho Andersen authored Nov 11, 2015

Implementing c/r of bridges with slaves shouldn't be too hard (viz. the
comment), but this is all I need to for right now.

v2: remove extra debug statement
v3: * remember to close fd in dump_bridge
    * use "known" buffer length and snprintf for spath in dump_bridge
    * change brace style
Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

8a95be06

11 Nov, 2015 6 commits

collect: Print info about collection finish · b67bde83
Pavel Emelyanov authored Nov 11, 2015
```
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
b67bde83

add seccomp syscall · a0470e9b

Tycho Andersen authored Nov 04, 2015

We'll use this when restoring eBPF programs in FILTER mode.
Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

a0470e9b

rst: unmap restore memory after seccomp restore · 934c3125

Tycho Andersen authored Nov 04, 2015

In order to restore seccomp filters, we need to have access to dynamically
allocated memory from the restorer blob, so we should unmap this memory
afterwards. In order to do this, we need to suspend seccomp earlier, right
after we attach to the tasks instead of just before we do the unmap of the
restorer blob itself.
Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

934c3125

unix: turn down unlink warning · 5577c2e8

Tycho Andersen authored Nov 10, 2015

This isn't really an, so let's call it a warning.
Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

5577c2e8

jenkins: add script to create a test container · 1c179541

Andrew Vagin authored Nov 10, 2015

We want to execute tests in an isolated environment
Signed-off-by: Andrew Vagin <avagin@openvz.org>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

1c179541

jenkins: Move regular test on .py · 20088c4f
Pavel Emelyanov authored Nov 11, 2015
```
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
```
20088c4f

10 Nov, 2015 1 commit

Revert "rst: Re-use opened fd when restoring private mappings" · 6f0681c1

Pavel Emelyanov authored Nov 10, 2015

This reverts commit 73cb87f9.

Two reasons: individual VMA-s may require different open flags
and ghost and link-remap files should be properly unlinked at
the end of open_path().

Need some more intelligent solution to this.

6f0681c1