From 1d87eb3ff2bab5d0a4aeb0208bc87412f9348237 Mon Sep 17 00:00:00 2001 From: Richard Purdie Date: Wed, 9 May 2007 16:11:18 +0000 Subject: oprofile: Update patches git-svn-id: https://svn.o-hand.com/repos/poky/trunk@1637 311d38ba-8fff-0310-9ca6-ca027cbcb966 --- .../packages/oprofile/oprofile/cg_collection.patch | 289 ++++++++++++++++++++ .../oprofile/oprofile/oparchive-debug-dir.patch | 42 ++- .../oprofile/oprofile/oparchive-list-files.patch | 71 +++++ .../packages/oprofile/oprofile/oprofile_eabi.patch | 12 + .../packages/oprofile/oprofile/xml_callgraph.patch | 296 +++------------------ meta/packages/oprofile/oprofile_cvs.bb | 3 +- 6 files changed, 445 insertions(+), 268 deletions(-) create mode 100644 meta/packages/oprofile/oprofile/cg_collection.patch (limited to 'meta/packages/oprofile') diff --git a/meta/packages/oprofile/oprofile/cg_collection.patch b/meta/packages/oprofile/oprofile/cg_collection.patch new file mode 100644 index 000000000..a87ec7247 --- /dev/null +++ b/meta/packages/oprofile/oprofile/cg_collection.patch @@ -0,0 +1,289 @@ + ChangeLog | 12 ++++++++++++ + libpp/callgraph_container.cpp | 21 +++++++++++++-------- + libpp/callgraph_container.h | 7 ++++--- + libpp/format_output.cpp | 22 +++++++++++----------- + libpp/format_output.h | 2 +- + libpp/symbol.h | 14 ++++++++------ + libpp/symbol_sort.cpp | 17 ----------------- + libpp/symbol_sort.h | 6 ------ + pp/opreport.cpp | 3 ++- + 9 files changed, 51 insertions(+), 53 deletions(-) + +Index: oprofile/libpp/callgraph_container.cpp +=================================================================== +--- oprofile.orig/libpp/callgraph_container.cpp ++++ oprofile/libpp/callgraph_container.cpp +@@ -379,12 +379,15 @@ process(count_array_t total, double thre + + process_children(sym, threshold); + +- cg_syms.push_back(sym); ++ cg_syms_objs.push_back(sym); + } ++ ++ for (unsigned int i = 0; i < cg_syms_objs.size(); i++) ++ cg_syms.push_back(&cg_syms_objs[i]); + } + + +-cg_collection arc_recorder::get_symbols() const ++symbol_collection arc_recorder::get_symbols() const + { + return cg_syms; + } +@@ -580,12 +583,14 @@ column_flags callgraph_container::output + column_flags output_hints = cf_none; + + // FIXME: costly: must we access directly recorder map ? +- cg_collection syms = recorder.get_symbols(); ++ symbol_collection syms = recorder.get_symbols(); + +- cg_collection::const_iterator it; +- cg_collection::const_iterator const end = syms.end(); +- for (it = syms.begin(); it != end; ++it) +- output_hints = it->output_hint(output_hints); ++ symbol_collection::iterator it; ++ symbol_collection::iterator const end = syms.end(); ++ for (it = syms.begin(); it != end; ++it) { ++ cg_symbol const *cg_symb = dynamic_cast(*it); ++ output_hints = cg_symb->output_hint(output_hints); ++ } + + return output_hints; + } +@@ -597,7 +602,7 @@ count_array_t callgraph_container::sampl + } + + +-cg_collection callgraph_container::get_symbols() const ++symbol_collection callgraph_container::get_symbols() const + { + return recorder.get_symbols(); + } +Index: oprofile/libpp/callgraph_container.h +=================================================================== +--- oprofile.orig/libpp/callgraph_container.h ++++ oprofile/libpp/callgraph_container.h +@@ -53,7 +53,7 @@ public: + count_array_t const & arc_count); + + /// return all the cg symbols +- cg_collection get_symbols() const; ++ symbol_collection get_symbols() const; + + /** + * After population, build the final output, and do +@@ -91,7 +91,8 @@ private: + map_t sym_map; + + /// final output data +- cg_collection cg_syms; ++ symbol_collection cg_syms; ++ cg_collection_objs cg_syms_objs; + }; + + +@@ -126,7 +127,7 @@ public: + count_array_t samples_count() const; + + // return all the cg symbols +- cg_collection get_symbols() const; ++ symbol_collection get_symbols() const; + + private: + /** +Index: oprofile/libpp/format_output.cpp +=================================================================== +--- oprofile.orig/libpp/format_output.cpp ++++ oprofile/libpp/format_output.cpp +@@ -489,7 +489,7 @@ cg_formatter::cg_formatter(callgraph_con + } + + +-void cg_formatter::output(ostream & out, cg_collection const & syms) ++void cg_formatter::output(ostream & out, symbol_collection const & syms) + { + // amount of spacing prefixing child and parent lines + string const child_parent_prefix(" "); +@@ -498,37 +498,37 @@ void cg_formatter::output(ostream & out, + + out << string(79, '-') << endl; + +- cg_collection::const_iterator it; +- cg_collection::const_iterator end = syms.end(); ++ symbol_collection::const_iterator it; ++ symbol_collection::const_iterator end = syms.end(); + + for (it = syms.begin(); it < end; ++it) { +- cg_symbol const & sym = *it; ++ cg_symbol const *sym = dynamic_cast(*it); + + cg_symbol::children::const_iterator cit; +- cg_symbol::children::const_iterator cend = sym.callers.end(); ++ cg_symbol::children::const_iterator cend = sym->callers.end(); + + counts_t c; + if (global_percent) + c.total = counts.total; + else +- c.total = sym.total_caller_count; ++ c.total = sym->total_caller_count; + +- for (cit = sym.callers.begin(); cit != cend; ++cit) { ++ for (cit = sym->callers.begin(); cit != cend; ++cit) { + out << child_parent_prefix; + do_output(out, *cit, cit->sample, c); + } + +- do_output(out, sym, sym.sample, counts); ++ do_output(out, *sym, sym->sample, counts); + + c = counts_t(); + if (global_percent) + c.total = counts.total; + else +- c.total = sym.total_callee_count; ++ c.total = sym->total_callee_count; + +- cend = sym.callees.end(); ++ cend = sym->callees.end(); + +- for (cit = sym.callees.begin(); cit != cend; ++cit) { ++ for (cit = sym->callees.begin(); cit != cend; ++cit) { + out << child_parent_prefix; + do_output(out, *cit, cit->sample, c); + } +Index: oprofile/libpp/symbol.h +=================================================================== +--- oprofile.orig/libpp/symbol.h ++++ oprofile/libpp/symbol.h +@@ -56,8 +56,11 @@ struct sample_entry { + + + /// associate a symbol with a file location, samples count and vma address +-struct symbol_entry { ++class symbol_entry { ++public: + symbol_entry() : size(0) {} ++ virtual ~symbol_entry() {} ++ + /// which image this symbol belongs to + image_name_id image_name; + /// owning application name: identical to image name if profiling +@@ -93,7 +96,8 @@ typedef std::vector children; +@@ -109,10 +113,8 @@ struct cg_symbol : public symbol_entry { + count_array_t total_callee_count; + }; + +- +-/// a collection of sorted callgraph symbols +-typedef std::vector cg_collection; +- ++/// a collection of sorted callgraph symbol objects ++typedef std::vector cg_collection_objs; + + /// for storing diff %ages + typedef growable_vector diff_array_t; +Index: oprofile/libpp/symbol_sort.cpp +=================================================================== +--- oprofile.orig/libpp/symbol_sort.cpp ++++ oprofile/libpp/symbol_sort.cpp +@@ -146,23 +146,6 @@ sort(symbol_collection & syms, bool reve + + + void sort_options:: +-sort(cg_collection & syms, bool reverse_sort, bool lf) const +-{ +- long_filenames = lf; +- +- vector sort_option(options); +- for (sort_order cur = first; cur != last; cur = sort_order(cur + 1)) { +- if (find(sort_option.begin(), sort_option.end(), cur) == +- sort_option.end()) +- sort_option.push_back(cur); +- } +- +- stable_sort(syms.begin(), syms.end(), +- symbol_compare(sort_option, reverse_sort)); +-} +- +- +-void sort_options:: + sort(diff_collection & syms, bool reverse_sort, bool lf) const + { + long_filenames = lf; +Index: oprofile/libpp/symbol_sort.h +=================================================================== +--- oprofile.orig/libpp/symbol_sort.h ++++ oprofile/libpp/symbol_sort.h +@@ -44,12 +44,6 @@ struct sort_options { + /** + * Sort the given container by the given criteria. + */ +- void sort(cg_collection & syms, bool reverse_sort, +- bool long_filenames) const; +- +- /** +- * Sort the given container by the given criteria. +- */ + void sort(diff_collection & syms, bool reverse_sort, + bool long_filenames) const; + +Index: oprofile/pp/opreport.cpp +=================================================================== +--- oprofile.orig/pp/opreport.cpp ++++ oprofile/pp/opreport.cpp +@@ -445,7 +445,8 @@ void output_cg_symbols(callgraph_contain + { + column_flags output_hints = cg.output_hint(); + +- cg_collection symbols = cg.get_symbols(); ++ symbol_collection symbols = cg.get_symbols(); ++ + options::sort_by.sort(symbols, options::reverse_sort, + options::long_filenames); + +Index: oprofile/ChangeLog +=================================================================== +--- oprofile.orig/ChangeLog ++++ oprofile/ChangeLog +@@ -1,5 +1,17 @@ + 2007-05-09 Richard Purdie + ++ * libpp/callgraph_container.cpp: ++ * libpp/callgraph_container.h: ++ * libpp/format_output.cpp: ++ * libpp/format_output.h: ++ * libpp/symbol.h: ++ * libpp/symbol_sort.cpp: ++ * libpp/symbol_sort.h: ++ * pp/opreport.cpp: Convert cg_collection to symbol_collection and use ++ dynamic casting allowing more code reuse for callgraphs ++ ++2007-05-09 Richard Purdie ++ + * oparchive.cpp: + * oparchive_options.cpp: + * oparchive_options.h: Add --list-files option to list all files that +Index: oprofile/libpp/format_output.h +=================================================================== +--- oprofile.orig/libpp/format_output.h ++++ oprofile/libpp/format_output.h +@@ -201,7 +201,7 @@ public: + + /** output callgraph information according to the previously format + * specifier set by call(s) to add_format() */ +- void output(std::ostream & out, cg_collection const & syms); ++ void output(std::ostream & out, symbol_collection const & syms); + }; + + /// class to output a columned format symbols plus diff values diff --git a/meta/packages/oprofile/oprofile/oparchive-debug-dir.patch b/meta/packages/oprofile/oprofile/oparchive-debug-dir.patch index 43a4bbd8c..ab925c904 100644 --- a/meta/packages/oprofile/oprofile/oparchive-debug-dir.patch +++ b/meta/packages/oprofile/oprofile/oparchive-debug-dir.patch @@ -2,7 +2,21 @@ Index: oprofile/pp/oparchive.cpp =================================================================== --- oprofile.orig/pp/oparchive.cpp +++ oprofile/pp/oparchive.cpp -@@ -97,8 +97,17 @@ int oparchive(options::spec const & spec +@@ -84,9 +84,10 @@ int oparchive(options::spec const & spec + exe_archive_file); + + /* If there are any debuginfo files, copy them over. +- * Need to copy the debug info file in the same +- * directory as the executable. The /usr/lib/debug +- * search path is not going to work. ++ * Need to copy the debug info file to somewhere we'll ++ * find it - executable location + "/.debug" ++ * to avoid overwriting files with the same name. The ++ * /usr/lib/debug search path is not going to work. + */ + bfd * ibfd = open_bfd(exe_name); + if (ibfd) { +@@ -97,8 +98,15 @@ int oparchive(options::spec const & spec if (find_separate_debug_file(ibfd, dirname, global, debug_filename)) { /* found something copy it over */ @@ -10,15 +24,29 @@ Index: oprofile/pp/oparchive.cpp - dirname + "/" + + string dest_debug_dir = options::outdirectory + + dirname + "/.debug/"; -+ if (mkdir(dest_debug_dir.c_str(),0755) < 0) { -+ if (errno!=EEXIST) { -+ cerr << "Unable to create directory: " -+ << dest_debug_dir << "." << endl; -+ exit (EXIT_FAILURE); -+ } ++ if (create_dir(dest_debug_dir.c_str())) { ++ cerr << "Unable to create directory: " ++ << dest_debug_dir << "." << endl; ++ exit (EXIT_FAILURE); + } + + string dest_debug = dest_debug_dir + op_basename(debug_filename); copy_one_file(image_ok, debug_filename, dest_debug); } +Index: oprofile/ChangeLog +=================================================================== +--- oprofile.orig/ChangeLog ++++ oprofile/ChangeLog +@@ -1,5 +1,11 @@ + 2007-05-09 Richard Purdie + ++ * oprofile/pp/oparchive.cpp: If the debug files have the same name ++ as the original binary, the binary gets overwritten. Add in a .debug ++ directory to avoid this. ++ ++2007-05-09 Richard Purdie ++ + * oprofile/daemon/opd_cookie.c: Fix syscall for ARM EABI + + 2007-04-25 Manoj Ekbote diff --git a/meta/packages/oprofile/oprofile/oparchive-list-files.patch b/meta/packages/oprofile/oprofile/oparchive-list-files.patch index 93cf3cdb8..8d92efead 100644 --- a/meta/packages/oprofile/oprofile/oparchive-list-files.patch +++ b/meta/packages/oprofile/oprofile/oparchive-list-files.patch @@ -14,6 +14,45 @@ Index: oprofile/pp/oparchive.cpp if (!copy_file(source, dest) && err == image_ok) { cerr << "can't copy from " << source << " to " << dest << " cause: " << strerror(errno) << endl; +@@ -46,7 +51,7 @@ int oparchive(options::spec const & spec + handle_options(spec); + + /* Check to see if directory can be created */ +- if (create_path(options::outdirectory.c_str())) { ++ if (!options::list_files && create_path(options::outdirectory.c_str())) { + cerr << "Unable to create directory for " + << options::outdirectory << "." << endl; + exit (EXIT_FAILURE); +@@ -73,7 +78,8 @@ int oparchive(options::spec const & spec + + cverb << vdebug << exe_name << endl; + /* Create directory for executable file. */ +- if (create_path(exe_archive_file.c_str())) { ++ if (!options::list_files ++ && create_path(exe_archive_file.c_str())) { + cerr << "Unable to create directory for " + << exe_archive_file << "." << endl; + exit (EXIT_FAILURE); +@@ -100,7 +106,8 @@ int oparchive(options::spec const & spec + /* found something copy it over */ + string dest_debug_dir = options::outdirectory + + dirname + "/.debug/"; +- if (create_dir(dest_debug_dir.c_str())) { ++ if (!options::list_files ++ && create_dir(dest_debug_dir.c_str())) { + cerr << "Unable to create directory: " + << dest_debug_dir << "." << endl; + exit (EXIT_FAILURE); +@@ -130,7 +137,8 @@ int oparchive(options::spec const & spec + + cverb << vdebug << (sample_name) << endl; + cverb << vdebug << " destp " << sample_archive_file << endl; +- if (create_path(sample_archive_file.c_str())) { ++ if (!options::list_files ++ && create_path(sample_archive_file.c_str())) { + cerr << "Unable to create directory for " + << sample_archive_file << "." << endl; + exit (EXIT_FAILURE); Index: oprofile/pp/oparchive_options.cpp =================================================================== --- oprofile.orig/pp/oparchive_options.cpp @@ -58,3 +97,35 @@ Index: oprofile/pp/oparchive_options.h } /// All the chosen sample files. +Index: oprofile/ChangeLog +=================================================================== +--- oprofile.orig/ChangeLog ++++ oprofile/ChangeLog +@@ -1,5 +1,12 @@ + 2007-05-09 Richard Purdie + ++ * oparchive.cpp: ++ * oparchive_options.cpp: ++ * oparchive_options.h: Add --list-files option to list all files that ++ would be handled by oparchive. ++ ++2007-05-09 Richard Purdie ++ + * oprofile/pp/oparchive.cpp: If the debug files have the same name + as the original binary, the binary gets overwritten. Add in a .debug + directory to avoid this. +Index: oprofile/doc/oparchive.1.in +=================================================================== +--- oprofile.orig/doc/oparchive.1.in ++++ oprofile/doc/oparchive.1.in +@@ -50,6 +50,10 @@ Output to the given directory. There is + Do not include application-specific images for libraries, kernel modules + and the kernel. This option only makes sense if the profile session + used --separate. ++.br ++.TP ++.BI "--list-files / -l" ++Only list the files that would be archived, don't copy them. + + .SH ENVIRONMENT + No special environment variables are recognised by oparchive. diff --git a/meta/packages/oprofile/oprofile/oprofile_eabi.patch b/meta/packages/oprofile/oprofile/oprofile_eabi.patch index 7388deef3..2ae269f86 100644 --- a/meta/packages/oprofile/oprofile/oprofile_eabi.patch +++ b/meta/packages/oprofile/oprofile/oprofile_eabi.patch @@ -16,3 +16,15 @@ Index: oprofile/daemon/opd_cookie.c static inline int lookup_dcookie(cookie_t cookie, char * buf, size_t size) { return syscall(__NR_lookup_dcookie, +Index: oprofile/ChangeLog +=================================================================== +--- oprofile.orig/ChangeLog ++++ oprofile/ChangeLog +@@ -1,3 +1,7 @@ ++2007-05-09 Richard Purdie ++ ++ * oprofile/daemon/opd_cookie.c: Fix syscall for ARM EABI ++ + 2007-04-25 Manoj Ekbote + + * events/mips/sb1/events: fix SB1 events diff --git a/meta/packages/oprofile/oprofile/xml_callgraph.patch b/meta/packages/oprofile/oprofile/xml_callgraph.patch index 1ffe6f69b..f089e7dc4 100644 --- a/meta/packages/oprofile/oprofile/xml_callgraph.patch +++ b/meta/packages/oprofile/oprofile/xml_callgraph.patch @@ -1,162 +1,18 @@ ---- - libpp/callgraph_container.cpp | 22 ++- - libpp/callgraph_container.h | 7 - - libpp/format_output.cpp | 245 ++++++++++++++++++++++++++++++++++++------ - libpp/format_output.h | 41 +++++-- - libpp/symbol.h | 15 +- - libpp/symbol_sort.cpp | 17 -- - libpp/symbol_sort.h | 6 - - libpp/xml_utils.cpp | 17 -- - libutil++/xml_output.cpp | 3 - libutil++/xml_output.h | 3 - pp/opreport.cpp | 40 +++++- - pp/opreport_options.cpp | 5 - 12 files changed, 314 insertions(+), 107 deletions(-) + ChangeLog | 10 ++ + libpp/format_output.cpp | 178 +++++++++++++++++++++++++++++++++++++++++------ + libpp/format_output.h | 40 ++++++++-- + libpp/xml_utils.cpp | 17 +--- + libutil++/xml_output.cpp | 3 + libutil++/xml_output.h | 3 + pp/opreport.cpp | 37 +++++++-- + pp/opreport_options.cpp | 5 - + 8 files changed, 238 insertions(+), 55 deletions(-) + -Index: oprofile/libpp/callgraph_container.cpp -=================================================================== ---- oprofile.orig/libpp/callgraph_container.cpp -+++ oprofile/libpp/callgraph_container.cpp -@@ -379,12 +379,15 @@ process(count_array_t total, double thre - - process_children(sym, threshold); - -- cg_syms.push_back(sym); -+ cg_syms_objs.push_back(sym); - } -+ -+ for (unsigned int i = 0; i < cg_syms_objs.size(); i++) -+ cg_syms.push_back(&cg_syms_objs[i]); - } - - --cg_collection arc_recorder::get_symbols() const -+symbol_collection arc_recorder::get_symbols() const - { - return cg_syms; - } -@@ -580,12 +583,14 @@ column_flags callgraph_container::output - column_flags output_hints = cf_none; - - // FIXME: costly: must we access directly recorder map ? -- cg_collection syms = recorder.get_symbols(); -+ symbol_collection syms = recorder.get_symbols(); - -- cg_collection::const_iterator it; -- cg_collection::const_iterator const end = syms.end(); -- for (it = syms.begin(); it != end; ++it) -- output_hints = it->output_hint(output_hints); -+ symbol_collection::iterator it; -+ symbol_collection::iterator const end = syms.end(); -+ for (it = syms.begin(); it != end; ++it) { -+ cg_symbol const *cg_symb = dynamic_cast(*it); -+ output_hints = cg_symb->output_hint(output_hints); -+ } - - return output_hints; - } -@@ -597,7 +602,7 @@ count_array_t callgraph_container::sampl - } - - --cg_collection callgraph_container::get_symbols() const -+symbol_collection callgraph_container::get_symbols() const - { - return recorder.get_symbols(); - } -Index: oprofile/libpp/callgraph_container.h -=================================================================== ---- oprofile.orig/libpp/callgraph_container.h -+++ oprofile/libpp/callgraph_container.h -@@ -53,7 +53,7 @@ public: - count_array_t const & arc_count); - - /// return all the cg symbols -- cg_collection get_symbols() const; -+ symbol_collection get_symbols() const; - - /** - * After population, build the final output, and do -@@ -91,7 +91,8 @@ private: - map_t sym_map; - - /// final output data -- cg_collection cg_syms; -+ symbol_collection cg_syms; -+ cg_collection_objs cg_syms_objs; - }; - - -@@ -126,7 +127,7 @@ public: - count_array_t samples_count() const; - - // return all the cg symbols -- cg_collection get_symbols() const; -+ symbol_collection get_symbols() const; - - private: - /** Index: oprofile/libpp/format_output.cpp =================================================================== --- oprofile.orig/libpp/format_output.cpp +++ oprofile/libpp/format_output.cpp -@@ -489,7 +489,7 @@ cg_formatter::cg_formatter(callgraph_con - } - - --void cg_formatter::output(ostream & out, cg_collection const & syms) -+void cg_formatter::output(ostream & out, symbol_collection const & syms) - { - // amount of spacing prefixing child and parent lines - string const child_parent_prefix(" "); -@@ -498,37 +498,37 @@ void cg_formatter::output(ostream & out, - - out << string(79, '-') << endl; - -- cg_collection::const_iterator it; -- cg_collection::const_iterator end = syms.end(); -+ symbol_collection::const_iterator it; -+ symbol_collection::const_iterator end = syms.end(); - - for (it = syms.begin(); it < end; ++it) { -- cg_symbol const & sym = *it; -+ cg_symbol const *sym = dynamic_cast(*it); - - cg_symbol::children::const_iterator cit; -- cg_symbol::children::const_iterator cend = sym.callers.end(); -+ cg_symbol::children::const_iterator cend = sym->callers.end(); - - counts_t c; - if (global_percent) - c.total = counts.total; - else -- c.total = sym.total_caller_count; -+ c.total = sym->total_caller_count; - -- for (cit = sym.callers.begin(); cit != cend; ++cit) { -+ for (cit = sym->callers.begin(); cit != cend; ++cit) { - out << child_parent_prefix; - do_output(out, *cit, cit->sample, c); - } - -- do_output(out, sym, sym.sample, counts); -+ do_output(out, *sym, sym->sample, counts); - - c = counts_t(); - if (global_percent) - c.total = counts.total; - else -- c.total = sym.total_callee_count; -+ c.total = sym->total_callee_count; - -- cend = sym.callees.end(); -+ cend = sym->callees.end(); - -- for (cit = sym.callees.begin(); cit != cend; ++cit) { -+ for (cit = sym->callees.begin(); cit != cend; ++cit) { - out << child_parent_prefix; - do_output(out, *cit, cit->sample, c); - } @@ -562,6 +562,20 @@ ostringstream bytes_out; map symbol_data_table; size_t symbol_data_index = 0; @@ -306,7 +162,7 @@ Index: oprofile/libpp/format_output.cpp +} + +void xml_cg_formatter:: -+output_symbol_core(ostream & out, cg_symbol::children const cg_symb, ++output_symbol_core(ostream & out, cg_symbol::children const cg_symb, + string const selfname, string const qname, + size_t lo, size_t hi, bool is_module, tag_t tag) +{ @@ -338,7 +194,7 @@ Index: oprofile/libpp/format_output.cpp + + string const symname = symbol_names.name(cit->name); + assert(symname.size() > 0); -+ ++ + string const symqname = module + ":" + symname; + + // Find any self references and handle @@ -387,7 +243,7 @@ Index: oprofile/libpp/format_output.cpp + + string const name = symbol_names.name(symb->name); + assert(name.size() > 0); -+ ++ + string const image = get_image_name(symb->image_name, true); + string const qname = image + ":" + name; + @@ -418,15 +274,6 @@ Index: oprofile/libpp/format_output.h =================================================================== --- oprofile.orig/libpp/format_output.h +++ oprofile/libpp/format_output.h -@@ -201,7 +201,7 @@ public: - - /** output callgraph information according to the previously format - * specifier set by call(s) to add_format() */ -- void output(std::ostream & out, cg_collection const & syms); -+ void output(std::ostream & out, symbol_collection const & syms); - }; - - /// class to output a columned format symbols plus diff values @@ -227,7 +227,7 @@ private: class xml_formatter : public formatter { public: @@ -496,99 +343,14 @@ Index: oprofile/libpp/format_output.h + /// container we work from + callgraph_container const * callgraph; + -+ void output_symbol_core(std::ostream & out, -+ cg_symbol::children const cg_symb, ++ void output_symbol_core(std::ostream & out, ++ cg_symbol::children const cg_symb, + std::string const selfname, std::string const qname, + size_t lo, size_t hi, bool is_module, tag_t tag); +}; } // namespace format_output -Index: oprofile/libpp/symbol.h -=================================================================== ---- oprofile.orig/libpp/symbol.h -+++ oprofile/libpp/symbol.h -@@ -56,8 +56,11 @@ struct sample_entry { - - - /// associate a symbol with a file location, samples count and vma address --struct symbol_entry { -+class symbol_entry { -+public: - symbol_entry() : size(0) {} -+ virtual ~symbol_entry() {} -+ - /// which image this symbol belongs to - image_name_id image_name; - /// owning application name: identical to image name if profiling -@@ -93,7 +96,8 @@ typedef std::vector children; -@@ -109,10 +113,8 @@ struct cg_symbol : public symbol_entry { - count_array_t total_callee_count; - }; - -- --/// a collection of sorted callgraph symbols --typedef std::vector cg_collection; -- -+/// a collection of sorted callgraph symbol objects -+typedef std::vector cg_collection_objs; - - /// for storing diff %ages - typedef growable_vector diff_array_t; -Index: oprofile/libpp/symbol_sort.cpp -=================================================================== ---- oprofile.orig/libpp/symbol_sort.cpp -+++ oprofile/libpp/symbol_sort.cpp -@@ -146,23 +146,6 @@ sort(symbol_collection & syms, bool reve - - - void sort_options:: --sort(cg_collection & syms, bool reverse_sort, bool lf) const --{ -- long_filenames = lf; -- -- vector sort_option(options); -- for (sort_order cur = first; cur != last; cur = sort_order(cur + 1)) { -- if (find(sort_option.begin(), sort_option.end(), cur) == -- sort_option.end()) -- sort_option.push_back(cur); -- } -- -- stable_sort(syms.begin(), syms.end(), -- symbol_compare(sort_option, reverse_sort)); --} -- -- --void sort_options:: - sort(diff_collection & syms, bool reverse_sort, bool lf) const - { - long_filenames = lf; -Index: oprofile/libpp/symbol_sort.h -=================================================================== ---- oprofile.orig/libpp/symbol_sort.h -+++ oprofile/libpp/symbol_sort.h -@@ -44,12 +44,6 @@ struct sort_options { - /** - * Sort the given container by the given criteria. - */ -- void sort(cg_collection & syms, bool reverse_sort, -- bool long_filenames) const; -- -- /** -- * Sort the given container by the given criteria. -- */ - void sort(diff_collection & syms, bool reverse_sort, - bool long_filenames) const; - Index: oprofile/libpp/xml_utils.cpp =================================================================== --- oprofile.orig/libpp/xml_utils.cpp @@ -693,13 +455,7 @@ Index: oprofile/pp/opreport.cpp xml_out->show_details(options::details); out = xml_out; // for XML always output long filenames -@@ -445,25 +445,45 @@ void output_cg_symbols(callgraph_contain - { - column_flags output_hints = cg.output_hint(); - -- cg_collection symbols = cg.get_symbols(); -+ symbol_collection symbols = cg.get_symbols(); -+ +@@ -450,21 +450,40 @@ void output_cg_symbols(callgraph_contain options::sort_by.sort(symbols, options::reverse_sort, options::long_filenames); @@ -764,3 +520,23 @@ Index: oprofile/pp/opreport_options.cpp if (accumulated) { cerr << "--accumulated is incompatible with --xml" << endl; do_exit = true; +Index: oprofile/ChangeLog +=================================================================== +--- oprofile.orig/ChangeLog ++++ oprofile/ChangeLog +@@ -1,5 +1,15 @@ + 2007-05-09 Richard Purdie + ++ * libpp/format_output.cpp: ++ * libpp/format_output.h: ++ * libpp/xml_utils.cpp: ++ * libutil++/xml_output.cpp: ++ * libutil++/xml_output.h: ++ * pp/opreport.cpp: ++ * pp/opreport_options.cpp: Add callgraph XML output ++ ++2007-05-09 Richard Purdie ++ + * libpp/callgraph_container.cpp: + * libpp/callgraph_container.h: + * libpp/format_output.cpp: diff --git a/meta/packages/oprofile/oprofile_cvs.bb b/meta/packages/oprofile/oprofile_cvs.bb index b0ccc3ce6..dab1f093a 100644 --- a/meta/packages/oprofile/oprofile_cvs.bb +++ b/meta/packages/oprofile/oprofile_cvs.bb @@ -1,5 +1,5 @@ PV = "0.9.2+cvs${SRCDATE}" -PR = "r3" +PR = "r4" SECTION = "devel" DESCRIPTION = "OProfile is a system-wide profiler for Linux systems, capable \ of profiling all running code at low overhead." @@ -10,6 +10,7 @@ SRC_URI = "cvs://anonymous@oprofile.cvs.sourceforge.net/cvsroot/oprofile;module= file://oprofile_eabi.patch;patch=1 \ file://oparchive-debug-dir.patch;patch=1 \ file://oparchive-list-files.patch;patch=1 \ + file://cg_collection.patch;patch=1 \ file://xml_callgraph.patch;patch=1 \ file://acinclude.m4" S = "${WORKDIR}/oprofile" -- cgit v1.2.3