- 06 Feb, 2013 2 commits
-
-
Cyrill Gorcunov authored
In case if only "dump" procedure is invoked we left service "link_remap.%d" file(s) on disk. This prevents next "dump" procedure of same test to fail because we hit the situation where same named "link_remap.%d" already exist. So, if test is passed with "-d" option we remove the link remap file at the end. Also with this patch we start passing --link-remap option only for tests which have "unlink_" prefix in name. This applies some limitation on how test could be named, thus maybe we need more flexible solution here. https://bugzilla.openvz.org/show_bug.cgi?id=2489Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
is_blocked should be determined for each line independently. For example: 1: POSIX ADVISORY WRITE 1974 fd:01:392012 0 EOF 2: POSIX ADVISORY WRITE 1974 fd:01:392011 0 EOF 3: POSIX ADVISORY WRITE 1698 fd:01:1196619 0 EOF 4: POSIX ADVISORY WRITE 1698 fd:01:1196620 0 EOF 5: POSIX ADVISORY WRITE 1698 fd:01:1196621 0 EOF 6: POSIX ADVISORY WRITE 1689 fd:01:1429204 0 EOF 7: POSIX ADVISORY WRITE 1690 fd:01:1429202 0 EOF 8: POSIX ADVISORY WRITE 1690 fd:01:1429201 0 0 8: -> POSIX ADVISORY WRITE 1701 fd:01:1429201 0 0 9: POSIX ADVISORY WRITE 1688 fd:01:1429206 0 EOF 10: FLOCK ADVISORY WRITE 1694 fd:01:1196520 0 EOF 11: POSIX ADVISORY WRITE 862 00:0f:12421 0 EOF 12: POSIX ADVISORY WRITE 846 00:0f:12377 0 EOF 13: FLOCK ADVISORY WRITE 821 00:0f:11378 0 EOF 14: POSIX ADVISORY WRITE 459 00:0f:9826 0 EOF 15: POSIX ADVISORY WRITE 511 00:0f:10413 0 EOF Without this patch crtools returns the error: (00.000613) Error (proc_parse.c:1089): Invalid file lock info! It's because is_blocked is set in "true" for the line 8 and it becomes "true" for the line 9. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
-
- 05 Feb, 2013 2 commits
-
-
Cyrill Gorcunov authored
Otherwise if no such files present I'm getting warnings. Signed-off-by:
-
Cyrill Gorcunov authored
In case if there is the VMA which has a huge size (4,8,16 G) then the prealoocated bitmap space won't be enough to track all pages. Thus we call for system help invoking mmap() call with bitmap length needed. We simply assume that if such big VMA present the node has enough memory to provide us space for pages huge bitmap. Reported-by:
Adrian Reber <adrian@lisas.de> Tested-by:
-
- 29 Jan, 2013 3 commits
-
-
Andrey Vagin authored
Signed-off-by:
-
Andrey Vagin authored
Signed-off-by:
-
Alexander Kartashov authored
The section 5.2.1.2 of the AAPCS says that the stack must be 8-byte aligned and this rule is broken when the thread restore_task_with_children() is forked by the function fork_with_pid() since the variable ca and its field stack are likely to be 4-byte aligned. This patch forces 8-byte alingment of the field cr_clone_arg::stack. This made the following tests pass on ARM: * static/shm, * static/ipc_namespace. Particulary the unaligned stack results in incorrect passing of the 64-bit argument to the function snprintf() in the function sysctl_write_u64(). Signed-off-by:
Alexander Kartashov <alekskartashov@parallels.com> Signed-off-by:
-
- 25 Jan, 2013 4 commits
-
-
Pavel Emelyanov authored
This is already so, just move the check for netns case into collect_sockets. It knows it does so. Signed-off-by: -
Pavel Emelyanov authored
Signed-off-by: -
Cyrill Gorcunov authored
Signed-off-by:
-
Cyrill Gorcunov authored
No need for multiple sed calls, better pass all options in once. Signed-off-by:
-
- 24 Jan, 2013 5 commits
-
-
Alexander Kartashov authored
Andrew Vagin pointed out that the static testsuite compilation may run as standalone so the variable ARCH won't be available. This patch introduces the facility test/zdtm/Makefile.inc designated to detect the target machine automatically if the variable ARCH isn't defined. This makefile is supposed to be included into all test subsuite Makefiles. Andrew Vagin also pointed out that machine-dependent tests may be added into non-static subsuites as well so the macro adding machine-specific headers into CPPFLAGS goes to the file test/zdtm/Makefile.inc. Signed-off-by:
-
Cyrill Gorcunov authored
This patch introduce dependencies on .c files generation parsing import keywords in .proto files. In particular, the core.proto file refers to core-x86.proto which means we need to generate .[c|h] files from core-x86.proto earlier than handlig of core.proto. A positive side effect of this patch is that the compilation is independent now of order the proto files are declated in the make file. Instead of currently present manual order PROTO_FILES += core-x86.proto PROTO_FILES += core-arm.proto PROTO_FILES += core.proto one can write it in any arrangement, say PROTO_FILES += core.proto PROTO_FILES += core-arm.proto PROTO_FILES += core-x86.proto and it still be resolved correctly. BUT beware of circular import directives, we can't handle them now. Signed-off-by:
-
Cyrill Gorcunov authored
Some protobuf files do import another files so we should generate dependencies and rebuild the program when needed. For example core.proto imports core-x86.proto and core-arm.proto, thus if any of them are changed the final core.pb-c.c should be regenerated. Signed-off-by:
-
Cyrill Gorcunov authored
It's known to work on x86-64 so make the build procedure a bit faster here. Signed-off-by:
-
Cyrill Gorcunov authored
Actually the former code was being expanding uname_M variable which is not set that early (this was not a problem since we assign it a line below, still it was a dead code which should be fixed). Also update | ifeq ($(findstring arm,$(ARCH)),arm) to simple | ifeq ($(ARCH),arm) Reported-by:
-
- 23 Jan, 2013 5 commits
-
-
Cyrill Gorcunov authored
To unweave arch dependent part from general core.proto file. Signed-off-by:
-
Cyrill Gorcunov authored
We do check for proper architectrue in a makefile, no need for additional guards. After all porting the project for x86-32 is a far more bigger than "just implement proper core for parts covered by #else /* CONFIG_X86_64 */". Signed-off-by:
-
Cyrill Gorcunov authored
Since we don't support x86-32 yet, issue an error early. Signed-off-by:
-
Cyrill Gorcunov authored
Signed-off-by:
-
Cyrill Gorcunov authored
Signed-off-by:
-
- 22 Jan, 2013 7 commits
-
-
Alexander Kartashov authored
Signed-off-by:
-
Alexander Kartashov authored
Signed-off-by:
-
Alexander Kartashov authored
The file lib/atomic.h is the only machine-dependent file in the test suite that breaks compilation of the test static/session01 on ARM. The file is moved into a separate directory and the build system is modified accordingly to prevent this. Signed-off-by:
-
Andrey Vagin authored
All tests which executed in pid namespaces can be executed in parallel. $ make zdtm_ns real 0m54.806s user 0m2.779s sys 0m4.338s $ make -j zdtm_ns real 0m11.587s user 0m3.361s sys 0m5.134s Another feature of this patch is that the subset of parallel tests is executed before the subset of sequential tests. If a first subset are not failed, the second one has a small chance to fail, so a most part of bugs will be found in a few first seconds. Signed-off-by:
-
Andrey Vagin authored
It's required for parallel execution of tests. Signed-off-by:
-
Andrey Vagin authored
Signed-off-by:
-
Andrey Vagin authored
Signed-off-by:
-
- 21 Jan, 2013 7 commits
-
-
Andrey Vagin authored
Signed-off-by:
-
Andrey Vagin authored
Signed-off-by:
-
Andrey Vagin authored
A test should get file names from arguments Signed-off-by:
-
Andrey Vagin authored
Signed-off-by:
-
Andrey Vagin authored
In this case they will be executed in other name-spaces. Signed-off-by:
-
Andrey Vagin authored
This patch can be committed instead of: [PATCH 1/6] cr-dump: move parasite_drain_fds_seized out of dump_task_files [PATCH 2/6] cr-dump: fix dumping file locks in a mount namespace readlink is not required here and a file can be unavailable, if a process is in another mnt namespace Signed-off-by:
-
Cyrill Gorcunov authored
On glibc-2.16 I'm getting | sched_prio00.c: In function ‘main’: | sched_prio00.c:48:3: error: implicit declaration of function ‘setpriority’ [-Werror=implicit-function-declaration] | sched_prio00.c:48:19: error: ‘PRIO_PROCESS’ undeclared (first use in this function) | sched_prio00.c:48:19: note: each undeclared identifier is reported only once for each function it appears in | sched_prio00.c:60:3: error: implicit declaration of function ‘getpriority’ [-Werror=implicit-function-declaration] fix it adding headers needed. Signed-off-by:
-
- 18 Jan, 2013 5 commits
-
-
Pavel Emelyanov authored
When we've read all pstree-items and their ids we can get the desired clone-flags early and avoid all these dances with flag calculations in fork_with_pid and company. Signed-off-by: -
Pavel Emelyanov authored
Signed-off-by: -
Pavel Emelyanov authored
Helpers don't have any IDs on board and thus checks for "is my X shared with parent" will be wrong. Signed-off-by: -
Pavel Emelyanov authored
Signed-off-by: -
Pavel Emelyanov authored
In old images we had no clue whether the root task is to be created in new namespaces. Remember using the --namespaces option for explicitl specification of this fact. Signed-off-by:
-
