mbox series

[v2,0/4] tracing/probes: Fix bugs in process_fetch_insn

Message ID 168873724526.2687993.15242662075324919195.stgit@mhiramat.roam.corp.google.com (mailing list archive)
Headers show
Series tracing/probes: Fix bugs in process_fetch_insn | expand

Message

Masami Hiramatsu (Google) July 7, 2023, 1:40 p.m. UTC 
Hi,

Here are the 2nd version of fix bugs in process_fetch_insn_*().
The previous version is here;

https://lore.kernel.org/all/168830922841.2278819.9165254236027770818.stgit@mhiramat.roam.corp.google.com/

The first patch [1/4] just updated the description. [2/4] has been
changed because previous one did not consider to store the array data.
So instead of exit, just clear the 'ret' if it has any error code.
[3/4] reverts the patch which did not work. And [4/4] makes each
fetch_store_string*() always updates the data_loc, instead of clearing
it in store_trace_args().

Thank you,

---

Masami Hiramatsu (Google) (4):
      tracing/probes: Fix to avoid double count of the string length on the array
      tracing/probes: Fix not to count error code to total length
      Revert "tracing: Add "(fault)" name injection to kernel probes"
      tracing/probes: Fix to record 0-length data_loc in fetch_store_string*() if fails


 kernel/trace/trace_events_synth.c |    2 +-
 kernel/trace/trace_probe_kernel.h |   29 ++++-------------------------
 kernel/trace/trace_probe_tmpl.h   |   10 +++++-----
 kernel/trace/trace_uprobe.c       |    3 ++-
 4 files changed, 12 insertions(+), 32 deletions(-)

--
Masami Hiramatsu (Google) <mhiramat@kernel.org>