- 13 Jan, 2012 16 commits
-
-
Cyrill Gorcunov authored
No func changes. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Pavel Emelyanov authored
We will later need other fields of this file, so let's parse it right now in a way, that allows to easily get new fields. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Pavel Emelyanov authored
Start with /proc/pid/maps parser Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Cyrill Gorcunov authored
Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Cyrill Gorcunov authored
Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Cyrill Gorcunov authored
Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Cyrill Gorcunov authored
* @xemul: crtools: Collect dumping fd parameters into one place crtools: Toss dump_one_fd args around crtools: Rename fd to lfd in dump_one_fd crtools: Sanitize pstree construction crtools: Remove unused printk_registers and co crtools: Deduplicate file info showing code crtools: Merge pstree collecting into showing crtools: Remove unused and wrong arrays from pstree image crtools: Remove lseeks after prep_cr_ calls crtools: Cleanup collect_pstree in cr-show Conflicts: cr-dump.c include/sockets.h sockets.c The conflicts are mostly because of commit 995ef5ecSigned-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Cyrill Gorcunov authored
No need to track in a weird way. It's small structure. Moreover, the rest of parasite code doesn't count for space allocated _after_ the main control block. We simply had happened to not hit any problems before. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Cyrill Gorcunov authored
This allows us to get rid of open-coded "/proc/pid/X". Based-on-patch-from: Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Acked-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Acked-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
It basically reverts e189efc1 which was overdone one. Reported-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Acked-by:
Pavel Emelyanov <xemul@parallels.com>
-
Stanislav Kinsbursky authored
NLM_F_DUMP is already defined as (NLM_F_ROOT|NLM_F_MATCH) Signed-off-by:
Stanislav Kinsbursky <skinsbursky@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Stanislav Kinsbursky authored
Signed-off-by:
Stanislav Kinsbursky <skinsbursky@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Andrey Vagin authored
Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Cyrill Gorcunov authored
Sockets are special and we reuse fds in accept jobs. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Acked-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
[avagin@: Drop redundant fcntl call, and err variable] Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Acked-by:
Pavel Emelyanov <xemul@parallels.com>
-
- 12 Jan, 2012 24 commits
-
-
Cyrill Gorcunov authored
* @xemul-mails: crtools: Collect dumping fd parameters into one place crtools: Toss dump_one_fd args around crtools: Rename fd to lfd in dump_one_fd crtools: Sanitize pstree construction crtools: Remove unused printk_registers and co crtools: Deduplicate file info showing code crtools: Merge pstree collecting into showing crtools: Remove unused and wrong arrays from pstree image crtools: Remove lseeks after prep_cr_ calls crtools: Cleanup collect_pstree in cr-show Conflicts: cr-show.c Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Pavel Emelyanov authored
Introduce the fd_parms structure, that contains all we need to describe open fd during dump -- its name (int), pos, flags and id. The read_fd_parms routing fills one once and passes along the stack. This reduces the amount of arguments significantly. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Pavel Emelyanov authored
Make the dump_one_fd fn accept local fd and integer target fd values instead of target fd dir and target fd string name. This saves couple of atoi-s (done once) and makes the next patch simpler. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Pavel Emelyanov authored
This variable is actually a local fd, so reflect this in its name. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Pavel Emelyanov authored
Rename find_pstree_entry into add_<one> (since it doesn't find it) and move list adding into it. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Pavel Emelyanov authored
Print common info once, then append path if required. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Pavel Emelyanov authored
1. There's no need in collecting children list in shower 2. There's no need in reading file twice -- we can collect and show everything in one go. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Pavel Emelyanov authored
Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Pavel Emelyanov authored
The cr preparation functions to read magic from files and thus this lseek is not required. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Pavel Emelyanov authored
Rename this into read_pstree and remove unneeded 2nd argument. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Cyrill Gorcunov authored
It was being done intentionally to be able to call close_cr_fdset several times in a row, bring this ability back. Otherwise I'm getting glibc complains about attemt to free already freed memory. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Cyrill Gorcunov authored
It's a rudiment from Elf-time code. Not needed anymore. Reported-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Cyrill Gorcunov authored
In case if relative bind path found we skip such socket but forget to free previously allocated memory. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Andrey Vagin authored
We collect all unix sockets in the current net namespace, but a target process uses a few of them, so we may skip unsupported sockets and if it is used by a target process, lookup_socket returns error. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Acked-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Andrey Vagin authored
It's workaround for RHEL6, it contains old headers. In file included from libnetlink.c:2: /usr/include/linux/netlink.h:34: error: expected specifier-qualifier-list before ‘sa_family_t’ Signed-off-by:
Andrey Vagin <avagin@openvz.org> Acked-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Andrey Vagin authored
open_fdinfo calls move_img_fd, so other functions should not care about it Signed-off-by:
Andrey Vagin <avagin@openvz.org> Acked-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Andrey Vagin authored
Signed-off-by:
Andrey Vagin <avagin@openvz.org> Acked-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Andrey Vagin authored
Signed-off-by:
Andrey Vagin <avagin@openvz.org> Acked-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Andrey Vagin authored
Signed-off-by:
Andrey Vagin <avagin@openvz.org> Acked-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Pavel Emelyanov authored
The same as previous patch -- no need in two separate calls. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Pavel Emelyanov authored
They always go in pairs so there's no need in two calls. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Pavel Emelyanov authored
Path is not needed there -- we can call the get_image_path() in prep_cr_fdset_ routines and in parasite-syscall.c when required. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-
Pavel Emelyanov authored
Remove unneeded path argument (we do know the file data is read from) and actually bogus show_header boolean. Also introduce two helpers for showing header and tail. Signed-off-by:
Pavel Emelyanov <xemul@parallels.com> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org>
-