- 14 Feb, 2013 2 commits
-
-
Andrey Vagin authored
If a TCP socket will get live-migrated from one box to another the timestamps (which are typically ON) will get screwed up -- the new kernel will generate TS values that has nothing to do with what they were on dump. The solution is to yet again fix the kernel and put a "timestamp offset" on a socket. v2: don't fail if TCP_TIMESTAMP is unsupported Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
It's been reported that we do not test if the tracee is 32bit task running kn 64bit kernel. This patch adds such test. https://bugzilla.openvz.org/show_bug.cgi?id=2505Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
-
- 11 Feb, 2013 2 commits
-
-
Cyrill Gorcunov authored
This allows us to have a unique place where version lives and what is more important other subsystems (such as docs generation) may reuse version definitions. At moment EXTRA (which corresponds kernels -rc tag) and NAME is not yet used, but I desided to put them in place for future needs. Signed-off-by:
-
Cyrill Gorcunov authored
But rather exit gracefully. Signed-off-by:
-
- 07 Feb, 2013 3 commits
-
-
Pavel Emelyanov authored
Make all locks code be in one place. Signed-off-by: -
Pavel Emelyanov authored
Signed-off-by: -
Pavel Emelyanov authored
It's a file dumping fn, it should be there. Signed-off-by:
-
- 06 Feb, 2013 7 commits
-
-
Cyrill Gorcunov authored
Otherwise I'm getting warnings (especially in zdtm) when we are trying to remove non-generated .d files. Signed-off-by:
-
Cyrill Gorcunov authored
We don't use it at all. Signed-off-by:
-
Cyrill Gorcunov authored
It's good to know which family we've met. Signed-off-by:
-
Cyrill Gorcunov authored
To be able to test it one need indeed enough physical memory on the node (at least 16G). Thus I don't include this test for regular passes. Inspired by commit 40dcaf88Signed-off-by:
-
Cyrill Gorcunov authored
Otherwise pie may not be re-built if headers are changed. https://bugzilla.openvz.org/show_bug.cgi?id=2465Reported-by:
-
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:
-
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:
-
- 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:
-
