Skip to content

Commit

Permalink
Merge tag 'probes-fixes-v6.6-rc7' of git://git.kernel.org/pub/scm/lin…
Browse files Browse the repository at this point in the history
…ux/kernel/git/trace/linux-trace

Pull probes fixes from Masami Hiramatsu:

 - tracing/kprobes: Fix kernel-doc warnings for the variable length
   arguments

 - tracing/kprobes: Fix to count the symbols in modules even if the
   module name is not specified so that user can probe the symbols in
   the modules without module name

* tag 'probes-fixes-v6.6-rc7' of git://git.kernel.org/pub/scm/linux/kernel/git/trace/linux-trace:
  tracing/kprobes: Fix symbol counting logic by looking at modules as well
  tracing/kprobes: Fix the description of variable length arguments
  • Loading branch information
torvalds committed Oct 28, 2023
2 parents bd80d2e + 926fe78 commit 51a7691
Showing 1 changed file with 22 additions and 6 deletions.
28 changes: 22 additions & 6 deletions kernel/trace/trace_kprobe.c
Original file line number Diff line number Diff line change
Expand Up @@ -714,14 +714,30 @@ static int count_symbols(void *data, unsigned long unused)
return 0;
}

struct sym_count_ctx {
unsigned int count;
const char *name;
};

static int count_mod_symbols(void *data, const char *name, unsigned long unused)
{
struct sym_count_ctx *ctx = data;

if (strcmp(name, ctx->name) == 0)
ctx->count++;

return 0;
}

static unsigned int number_of_same_symbols(char *func_name)
{
unsigned int count;
struct sym_count_ctx ctx = { .count = 0, .name = func_name };

kallsyms_on_each_match_symbol(count_symbols, func_name, &ctx.count);

count = 0;
kallsyms_on_each_match_symbol(count_symbols, func_name, &count);
module_kallsyms_on_each_symbol(NULL, count_mod_symbols, &ctx);

return count;
return ctx.count;
}

static int __trace_kprobe_create(int argc, const char *argv[])
Expand Down Expand Up @@ -1007,7 +1023,7 @@ EXPORT_SYMBOL_GPL(kprobe_event_cmd_init);
* @name: The name of the kprobe event
* @loc: The location of the kprobe event
* @kretprobe: Is this a return probe?
* @args: Variable number of arg (pairs), one pair for each field
* @...: Variable number of arg (pairs), one pair for each field
*
* NOTE: Users normally won't want to call this function directly, but
* rather use the kprobe_event_gen_cmd_start() wrapper, which automatically
Expand Down Expand Up @@ -1080,7 +1096,7 @@ EXPORT_SYMBOL_GPL(__kprobe_event_gen_cmd_start);
/**
* __kprobe_event_add_fields - Add probe fields to a kprobe command from arg list
* @cmd: A pointer to the dynevent_cmd struct representing the new event
* @args: Variable number of arg (pairs), one pair for each field
* @...: Variable number of arg (pairs), one pair for each field
*
* NOTE: Users normally won't want to call this function directly, but
* rather use the kprobe_event_add_fields() wrapper, which
Expand Down

0 comments on commit 51a7691

Please sign in to comment.