diff options
author | Tom Zanussi <tom.zanussi@intel.com> | 2011-09-03 23:18:32 -0500 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2011-09-05 20:25:45 +0100 |
commit | 6ca3b3107da861491279dc6c99fc3922a297f771 (patch) | |
tree | 4d62a68f18602bdc6c2d3eefed022df9fab17aa4 | |
parent | b6aa7037486a9be7272f01ed1b71448681ed8fdc (diff) | |
download | poky-6ca3b3107da861491279dc6c99fc3922a297f771.tar.gz |
systemtap: upgrade to 1.6
Also enable for arm, since systemtap now works on arm and remove the
gcc-4.6 compile fix patch since the problems it addresses have been
fixed upstream.
(From OE-Core rev: 09fa8e139b0c06f961927ac192d644bebdf31abf)
Signed-off-by: Tom Zanussi <tom.zanussi@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | meta/recipes-kernel/systemtap/systemtap/fix_for_compilation_with_gcc-4.6.0.patch | 119 | ||||
-rw-r--r-- | meta/recipes-kernel/systemtap/systemtap_git.bb | 15 |
2 files changed, 7 insertions, 127 deletions
diff --git a/meta/recipes-kernel/systemtap/systemtap/fix_for_compilation_with_gcc-4.6.0.patch b/meta/recipes-kernel/systemtap/systemtap/fix_for_compilation_with_gcc-4.6.0.patch deleted file mode 100644 index a32217586f..0000000000 --- a/meta/recipes-kernel/systemtap/systemtap/fix_for_compilation_with_gcc-4.6.0.patch +++ /dev/null | |||
@@ -1,119 +0,0 @@ | |||
1 | Upstream-Status: Pending | ||
2 | |||
3 | Fix following compiler errors with gcc 4.6.0: | ||
4 | Nitin A Kamble <nitin.a.kamble@intel.com> 2011/05/09 | ||
5 | |||
6 | | tapsets.cxx: In member function 'void sdt_query::handle_probe_entry()': | ||
7 | | tapsets.cxx:5295:8: error: variable 'kprobe_found' set but not used [-Werror=unused-but-set-variable] | ||
8 | | tapsets.cxx: In member function 'virtual void hwbkpt_builder::build(systemtap_session&, probe*, probe_point*, const literal_map_t&, std::vector<derived_probe*>&)': | ||
9 | | tapsets.cxx:7655:18: error: variable 'has_symbol_str' set but not used [-Werror=unused-but-set-variable] | ||
10 | | cc1plus: all warnings being treated as errors | ||
11 | | | ||
12 | | make[2]: *** [stap-tapsets.o] Error 1 | ||
13 | | translate.cxx: In member function 'virtual void c_unparser::visit_print_format(print_format*)': | ||
14 | | translate.cxx:4431:6: error: variable 'width_ix' set but not used [-Werror=unused-but-set-variable] | ||
15 | | cc1plus: all warnings being treated as errors | ||
16 | | | ||
17 | | make[2]: *** [stap-translate.o] Error 1 | ||
18 | | common.c: In function 'fatal_handler': | ||
19 | | common.c:397:13: error: variable 'rc' set but not used [-Werror=unused-but-set-variable] | ||
20 | | cc1: all warnings being treated as errors | ||
21 | | | ||
22 | | make[3]: *** [common.o] Error 1 | ||
23 | | make[3]: *** Waiting for unfinished jobs.... | ||
24 | | common.c: In function 'fatal_handler': | ||
25 | | common.c:397:13: error: variable 'rc' set but not used [-Werror=unused-but-set-variable] | ||
26 | | cc1: all warnings being treated as errors | ||
27 | | | ||
28 | | mainloop.c: In function 'chld_proc': | ||
29 | | mainloop.c:56:11: error: variable 'rc' set but not used [-Werror=unused-but-set-variable] | ||
30 | | mainloop.c: In function 'stp_main_loop': | ||
31 | | mainloop.c:623:17: error: variable 'rc' set but not used [-Werror=unused-but-set-variable] | ||
32 | | cc1: all warnings being treated as errors | ||
33 | | | ||
34 | |||
35 | Index: git/tapsets.cxx | ||
36 | =================================================================== | ||
37 | --- git.orig/tapsets.cxx | ||
38 | +++ git/tapsets.cxx | ||
39 | @@ -5292,7 +5292,6 @@ sdt_query::handle_probe_entry() | ||
40 | probe *new_base = convert_location(); | ||
41 | probe_point *new_location = new_base->locations[0]; | ||
42 | |||
43 | - bool kprobe_found = false; | ||
44 | bool need_debug_info = false; | ||
45 | |||
46 | Dwarf_Addr bias; | ||
47 | @@ -5302,7 +5301,6 @@ sdt_query::handle_probe_entry() | ||
48 | if (have_kprobe()) | ||
49 | { | ||
50 | convert_probe(new_base); | ||
51 | - kprobe_found = true; | ||
52 | // Expand the local variables in the probe body | ||
53 | sdt_kprobe_var_expanding_visitor svv (module_val, | ||
54 | provider_name, | ||
55 | @@ -7652,7 +7650,7 @@ hwbkpt_builder::build(systemtap_session | ||
56 | { | ||
57 | string symbol_str_val; | ||
58 | int64_t hwbkpt_address, len; | ||
59 | - bool has_addr, has_symbol_str, has_write, has_rw, has_len; | ||
60 | + bool has_addr, has_write, has_rw, has_len; | ||
61 | |||
62 | if (! (sess.kernel_config["CONFIG_PERF_EVENTS"] == string("y"))) | ||
63 | throw semantic_error ("CONFIG_PERF_EVENTS not available on this kernel", | ||
64 | @@ -7662,7 +7660,6 @@ hwbkpt_builder::build(systemtap_session | ||
65 | location->components[0]->tok); | ||
66 | |||
67 | has_addr = get_param (parameters, TOK_HWBKPT, hwbkpt_address); | ||
68 | - has_symbol_str = get_param (parameters, TOK_HWBKPT, symbol_str_val); | ||
69 | has_len = get_param (parameters, TOK_LENGTH, len); | ||
70 | has_write = (parameters.find(TOK_HWBKPT_WRITE) != parameters.end()); | ||
71 | has_rw = (parameters.find(TOK_HWBKPT_RW) != parameters.end()); | ||
72 | Index: git/translate.cxx | ||
73 | =================================================================== | ||
74 | --- git.orig/translate.cxx | ||
75 | +++ git/translate.cxx | ||
76 | @@ -4428,7 +4428,7 @@ c_unparser::visit_print_format (print_fo | ||
77 | continue; | ||
78 | |||
79 | /* Take note of the width and precision arguments, if any. */ | ||
80 | - int width_ix = -1, prec_ix= -1; | ||
81 | + int __attribute((__unused__)) width_ix = -1, prec_ix= -1; | ||
82 | if (components[i].widthtype == print_format::width_dynamic) | ||
83 | width_ix = arg_ix++; | ||
84 | if (components[i].prectype == print_format::prec_dynamic) | ||
85 | Index: git/runtime/staprun/common.c | ||
86 | =================================================================== | ||
87 | --- git.orig/runtime/staprun/common.c | ||
88 | +++ git/runtime/staprun/common.c | ||
89 | @@ -394,7 +394,7 @@ void parse_modpath(const char *inpath) | ||
90 | #define ERR_MSG "\nUNEXPECTED FATAL ERROR in staprun. Please file a bug report.\n" | ||
91 | static void fatal_handler (int signum) | ||
92 | { | ||
93 | - int rc; | ||
94 | + int __attribute__((__unused__)) rc; | ||
95 | char *str = strsignal(signum); | ||
96 | rc = write (STDERR_FILENO, ERR_MSG, sizeof(ERR_MSG)); | ||
97 | rc = write (STDERR_FILENO, str, strlen(str)); | ||
98 | Index: git/runtime/staprun/mainloop.c | ||
99 | =================================================================== | ||
100 | --- git.orig/runtime/staprun/mainloop.c | ||
101 | +++ git/runtime/staprun/mainloop.c | ||
102 | @@ -53,7 +53,7 @@ static void *signal_thread(void *arg) | ||
103 | |||
104 | static void chld_proc(int signum) | ||
105 | { | ||
106 | - int32_t rc, btype = STP_EXIT; | ||
107 | + int32_t __attribute__((__unused__)) rc, btype = STP_EXIT; | ||
108 | dbug(2, "chld_proc %d (%s)\n", signum, strsignal(signum)); | ||
109 | pid_t pid = waitpid(-1, NULL, WNOHANG); | ||
110 | if (pid != target_pid) | ||
111 | @@ -620,7 +620,7 @@ int stp_main_loop(void) | ||
112 | { | ||
113 | /* module asks us to start exiting, so send STP_EXIT */ | ||
114 | dbug(2, "got STP_REQUEST_EXIT\n"); | ||
115 | - int32_t rc, btype = STP_EXIT; | ||
116 | + int32_t __attribute__((__unused__))rc, btype = STP_EXIT; | ||
117 | rc = write(control_channel, &btype, sizeof(btype)); | ||
118 | break; | ||
119 | } | ||
diff --git a/meta/recipes-kernel/systemtap/systemtap_git.bb b/meta/recipes-kernel/systemtap/systemtap_git.bb index e8aa54f683..732d9a7685 100644 --- a/meta/recipes-kernel/systemtap/systemtap_git.bb +++ b/meta/recipes-kernel/systemtap/systemtap_git.bb | |||
@@ -1,15 +1,14 @@ | |||
1 | DESCRIPTION = "SystemTap - script-directed dynamic tracing and performance analysis tool for Linux" | 1 | DESCRIPTION = "SystemTap - script-directed dynamic tracing and performance analysis tool for Linux" |
2 | LICENSE = "GPLv2" | 2 | LICENSE = "GPLv2" |
3 | LIC_FILES_CHKSUM = "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f" | 3 | LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263" |
4 | 4 | ||
5 | DEPENDS = "elfutils sqlite3" | 5 | DEPENDS = "elfutils sqlite3" |
6 | 6 | ||
7 | SRCREV = "4ab3a1863bf4f472acae7a809bf2b38d91658aa8" | 7 | SRCREV = "820f2d22fc47fad6e09ba886efb9b91e1247cb39" |
8 | PR = "r4" | 8 | PR = "r0" |
9 | PV = "1.4+git${SRCPV}" | 9 | PV = "1.6+git${SRCPV}" |
10 | 10 | ||
11 | SRC_URI = "git://sources.redhat.com/git/systemtap.git;protocol=git \ | 11 | SRC_URI = "git://sources.redhat.com/git/systemtap.git;protocol=git \ |
12 | file://fix_for_compilation_with_gcc-4.6.0.patch \ | ||
13 | " | 12 | " |
14 | 13 | ||
15 | EXTRA_OECONF += "--with-libelf=${STAGING_DIR_TARGET} --without-rpm \ | 14 | EXTRA_OECONF += "--with-libelf=${STAGING_DIR_TARGET} --without-rpm \ |
@@ -23,11 +22,11 @@ EXTRA_OECONF += "--with-libelf=${STAGING_DIR_TARGET} --without-rpm \ | |||
23 | SRC_URI[md5sum] = "cb202866ed704c44a876d041f788bdee" | 22 | SRC_URI[md5sum] = "cb202866ed704c44a876d041f788bdee" |
24 | SRC_URI[sha256sum] = "8ffe35caec0d937bd23fd78a3a8d94b58907cc0de0330b35e38f9f764815c459" | 23 | SRC_URI[sha256sum] = "8ffe35caec0d937bd23fd78a3a8d94b58907cc0de0330b35e38f9f764815c459" |
25 | 24 | ||
26 | # systemtap doesn't work on arm and doesn't support mips | 25 | # systemtap doesn't support mips |
27 | COMPATIBLE_HOST = '(x86_64.*|i.86.*|powerpc.*)-linux' | 26 | COMPATIBLE_HOST = '(x86_64.*|i.86.*|powerpc.*|arm.*)-linux' |
28 | 27 | ||
29 | S = "${WORKDIR}/git" | 28 | S = "${WORKDIR}/git" |
30 | 29 | ||
31 | inherit autotools | 30 | inherit autotools gettext |
32 | 31 | ||
33 | FILES_${PN}-dbg += "${libexecdir}/systemtap/.debug" | 32 | FILES_${PN}-dbg += "${libexecdir}/systemtap/.debug" |