- 19 Feb, 2013 3 commits
-
-
Cyrill Gorcunov authored
In case if here is a fast cpu and make is done in parallel we may have the following error | GEN arch/x86/sys-exec-tbl.c | GEN arch/x86/syscalls.S | CC arch/x86/crtools.o | CC arch/x86/cpu.o | CC arch/x86/syscalls.o | LINK arch/x86/syscalls.built-in.o | In file included from include/restorer.h:12:0, | from arch/x86/crtools.c:19: | include/lock.h: In function ‘futex_set_and_wake’: | include/lock.h:56:2: error: implicit declaration of function ‘sys_futex’ [-Werror=implicit-function-declaration] | cc1: all warnings being treated as errors | make[1]: *** [arch/x86/crtools.o] Error 1 | make: *** [arch/x86] Error 2 This is because crtools.c includes syscall.h but this header is generated by a special script. So use $(deps-after) feature provided by our make engine making sure the header is generated before the deps are started to build. Reported-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Tested-by:
Andrew Vagin <avagin@parallels.com> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Cyrill Gorcunov authored
Since we generate some headers the deps creation may need to be ordered, thus add deps-after variable for this case. Signed-off-by:
-
Cyrill Gorcunov authored
Some of ours headers (such as syscall.h) are clashes with system headers names. So we need to be sure that the headers we include as | #include "something.h" being searched in known place. In particular on some machines it it already produced problems. This btw revealved a problem in cr-dump.c -- we've had #include <parasite.h> there. Fix it. Signed-off-by:
-
- 18 Feb, 2013 17 commits
-
-
Cyrill Gorcunov authored
The FPU data is quite CPU-type oriented, thus move it to asm/fpu.h. Signed-off-by:
-
Cyrill Gorcunov authored
No need to propagate special flags we use for syscalls library to all arch specific objects. Make them syscalls local. Signed-off-by:
-
Cyrill Gorcunov authored
They are really depends on CPU we're running on. Signed-off-by:
-
Cyrill Gorcunov authored
This allows to reuse the code and make rebuild to work correctly. Signed-off-by:
-
Cyrill Gorcunov authored
That's correct syntax. Signed-off-by:
-
Cyrill Gorcunov authored
Signed-off-by:
-
Cyrill Gorcunov authored
I occasionally used assignments instead of appending manpages for generation. Fix it. Signed-off-by:
-
Cyrill Gorcunov authored
This is a bit tricky since the build infrastructure we're switching to is targeted to per-directory targets if simplified Makefiles are used. So I did the following - move rules to compile general sources to Makefile.crtools - main Makefiles now simply calls for scritps/Makefile.build help to generate built-in.o target Also note that we clean up suffixes for implicit rules with .SUFFIXES: this is done on purpose to eliminate any possible side effect in future -- all rules _must_ be described explicitly. Because of implicit rules used in zdtm building procedure I can't use of stict makefile mode and add MAKEFLAGS := -r to the top of the general Makefile. This should be done one day latter (if ever). Signed-off-by:
-
Cyrill Gorcunov authored
We're building project from toplevel directory so no need for absolute paths. Signed-off-by:
-
Cyrill Gorcunov authored
Signed-off-by:
-
Cyrill Gorcunov authored
Because we need util-net.c for both PIE code and a regular code, just move it to pie/ directory and build with -fpie option, this way we can reuse compiled file in main program without additional compilation pass. Signed-off-by:
-
Cyrill Gorcunov authored
Signed-off-by:
-
Cyrill Gorcunov authored
Signed-off-by:
-
Cyrill Gorcunov authored
Signed-off-by:
-
Cyrill Gorcunov authored
This script is engine for simplified Makefiles. I tried to make it somewhat close to what is used to build linux kernel. Signed-off-by:
-
Cyrill Gorcunov authored
This helpers will be used in rules generation for new make infrastructure. At moment only a few helpers here -- to compile *.c and *.S files, and generate *.d,*.s,*.i files. Signed-off-by:
-
Cyrill Gorcunov authored
At moment built-in .SUFFIXES variable is not cleaned which slows down the building procedure. Add a dummy rule thus the make won't try to lookup rules for scripts/Makefile.version (this happens when last resort rule is used). Note that this is rather a stub for future convenience which doesn't affect much current build procedure. Signed-off-by:
-
- 15 Feb, 2013 3 commits
-
-
Cyrill Gorcunov authored
It's already defined as RM := rm -f Signed-off-by:
-
Cyrill Gorcunov authored
- Drop parasite \Space in NORETURN definition - Align likely/unlikely Signed-off-by:
-
Cyrill Gorcunov authored
No need to walk up the directories if we need to include protobuf file. This was always a bad use of ability to walk the filesystem from other headers. Same time we don't need -I$(SRC_DIR)/protobuf/ in general makefile anymore. [xemul: Small fixlet in head Makefile, since patch it out-of-order] Signed-off-by:
-
- 14 Feb, 2013 3 commits
-
-
Andrey Vagin authored
Signed-off-by:
-
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:
-
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:
-
- 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:
-
