Messages in this thread | | | Date | Tue, 4 Dec 2018 14:41:45 -0500 | From | Steven Rostedt <> | Subject | Re: [PATCH 3/9] tools/lib/traceevent: Install trace-seq.h API header file |
| |
On Tue, 4 Dec 2018 16:47:39 +0900 Namhyung Kim <namhyung@kernel.org> wrote:
> > @@ -302,6 +302,7 @@ install_headers: > > $(call QUIET_INSTALL, headers) \ > > $(call do_install,event-parse.h,$(prefix)/include/traceevent,644); \ > > $(call do_install,event-utils.h,$(prefix)/include/traceevent,644); \ > > + $(call do_install,trace-seq.h,$(prefix)/include/traceevent,644); \ > > $(call do_install,kbuffer.h,$(prefix)/include/traceevent,644) > > Do you still wanna have 'traceevent' directory prefix? I just > sometimes feel a bit annoying to type it. ;-)
I'd still want the separate directory for it. I'll probably have a ftrace.h file added to this for ftrace specific code in the future.
> > Or you can rename it something like 'tep' or 'libtep' - and hopefully > having only single header file to include.. >
Hmm, I wonder if we should just call the directory "trace"?
-- Steve
| |