Yichun Zhang (agentz - [PATCH v3 0/1] PR11096: Add support for the "module" argument to @var (original) (raw)

This is the mail archive of the systemtap@sourceware.orgmailing list for the systemtap project.

• From: "Yichun Zhang (agentzh)"
• To: systemtap at sourceware dot org
• Cc: Josh Stone , "Yichun Zhang (agentzh)"
• Date: Tue, 25 Jun 2013 22:42:22 -0700
• Subject: [PATCH v3 0/1] PR11096: Add support for the "module" argument to @var
• References: <51C8EF57 dot 2050801 at redhat dot com>

From: "Yichun Zhang (agentzh)" agentzh@gmail.com

v3:

• Fix typos in the commit message.
• Mark PR numbers in test files at_var_timer_profile.exp and at_var_pie.exp.
• Remove the useless field "cu_name" from cast_op.
• Fix @var("name") that does not match any local variables: we did not scan all the CUs for a match in globals in the current module. Add a corresponding test case to the test file at_var_cu.exp.
• Fix the issue that too many error messages (per CU) are printed when @var fails to resolve. Now we only print one error message for each of the modules. Add test file at_var_unresolved.exp for this case.
• Remove a inappropriate try-catch block in dwarf_atvar_expanding_visitor::visit_atvar_op.
• Fix the issue when accessing global variables in PIE and DSO via @var.

Yichun Zhang (agentzh) (1): PR11096: Add support for the "module" argument to @var

dwflpp.cxx | 12 +- elaborate.cxx | 48 ++++ elaborate.h | 1 + parse.cxx | 29 +- staptree.cxx | 76 +++++- staptree.h | 17 +- tapsets.cxx | 300 ++++++++++++++++----- testsuite/systemtap.base/at_var.exp | 3 + testsuite/systemtap.base/at_var_cu.exp | 57 ++++ testsuite/systemtap.base/at_var_cu.stp | 33 +++ testsuite/systemtap.base/at_var_cu_1.c | 14 + testsuite/systemtap.base/at_var_cu_2.c | 13 + testsuite/systemtap.base/at_var_cu_3.c | 13 + testsuite/systemtap.base/at_var_func.exp | 48 ++++ testsuite/systemtap.base/at_var_func.stp | 21 ++ testsuite/systemtap.base/at_var_lvalue.c | 29 ++ testsuite/systemtap.base/at_var_lvalue.exp | 34 +++ testsuite/systemtap.base/at_var_lvalue.stp | 25 ++ testsuite/systemtap.base/at_var_mark.exp | 5 +- testsuite/systemtap.base/at_var_mark_func.exp | 26 ++ testsuite/systemtap.base/at_var_mark_func.stp | 24 ++ testsuite/systemtap.base/at_var_pie.exp | 50 ++++ testsuite/systemtap.base/at_var_timer_profile.c | 27 ++ testsuite/systemtap.base/at_var_timer_profile.exp | 51 ++++ testsuite/systemtap.base/at_var_timer_profile.stp | 23 ++ testsuite/systemtap.base/at_var_tracepoint.exp | 10 + testsuite/systemtap.base/at_var_tracepoint.stp | 9 + testsuite/systemtap.base/at_var_unresolved.exp | 67 +++++ .../systemtap.base/at_var_unresolved_lvalue.exp | 42 +++ .../systemtap.base/at_var_unresolved_lvalue.stp | 5 + testsuite/systemtap.base/at_var_void_stmt.c | 27 ++ testsuite/systemtap.base/at_var_void_stmt.exp | 49 ++++ testsuite/systemtap.base/at_var_void_stmt.stp | 13 + testsuite/systemtap.base/global_var_kernel.exp | 5 +- testsuite/systemtap.base/global_var_kernel.stp | 7 + translate.cxx | 8 + 36 files changed, 1119 insertions(+), 102 deletions(-) create mode 100644 testsuite/systemtap.base/at_var_cu.exp create mode 100644 testsuite/systemtap.base/at_var_cu.stp create mode 100644 testsuite/systemtap.base/at_var_cu_1.c create mode 100644 testsuite/systemtap.base/at_var_cu_2.c create mode 100644 testsuite/systemtap.base/at_var_cu_3.c create mode 100644 testsuite/systemtap.base/at_var_func.exp create mode 100644 testsuite/systemtap.base/at_var_func.stp create mode 100644 testsuite/systemtap.base/at_var_lvalue.c create mode 100644 testsuite/systemtap.base/at_var_lvalue.exp create mode 100644 testsuite/systemtap.base/at_var_lvalue.stp create mode 100644 testsuite/systemtap.base/at_var_mark_func.exp create mode 100644 testsuite/systemtap.base/at_var_mark_func.stp create mode 100644 testsuite/systemtap.base/at_var_pie.exp create mode 100644 testsuite/systemtap.base/at_var_timer_profile.c create mode 100644 testsuite/systemtap.base/at_var_timer_profile.exp create mode 100644 testsuite/systemtap.base/at_var_timer_profile.stp create mode 100644 testsuite/systemtap.base/at_var_tracepoint.exp create mode 100644 testsuite/systemtap.base/at_var_tracepoint.stp create mode 100644 testsuite/systemtap.base/at_var_unresolved.exp create mode 100644 testsuite/systemtap.base/at_var_unresolved_lvalue.exp create mode 100644 testsuite/systemtap.base/at_var_unresolved_lvalue.stp create mode 100644 testsuite/systemtap.base/at_var_void_stmt.c create mode 100644 testsuite/systemtap.base/at_var_void_stmt.exp create mode 100644 testsuite/systemtap.base/at_var_void_stmt.stp

-- 1.7.11.7

• Follow-Ups:
  • [PATCH v3 1/1] PR11096: Add support for the "module" argument to @var
    * From: Yichun Zhang (agentzh)
• References:
  • Re: [PATCH v2 1/1] PR11096: Add support for the "module" argument to @var
    * From: Josh Stone