- 01 Sep, 2012 10 commits
-
-
Cyrill Gorcunov authored
Otherwise fstatat fails as > > $ ./crtools check > > Error (sk-unix.c: 283): Can't stat socket 12547 (./var/run/sdp): Bad file descriptor Reported-by:
"Muralidhar, Rajeev D" <rajeev.d.muralidhar@intel.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> CC: Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
pthread00 hangs if dprintf is used, and works well if fprintf is used. One more mention of this bug is here: http://sourceware.org/bugzilla/show_bug.cgi?id=12847Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
We can set a directory for log and image files. crtools sets it as a current directory and then creates all files in it. It works before we don't decide to change a mount name space. I suggest to open a log dir and create files for help openat. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
I added this options and I found, that it was a bad idea. Currently crtools prints PID for each message, so --log-pid can help if an error in the restorer code. 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
This tests passed already, so add them in the regular list. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Our general source code depends on headers generated during protobuf library building but if library is already built and *.ch files are removed we might hit a problem where dep files can't be generated. Thus add explicit rule pointing out that library depends on generated *.ch files. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
It's easier to handle things if we know that names in makefiles are never intersected. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
To eliminate code duplication. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> 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>
-
- 28 Aug, 2012 2 commits
-
-
Andrey Vagin authored
Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
Why do we need a new stack? We already have one and it can be used. We need to step a bit for executing a glibc clone() v2: Don't lose a page from a child's stack v3: Remove the defined constant STACK_SIZE Signed-off-by:
Andrey Vagin <avagin@openvz.org> Acked-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 27 Aug, 2012 7 commits
-
-
Cyrill Gorcunov authored
Variables `l' and `le' are short and named too similar which makes them hard to distinguish. Thus rename `l' to `le' (list entry) and `le' to `new_le'. This makes them easy to recognize. Also add braces around list_for_each_entry. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> 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
Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> 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
Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> 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>
-
Andrey Vagin authored
A disabling repair mode drops SO_REUSEADDR. We can set SO_REUSEADDR after disabling repair mode, but a small race window exists in this case. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 21 Aug, 2012 14 commits
-
-
Pavel Emelyanov authored
Stupid misprint Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
For debugging Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
* Don't dumplicate array when sending to parasite * Allocate necessary space in cr-dump for local fds/flags Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Otherwise we can get stuck waiting for more data in vain. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
The value in eax is positive on error. Callers expect it to be negative to report error. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Currently peer is %d, but this is wrong. It should be paired with inode easily. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
fd is shown in %d flags are shown in %#o Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Stanislav Kinsbursky authored
Signed-off-by:
Stanislav Kinsbursky <skinsbursky@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Stanislav Kinsbursky authored
New custom specificator: 'S': output as string (unprintable characters replaced by dots) Signed-off-by:
Stanislav Kinsbursky <skinsbursky@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
So try to check on extra descriptors a few times v2: move save_fds in a loop 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
It will be used for checking tcp connections, external unix sockets, etc Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
* Delete internal buffer, all data are written immediately in a log file * Add timestamp and pid in a message * All processes write log messages in one file Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 20 Aug, 2012 7 commits
-
-
Pavel Emelyanov authored
It's now quite simple. It's better to open-code one. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Just save the ii->ie on stack and do s/ii->ie/ie/ over the fn. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Pavel Emelyanov authored
After previous rework (513dde00) they are no longer needed. 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
Move logic from run_unix_connections to post_open(). I don't fix indents in this patch for easy reviewing. They are fixed in the next patch. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
All sockets are created with SO_REUSEADDR, it's needed for restoring. E.g.: A listen socket is created after a connected socket. Both of them are binded to one port. So SO_REUSEADDR should be restored, when all sockets on a port were created. This code creates a structure for each port of one type of sockets and accounts a number of sockets, which are not restored yet. Sockets have a hook post_open(), in which it waits when all sockets for a defined port would be created and then it will restore SO_REUSEADDR. struct port contains a type (udp, tcp, etc) and a port number. It doesn't contain family or addr, because it's extra loads of logic, which doesn't bring a significant profits. v2: fix according with comments from Pavel Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-