* [PATCH] libtracefs: New API to trace only specific functions by instance [not found] <[PATCH][WIP]libtracefs:New API to enable the filtering of functions> @ 2021-02-22 12:24 ` Sameeruddin shaik 2021-02-22 12:40 ` Sameeruddin Shaik 2021-02-22 14:52 ` Steven Rostedt 0 siblings, 2 replies; 4+ messages in thread From: Sameeruddin shaik @ 2021-02-22 12:24 UTC (permalink / raw) To: rostedt; +Cc: linux-trace-devel, Sameeruddin shaik This new API can be used to set the specific functions to be traced. tracefs_function_filter Signed-off-by: Sameeruddin shaik <sameeruddin.shaik8@gmail.com> diff --git a/include/tracefs-local.h b/include/tracefs-local.h index 187870e..e8ad1f9 100644 --- a/include/tracefs-local.h +++ b/include/tracefs-local.h @@ -20,6 +20,7 @@ void warning(const char *fmt, ...); int str_read_file(const char *file, char **buffer); char *trace_append_file(const char *dir, const char *name); char *trace_find_tracing_dir(void); +int controlled_write(const char *path, const char* const *filter, bool reset); #ifndef ACCESSPERMS #define ACCESSPERMS (S_IRWXU|S_IRWXG|S_IRWXO) /* 0777 */ diff --git a/include/tracefs.h b/include/tracefs.h index f3eec62..37eab87 100644 --- a/include/tracefs.h +++ b/include/tracefs.h @@ -50,6 +50,7 @@ int tracefs_trace_on(struct tracefs_instance *instance); int tracefs_trace_off(struct tracefs_instance *instance); int tracefs_trace_on_fd(int fd); int tracefs_trace_off_fd(int fd); +int tracefs_function_filter(struct tracefs_instance *instance, const char* const *filter, bool reset); /** * tracefs_trace_on_get_fd - Get a file descriptor of "tracing_on" in given instance diff --git a/src/tracefs-tools.c b/src/tracefs-tools.c index e2dfc7b..709f1e7 100644 --- a/src/tracefs-tools.c +++ b/src/tracefs-tools.c @@ -18,6 +18,7 @@ #include "tracefs-local.h" #define TRACE_CTRL "tracing_on" +#define TRACE_FILTER "set_ftrace_filter" static const char * const options_map[] = { "unknown", @@ -387,3 +388,28 @@ void tracefs_option_clear(struct tracefs_options_mask *options, enum tracefs_opt if (options && id > TRACEFS_OPTION_INVALID) options->mask &= ~(1ULL << (id - 1)); } + +/** + * tracefs_function_filter - write to set_ftrace_filter file to trace particular functions + * @instance: ftrace instance, can be NULL for top tracing instnace + * @filter: const pointer to an array of const function names + * @reset: Boolean value to reset the file + */ +int tracefs_function_filter(struct tracefs_instance *instance, const char * const *filters, bool reset) +{ + char *ftrace_filter_path; + int ret; + + if (!filters) + return -1; + ftrace_filter_path = tracefs_instance_get_file(instance, TRACE_FILTER); + + if (!ftrace_filter_path) + goto gracefully_free; + + ret = controlled_write(ftrace_filter_path, filters, reset); + + gracefully_free: + tracefs_put_tracing_file(ftrace_filter_path); + return ret; +} diff --git a/src/tracefs-utils.c b/src/tracefs-utils.c index 0e96f8e..1cd98a2 100644 --- a/src/tracefs-utils.c +++ b/src/tracefs-utils.c @@ -228,3 +228,23 @@ __hidden int str_read_file(const char *file, char **buffer) return size; } + +__hidden int controlled_write(const char *filter_path, const char * const *filters, bool reset) +{ + int size = 0; + int fd; + int i; + + if (!filters) + return -1; + if (reset) + fd = open(filter_path, O_WRONLY | O_APPEND | O_TRUNC); + else + fd = open(filter_path, O_WRONLY | O_APPEND); + if (fd < 0) + return -1; + for (i = 0; filters[i] != NULL ; i++) + size += write(fd, *(filters + i), strlen(*(filters + i))); + close(fd); + return size; +} -- 2.7.4 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] libtracefs: New API to trace only specific functions by instance 2021-02-22 12:24 ` [PATCH] libtracefs: New API to trace only specific functions by instance Sameeruddin shaik @ 2021-02-22 12:40 ` Sameeruddin Shaik 2021-02-21 15:28 ` Steven Rostedt 2021-02-22 14:52 ` Steven Rostedt 1 sibling, 1 reply; 4+ messages in thread From: Sameeruddin Shaik @ 2021-02-22 12:40 UTC (permalink / raw) To: Steven Rostedt; +Cc: Linux Trace Devel hi steve, As you suggested, i have used an array of const pointers to const strings. This is the version-1 of the patch, it breaks in write. I faced two problems here 1. The above code can't be able to write to set_ftrace_filter file string by string (for normal files its working fine), After every successful write i am getting the count bytes written as supplied size, but strings are not getting written in the set_ftrace_filter file. please give me any review or debug inputs on this 2.I don't know how to reply to previous thread using git send-email, i apologize for that, please do let me know how to reply to the older threads using the git send-email. Once i have the write functionality working fine, will implement the other features incrementally. Thanks, sameer. ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] libtracefs: New API to trace only specific functions by instance 2021-02-22 12:40 ` Sameeruddin Shaik @ 2021-02-21 15:28 ` Steven Rostedt 0 siblings, 0 replies; 4+ messages in thread From: Steven Rostedt @ 2021-02-21 15:28 UTC (permalink / raw) To: Sameeruddin Shaik; +Cc: Linux Trace Devel On Mon, 22 Feb 2021 18:10:40 +0530 Sameeruddin Shaik <sameeruddin.shaik8@gmail.com> wrote: > hi steve, > As you suggested, i have used an array of const pointers to const strings. > This is the version-1 of the patch, it breaks in write. > I faced two problems here > 1. The above code can't be able to write to set_ftrace_filter file > string by string (for normal files its working fine), After every > successful write i am getting the count bytes written as supplied > size, but strings are not getting written in the set_ftrace_filter > file. please give me any review or debug inputs on this I'll take a look at your patch tomorrow. > 2.I don't know how to reply to previous thread using git send-email, i > apologize for that, please do let me know how to reply to the older > threads > using the git send-email. Why do you need to reply to the previous thread with git send-email. New versions of a patch should be their own thread and not a reply to another thread. That said, if you look at the headers of the email you want to reply to, you'll see a Message-ID field. For example, your email that I'm replying to has the message id of: Message-ID: <CAK7tX=bRRtcsAPWuiF8i5T2WnO_kSMKY7ar01f3Ki4awAC08rQ@mail.gmail.com> Then I can use the part in between the '<' and '>' to send replies via git send-email. git send-email '--in-reply-to=CAK7tX=bRRtcsAPWuiF8i5T2WnO_kSMKY7ar01f3Ki4awAC08rQ@mail.gmail.com' Note the quotes used above. They may be needed to prevent bash from interpreting anything inside the message ID. And then the emails sent from git will be a reply to that email. > Once i have the write functionality working fine, will implement the > other features incrementally. OK. Again, I'll take a look at your patch on Monday. Thanks, -- Steve ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] libtracefs: New API to trace only specific functions by instance 2021-02-22 12:24 ` [PATCH] libtracefs: New API to trace only specific functions by instance Sameeruddin shaik 2021-02-22 12:40 ` Sameeruddin Shaik @ 2021-02-22 14:52 ` Steven Rostedt 1 sibling, 0 replies; 4+ messages in thread From: Steven Rostedt @ 2021-02-22 14:52 UTC (permalink / raw) To: Sameeruddin shaik; +Cc: linux-trace-devel On Mon, 22 Feb 2021 17:54:21 +0530 Sameeruddin shaik <sameeruddin.shaik8@gmail.com> wrote: > This new API can be used to set the specific functions to be traced. > > tracefs_function_filter > > Signed-off-by: Sameeruddin shaik <sameeruddin.shaik8@gmail.com> > > diff --git a/include/tracefs-local.h b/include/tracefs-local.h > index 187870e..e8ad1f9 100644 > --- a/include/tracefs-local.h > +++ b/include/tracefs-local.h > @@ -20,6 +20,7 @@ void warning(const char *fmt, ...); > int str_read_file(const char *file, char **buffer); > char *trace_append_file(const char *dir, const char *name); > char *trace_find_tracing_dir(void); > +int controlled_write(const char *path, const char* const *filter, bool reset); Note, we've been using the standard that all functions local to the library starts with "trace_" if they are not static. But I'm not sure why this function needs to be in the utilities, where else do you plan on using it? If there's only one user, keep it local to that file. The tracefs-utils.c file should only contain functions that have multiple use cases. > > #ifndef ACCESSPERMS > #define ACCESSPERMS (S_IRWXU|S_IRWXG|S_IRWXO) /* 0777 */ > diff --git a/include/tracefs.h b/include/tracefs.h > index f3eec62..37eab87 100644 > --- a/include/tracefs.h > +++ b/include/tracefs.h > @@ -50,6 +50,7 @@ int tracefs_trace_on(struct tracefs_instance *instance); > int tracefs_trace_off(struct tracefs_instance *instance); > int tracefs_trace_on_fd(int fd); > int tracefs_trace_off_fd(int fd); > +int tracefs_function_filter(struct tracefs_instance *instance, const char* const *filter, bool reset); > > /** > * tracefs_trace_on_get_fd - Get a file descriptor of "tracing_on" in given instance > diff --git a/src/tracefs-tools.c b/src/tracefs-tools.c > index e2dfc7b..709f1e7 100644 > --- a/src/tracefs-tools.c > +++ b/src/tracefs-tools.c > @@ -18,6 +18,7 @@ > #include "tracefs-local.h" > > #define TRACE_CTRL "tracing_on" > +#define TRACE_FILTER "set_ftrace_filter" > > static const char * const options_map[] = { > "unknown", > @@ -387,3 +388,28 @@ void tracefs_option_clear(struct tracefs_options_mask *options, enum tracefs_opt > if (options && id > TRACEFS_OPTION_INVALID) > options->mask &= ~(1ULL << (id - 1)); > } > + > +/** > + * tracefs_function_filter - write to set_ftrace_filter file to trace particular functions > + * @instance: ftrace instance, can be NULL for top tracing instnace typo "instnace". > + * @filter: const pointer to an array of const function names The description should not have its type, we can see that from the prototype. It should be what it is. * @filter: An array of function names or regex expressions, ending with a NULL pointer. > + * @reset: Boolean value to reset the file * @reset: Set to true to reset the filter before applying @filter. then in the body of the description (here), you can explain more. * The @filter is an array of strings, where each string will be use to set * a function or functions to be traced. They can be function names, a glob * expression, or a regex expressions, and all the functions that match * each filter will be enabled. * * If @reset is true, then all functions in the filter are cleared before * adding functions from @filter. Otherwise, the functions set by @filter * will be appended to the filter. Something like the above. That is, explain why things are happening not the what, as people can figure out the what from the code. > + */ > +int tracefs_function_filter(struct tracefs_instance *instance, const char * const *filters, bool reset) > +{ > + char *ftrace_filter_path; > + int ret; > + > + if (!filters) > + return -1; > + ftrace_filter_path = tracefs_instance_get_file(instance, TRACE_FILTER); > + > + if (!ftrace_filter_path) > + goto gracefully_free; > + > + ret = controlled_write(ftrace_filter_path, filters, reset); This can be a helper function for this and perhaps for the set_ftrace_notrace, but it should be local to this file. Especially, since this function should also look at the list of available filter functions as well. In fact, I'm thinking we may not even bother with the glob expressions and do the search of available filter functions instead (if the kernel accepts indexes). > + > + gracefully_free: > + tracefs_put_tracing_file(ftrace_filter_path); > + return ret; > +} > diff --git a/src/tracefs-utils.c b/src/tracefs-utils.c > index 0e96f8e..1cd98a2 100644 > --- a/src/tracefs-utils.c > +++ b/src/tracefs-utils.c > @@ -228,3 +228,23 @@ __hidden int str_read_file(const char *file, char **buffer) > > return size; > } > + > +__hidden int controlled_write(const char *filter_path, const char * const *filters, bool reset) > +{ > + int size = 0; > + int fd; > + int i; > + > + if (!filters) > + return -1; > + if (reset) > + fd = open(filter_path, O_WRONLY | O_APPEND | O_TRUNC); > + else > + fd = open(filter_path, O_WRONLY | O_APPEND); > + if (fd < 0) > + return -1; > + for (i = 0; filters[i] != NULL ; i++) > + size += write(fd, *(filters + i), strlen(*(filters + i))); The kernel expects a delimiter between function names, and it is not broken up by individual writes. You need to add a space before writing in another function name. -- Steve > + close(fd); > + return size; > +} ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-02-22 14:53 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <[PATCH][WIP]libtracefs:New API to enable the filtering of functions> 2021-02-22 12:24 ` [PATCH] libtracefs: New API to trace only specific functions by instance Sameeruddin shaik 2021-02-22 12:40 ` Sameeruddin Shaik 2021-02-21 15:28 ` Steven Rostedt 2021-02-22 14:52 ` Steven Rostedt
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).