On Tue, 4 Dec 2018 16:47:39 +0900
Namhyung Kim <namhy...@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

Reply via email to