Commit c9bd7796 authored by Tzvetomir Stoyanov's avatar Tzvetomir Stoyanov Committed by Arnaldo Carvalho de Melo

tools lib traceevent: Rename input arguments and local variables of...

tools lib traceevent: Rename input arguments and local variables of libtraceevent from pevent to tep

"pevent" to "tep" renaming of:
 - all "pevent" input arguments of libtraceevent internal functions.
 - all local "pevent" variables of libtraceevent.

This makes the implementation consistent with the chosen naming
convention, tep (trace event parser), and will avoid any confusion with
the old pevent name
Signed-off-by: default avatarTzvetomir Stoyanov <tstoyanov@vmware.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Link: http://lore.kernel.org/linux-trace-devel/20190401132111.13727-5-tstoyanov@vmware.com
Link: http://lkml.kernel.org/r/20190401164344.944953447@goodmis.orgSigned-off-by: default avatarSteven Rostedt (VMware) <rostedt@goodmis.org>
Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 6b1f4c42
This diff is collapsed.
...@@ -269,7 +269,7 @@ void tep_print_plugins(struct trace_seq *s, ...@@ -269,7 +269,7 @@ void tep_print_plugins(struct trace_seq *s,
} }
static void static void
load_plugin(struct tep_handle *pevent, const char *path, load_plugin(struct tep_handle *tep, const char *path,
const char *file, void *data) const char *file, void *data)
{ {
struct tep_plugin_list **plugin_list = data; struct tep_plugin_list **plugin_list = data;
...@@ -316,7 +316,7 @@ load_plugin(struct tep_handle *pevent, const char *path, ...@@ -316,7 +316,7 @@ load_plugin(struct tep_handle *pevent, const char *path,
*plugin_list = list; *plugin_list = list;
pr_stat("registering plugin: %s", plugin); pr_stat("registering plugin: %s", plugin);
func(pevent); func(tep);
return; return;
out_free: out_free:
...@@ -324,9 +324,9 @@ load_plugin(struct tep_handle *pevent, const char *path, ...@@ -324,9 +324,9 @@ load_plugin(struct tep_handle *pevent, const char *path,
} }
static void static void
load_plugins_dir(struct tep_handle *pevent, const char *suffix, load_plugins_dir(struct tep_handle *tep, const char *suffix,
const char *path, const char *path,
void (*load_plugin)(struct tep_handle *pevent, void (*load_plugin)(struct tep_handle *tep,
const char *path, const char *path,
const char *name, const char *name,
void *data), void *data),
...@@ -359,15 +359,15 @@ load_plugins_dir(struct tep_handle *pevent, const char *suffix, ...@@ -359,15 +359,15 @@ load_plugins_dir(struct tep_handle *pevent, const char *suffix,
if (strcmp(name + (strlen(name) - strlen(suffix)), suffix) != 0) if (strcmp(name + (strlen(name) - strlen(suffix)), suffix) != 0)
continue; continue;
load_plugin(pevent, path, name, data); load_plugin(tep, path, name, data);
} }
closedir(dir); closedir(dir);
} }
static void static void
load_plugins(struct tep_handle *pevent, const char *suffix, load_plugins(struct tep_handle *tep, const char *suffix,
void (*load_plugin)(struct tep_handle *pevent, void (*load_plugin)(struct tep_handle *tep,
const char *path, const char *path,
const char *name, const char *name,
void *data), void *data),
...@@ -378,7 +378,7 @@ load_plugins(struct tep_handle *pevent, const char *suffix, ...@@ -378,7 +378,7 @@ load_plugins(struct tep_handle *pevent, const char *suffix,
char *envdir; char *envdir;
int ret; int ret;
if (pevent->flags & TEP_DISABLE_PLUGINS) if (tep->flags & TEP_DISABLE_PLUGINS)
return; return;
/* /*
...@@ -386,8 +386,8 @@ load_plugins(struct tep_handle *pevent, const char *suffix, ...@@ -386,8 +386,8 @@ load_plugins(struct tep_handle *pevent, const char *suffix,
* check that first. * check that first.
*/ */
#ifdef PLUGIN_DIR #ifdef PLUGIN_DIR
if (!(pevent->flags & TEP_DISABLE_SYS_PLUGINS)) if (!(tep->flags & TEP_DISABLE_SYS_PLUGINS))
load_plugins_dir(pevent, suffix, PLUGIN_DIR, load_plugins_dir(tep, suffix, PLUGIN_DIR,
load_plugin, data); load_plugin, data);
#endif #endif
...@@ -397,7 +397,7 @@ load_plugins(struct tep_handle *pevent, const char *suffix, ...@@ -397,7 +397,7 @@ load_plugins(struct tep_handle *pevent, const char *suffix,
*/ */
envdir = getenv("TRACEEVENT_PLUGIN_DIR"); envdir = getenv("TRACEEVENT_PLUGIN_DIR");
if (envdir) if (envdir)
load_plugins_dir(pevent, suffix, envdir, load_plugin, data); load_plugins_dir(tep, suffix, envdir, load_plugin, data);
/* /*
* Now let the home directory override the environment * Now let the home directory override the environment
...@@ -413,7 +413,7 @@ load_plugins(struct tep_handle *pevent, const char *suffix, ...@@ -413,7 +413,7 @@ load_plugins(struct tep_handle *pevent, const char *suffix,
return; return;
} }
load_plugins_dir(pevent, suffix, path, load_plugin, data); load_plugins_dir(tep, suffix, path, load_plugin, data);
free(path); free(path);
} }
......
...@@ -256,7 +256,7 @@ static int event_match(struct tep_event *event, ...@@ -256,7 +256,7 @@ static int event_match(struct tep_event *event,
} }
static enum tep_errno static enum tep_errno
find_event(struct tep_handle *pevent, struct event_list **events, find_event(struct tep_handle *tep, struct event_list **events,
char *sys_name, char *event_name) char *sys_name, char *event_name)
{ {
struct tep_event *event; struct tep_event *event;
...@@ -299,8 +299,8 @@ find_event(struct tep_handle *pevent, struct event_list **events, ...@@ -299,8 +299,8 @@ find_event(struct tep_handle *pevent, struct event_list **events,
} }
} }
for (i = 0; i < pevent->nr_events; i++) { for (i = 0; i < tep->nr_events; i++) {
event = pevent->events[i]; event = tep->events[i];
if (event_match(event, sys_name ? &sreg : NULL, &ereg)) { if (event_match(event, sys_name ? &sreg : NULL, &ereg)) {
match = 1; match = 1;
if (add_event(events, event) < 0) { if (add_event(events, event) < 0) {
...@@ -1257,7 +1257,7 @@ static void filter_init_error_buf(struct tep_event_filter *filter) ...@@ -1257,7 +1257,7 @@ static void filter_init_error_buf(struct tep_event_filter *filter)
enum tep_errno tep_filter_add_filter_str(struct tep_event_filter *filter, enum tep_errno tep_filter_add_filter_str(struct tep_event_filter *filter,
const char *filter_str) const char *filter_str)
{ {
struct tep_handle *pevent = filter->tep; struct tep_handle *tep = filter->tep;
struct event_list *event; struct event_list *event;
struct event_list *events = NULL; struct event_list *events = NULL;
const char *filter_start; const char *filter_start;
...@@ -1313,7 +1313,7 @@ enum tep_errno tep_filter_add_filter_str(struct tep_event_filter *filter, ...@@ -1313,7 +1313,7 @@ enum tep_errno tep_filter_add_filter_str(struct tep_event_filter *filter,
} }
/* Find this event */ /* Find this event */
ret = find_event(pevent, &events, strim(sys_name), strim(event_name)); ret = find_event(tep, &events, strim(sys_name), strim(event_name));
if (ret < 0) { if (ret < 0) {
free_events(events); free_events(events);
free(this_event); free(this_event);
...@@ -1334,7 +1334,7 @@ enum tep_errno tep_filter_add_filter_str(struct tep_event_filter *filter, ...@@ -1334,7 +1334,7 @@ enum tep_errno tep_filter_add_filter_str(struct tep_event_filter *filter,
if (ret < 0) if (ret < 0)
rtn = ret; rtn = ret;
if (ret >= 0 && pevent->test_filters) { if (ret >= 0 && tep->test_filters) {
char *test; char *test;
test = tep_filter_make_string(filter, event->event->id); test = tep_filter_make_string(filter, event->event->id);
if (test) { if (test) {
...@@ -1459,7 +1459,7 @@ static int copy_filter_type(struct tep_event_filter *filter, ...@@ -1459,7 +1459,7 @@ static int copy_filter_type(struct tep_event_filter *filter,
const char *name; const char *name;
char *str; char *str;
/* Can't assume that the pevent's are the same */ /* Can't assume that the tep's are the same */
sys = filter_type->event->system; sys = filter_type->event->system;
name = filter_type->event->name; name = filter_type->event->name;
event = tep_find_event_by_name(filter->tep, sys, name); event = tep_find_event_by_name(filter->tep, sys, name);
...@@ -1697,7 +1697,7 @@ static int test_num(struct tep_event *event, struct tep_filter_arg *arg, ...@@ -1697,7 +1697,7 @@ static int test_num(struct tep_event *event, struct tep_filter_arg *arg,
static const char *get_field_str(struct tep_filter_arg *arg, struct tep_record *record) static const char *get_field_str(struct tep_filter_arg *arg, struct tep_record *record)
{ {
struct tep_event *event; struct tep_event *event;
struct tep_handle *pevent; struct tep_handle *tep;
unsigned long long addr; unsigned long long addr;
const char *val = NULL; const char *val = NULL;
unsigned int size; unsigned int size;
...@@ -1727,12 +1727,12 @@ static const char *get_field_str(struct tep_filter_arg *arg, struct tep_record * ...@@ -1727,12 +1727,12 @@ static const char *get_field_str(struct tep_filter_arg *arg, struct tep_record *
} else { } else {
event = arg->str.field->event; event = arg->str.field->event;
pevent = event->tep; tep = event->tep;
addr = get_value(event, arg->str.field, record); addr = get_value(event, arg->str.field, record);
if (arg->str.field->flags & (TEP_FIELD_IS_POINTER | TEP_FIELD_IS_LONG)) if (arg->str.field->flags & (TEP_FIELD_IS_POINTER | TEP_FIELD_IS_LONG))
/* convert to a kernel symbol */ /* convert to a kernel symbol */
val = tep_find_function(pevent, addr); val = tep_find_function(tep, addr);
if (val == NULL) { if (val == NULL) {
/* just use the hex of the string name */ /* just use the hex of the string name */
...@@ -1872,7 +1872,7 @@ int tep_event_filtered(struct tep_event_filter *filter, int event_id) ...@@ -1872,7 +1872,7 @@ int tep_event_filtered(struct tep_event_filter *filter, int event_id)
enum tep_errno tep_filter_match(struct tep_event_filter *filter, enum tep_errno tep_filter_match(struct tep_event_filter *filter,
struct tep_record *record) struct tep_record *record)
{ {
struct tep_handle *pevent = filter->tep; struct tep_handle *tep = filter->tep;
struct tep_filter_type *filter_type; struct tep_filter_type *filter_type;
int event_id; int event_id;
int ret; int ret;
...@@ -1883,7 +1883,7 @@ enum tep_errno tep_filter_match(struct tep_event_filter *filter, ...@@ -1883,7 +1883,7 @@ enum tep_errno tep_filter_match(struct tep_event_filter *filter,
if (!filter->filters) if (!filter->filters)
return TEP_ERRNO__NO_FILTER; return TEP_ERRNO__NO_FILTER;
event_id = tep_data_type(pevent, record); event_id = tep_data_type(tep, record);
filter_type = find_filter_type(filter, event_id); filter_type = find_filter_type(filter, event_id);
if (!filter_type) if (!filter_type)
......
...@@ -126,7 +126,7 @@ static int add_and_get_index(const char *parent, const char *child, int cpu) ...@@ -126,7 +126,7 @@ static int add_and_get_index(const char *parent, const char *child, int cpu)
static int function_handler(struct trace_seq *s, struct tep_record *record, static int function_handler(struct trace_seq *s, struct tep_record *record,
struct tep_event *event, void *context) struct tep_event *event, void *context)
{ {
struct tep_handle *pevent = event->tep; struct tep_handle *tep = event->tep;
unsigned long long function; unsigned long long function;
unsigned long long pfunction; unsigned long long pfunction;
const char *func; const char *func;
...@@ -136,12 +136,12 @@ static int function_handler(struct trace_seq *s, struct tep_record *record, ...@@ -136,12 +136,12 @@ static int function_handler(struct trace_seq *s, struct tep_record *record,
if (tep_get_field_val(s, event, "ip", record, &function, 1)) if (tep_get_field_val(s, event, "ip", record, &function, 1))
return trace_seq_putc(s, '!'); return trace_seq_putc(s, '!');
func = tep_find_function(pevent, function); func = tep_find_function(tep, function);
if (tep_get_field_val(s, event, "parent_ip", record, &pfunction, 1)) if (tep_get_field_val(s, event, "parent_ip", record, &pfunction, 1))
return trace_seq_putc(s, '!'); return trace_seq_putc(s, '!');
parent = tep_find_function(pevent, pfunction); parent = tep_find_function(tep, pfunction);
if (parent && ftrace_indent->set) if (parent && ftrace_indent->set)
index = add_and_get_index(parent, func, record->cpu); index = add_and_get_index(parent, func, record->cpu);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment