- 20 Nov, 2012 9 commits
-
-
Andrey Vagin authored
read_vmas will be called for zombies Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
read_vmas will be called bedore forking children to restore copy-on-write memory. v2: don't open an image one more time Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
Now this macros returns true only for anon | priv vmas. It will be expanded to file | priv in a follow patch. v2: check flags together. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
This code is borrowed from the Linux kernel sources. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
It will be used in restorer.c Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
A socket can get fin and a state will be changed on CLOSE_WAIT, which is not supported yet. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
We have a window between getting info about tcp connections and blocking them. #2419 v2: clean upV v3: don't update lengthes of queues for listen sockets, they don't used. v4: check that a state of a tcp connection is ESTABLISHED or CLOSE v5: * don't check state, because it can be changed only on TCP_CLOSE. In this case it will be changed again after restoring. * refresh a socket after enabling the repair mode Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 15 Nov, 2012 1 commit
-
-
Cyrill Gorcunov authored
- use pthread_sigmask instead of sigprocmask - show_sigset should take the size of dereferenced variable - zeroify sigsets before use, since libc has own definition for sigset which doesn't match one the kernel uses http://bugzilla.openvz.org/show_bug.cgi?id=2437Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Acked-by:
Andrew Vagin <avagin@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 13 Nov, 2012 3 commits
-
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Same as previous patch -- easier to read and possible to use tags. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
This makes code more readable, saves one ptr on stack and lets us jump into restorer code using tags. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 12 Nov, 2012 17 commits
-
-
Pavel Emelyanov authored
Not to spoil screen with false warnings. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
To test threads blocked signals restoration. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
To unify the code for both thread leader and regular threads we move blocked signals for thread leader into threads argument area and use restore_thread_common() helper. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
And add missing new line in message. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
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
I've moved dump_thread helper a bit lower in file since I've to call for find_thread_state helper. After all this groups all thread related functions in one slab. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
This patch starts using parasite_init_threads_seized and parasite_fini_threads_seized helpers to save per-thread data in parasite and remove it on cure procedure. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Just to be consistent with types we're using. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
We will need to extend the structure to keep not only tid/tid_addr but blocked signals as well, thus rename it to more generic parasite_dump_thread. The command PARASITE_CMD_DUMP_TID_ADDR renamed to PARASITE_CMD_DUMP_THREAD for the same sake. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
We will use them from crtools code to save and restore blocked signals mask of threads. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
- the init() now calls for init_thread() to save blocked signals mask in per-thread data - the fini() now calls for fini_thread() to restore the blocked signals, note: we could use thread_leader directly but i think calling for fini_thread() is more generic - dump_misc() now uses per-thread data Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
These helpers will be needed to save a blocked signals mask for dumpee threads. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
The per-thread information requires own space in parasite data. In particular we will keep the blocked signals bound to thread pids. For this sake the caller need to provide the parasite how many threads will be used to calculate space. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
It will hold the blocked signals for threads. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Otherwise we might get nil dereference in sigreturn restore. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
We will need to extend non-zombie tests. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Don't forget to test thread leader as well. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 11 Nov, 2012 2 commits
-
-
Cyrill Gorcunov authored
No func change. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Better to stick with the convention we're trying to follow in general -- negative codes for error. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 06 Nov, 2012 8 commits
-
-
Pavel Emelyanov authored
Linux kernel emulates anon-shared mappings by mapping internal tmpfs file in. We try to detect this by checking that the file under map is such, but do it with error -- major == 0 check is wrong, as regular tmpfs file can be such as well as btrfs or ecryptfs can screw things up. The only working way of doing this is to get the dev_t of this internal tmpfs mount. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Fix a few places where positive value returned as an error sign. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Fix a typo. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
It's a predicate function, boolean value is more appropriate here. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Vladimir Davydov authored
During live migration timer overrun can jump forward too. Signed-off-by:
Vladimir Davydov <vdavydov@parallels.com> Acked-by:
Stanislav Kinsbursky <skinsbursky@openvz.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
This sneaked during parasite error handling code rework. When parasite returned negative error code we should not invert it. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-