- 04 Dec, 2013 8 commits
-
-
Cyrill Gorcunov authored
In case if ghost file is on BTRFS subvolume we should not fail but test if there is a match. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Because socket migh lay on btrfs volume (thus the device number reported by diag module won't be the same as obtained from stat(2)) we need to do an additional test and try to resolve device number with help of btrfs engine. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
This helper serves to hide fs specifics (in particular btrfs) thus the caller won't need the details. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
This helper will cause BTRFS engine to collect all subvolumes. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
It is been found that BTRFS uses per subvolume device numbering scheme which causes some of our tests to fail if they are laying on this filesystem. Here we introduce minimal engine which parses subvolumes present on mount point. The interface is two functions - btrfs_parse_mountinfo: this one parses subvolume and caches the results - is_btrfs_subvol: test if device id is laying on subvolume In the code we assume that cpu we're running on is little endian, thus if one day we run on big ending machine the code must be updated accordingly. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
It will hold auxiliary data associated with mount point. We will use it for btrfs handling but in future it can be extended. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
We will need it for btrfs handling. Also print out the FS type for easier debug | (00.003545) type unsupported (cgroup) source cgroup 1c / @ /sys/fs/cgroup/blkio flags 30000e options blkio, | (00.003558) type unsupported (cgroup) source cgroup 1d / @ /sys/fs/cgroup/perf_event flags 30000e options perf_event, | (00.003571) type unsupported (cgroup) source cgroup 1e / @ /sys/fs/cgroup/hugetlb flags 30000e options hugetlb, | (00.003584) type unsupported (ext4) source /dev/sda2 800002 / @ / flags 300000 options data=ordered, | (00.003670) type tmpfs (tmpfs) source tmpfs 20 / @ /tmp flags 100000 options | (00.003696) type unsupported (mqueue) source mqueue d / @ /dev/mqueue flags 300000 options Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
@path is always nil here, we actually need @remap->path Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 03 Dec, 2013 4 commits
-
-
Cyrill Gorcunov authored
For debug purpose. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
We allocate mount_info with xzalloc, no need for additional NULL assignment. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
If we fail in xmalloc the function occasionally return 0 meaning that everything is fine. Don't do that, wait until routine complete. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
First thing people typically do with criu is run something in shell and try to dump it. This is typicall pitfall, as in that case we face external sid and tty problem. Print a message on stdout about this and advice wiki page to read. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 02 Dec, 2013 2 commits
-
-
Ruslan Kuprieiev authored
Currently we have a bug: if no leave_running is set in request, service won't send dump response. We must not send response only if it was a self-dump request and no leave_running option was set. Signed-off-by:
Ruslan Kuprieiev <kupruser@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>
-
- 29 Nov, 2013 2 commits
-
-
Cyrill Gorcunov authored
We will need it for additional handling once parsing of mount entry is complete (in particular btrfs requires additional processing to figure out subvolumes names). Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
We will need it for btrfs subvolumes handling. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 28 Nov, 2013 3 commits
-
-
Cyrill Gorcunov authored
It's designed specificly for that. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
The error message is rather confusing people. In worst case (if it happened that we need this uncollected socket), criu will print out real error message later. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Useful for bug hunting. | (00.005209) unix: Dropping path /mnt/disk1/new_subvol/criu/test/zdtm/live/static/sockets00.test | for unlinked bound sk 0x26.0x1d4e real 0x23.0x1d4e Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 25 Nov, 2013 1 commit
-
-
Pavel Emelyanov authored
The next version is ready. It looks like it's time to make it be the first major release, so here it is -- the v1.0. It's as functional as v0.8 but with a lot of bug fixes and improvements. And it's still :) compatible wrt images and cli options with older criu-s. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 22 Nov, 2013 3 commits
-
-
Andrey Vagin authored
Currently only addresses are compared. It's obviously not enough. * First of all the parent vma must be private. * Both vma-s must have the identical set of MAP_GROWSDOWN and MAP_FILES flags. * Both vma-s must be linked to the same file. https://bugzilla.openvz.org/show_bug.cgi?id=2824Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
shmid contains a file id for file mappings. It's required to determine, which VMA-s are cowed. The parent maps a VMA and saves premmaped address. Then child trys to determing, which VMA-s must be inhereted from parent, for that it compares addresses, flags and file id. We don't want to transfer vma_area-s in restorer, so when a VMA entry is copied in restorer memory, the premmaped address is save in shmid. 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>
-
- 21 Nov, 2013 3 commits
-
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Kenrel API is stable enough not to ignore it. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Kir Kolyshkin authored
Both fopen_proc() and opendir_proc() are calling open_proc(). If open_proc() fails, it prints an error message. Before this patch, a second error message was printed due to missing brackets. This second message is a bit more specific (it tells the exact file/dir we failed to open) but it is redundant, because more generic error was already printed by open_proc(). It is also can be misleading because for the second message we reuse the same errno while we should not. So, let's remove this second error message print by using brackets. Alternatively, we could leave this as is (just fixing indentation) and let two errors be printed -- there is nothing wrong with that, but I think in this case less messages is better. This is a fix to commit 5661d806. Signed-off-by:
Kir Kolyshkin <kir@openvz.org> Acked-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 20 Nov, 2013 3 commits
-
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
opts are criu-wide available. 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>
-
- 19 Nov, 2013 1 commit
-
-
Ruslan Kuprieiev authored
After set of patches which add --work-dir option we have an issue: logs are created relatively to current dir and not relatively to images dir(which is work dir, when --work-dir is not given). To solve this lets init log after chdir(work_dir) Signed-off-by:
Ruslan Kuprieiev <kupruser@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 18 Nov, 2013 1 commit
-
-
Ruslan Kuprieiev authored
After fixes with -W option we've changed the cwd at the time parent images are opened. Use the -at syscall to proerly access ones. [ Cleanup and comment from xemul@ ] Signed-off-by:
Ruslan Kuprieiev <kupruser@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 15 Nov, 2013 9 commits
-
-
Ruslan Kuprieiev authored
Stats must be in work dir, as logfiles and pidfiles. Signed-off-by:
Ruslan Kuprieiev <kupruser@gmail.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
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>
-
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>
-
Cyrill Gorcunov authored
We really have a mess of extern/non-extern declaration of functions in our headers. Always use extern for unification purpose. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
The file name was changed but ifdef/endif hadn't been updated. Fix it. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-