On Fri, 25 Sep 2020 11:30:06 -0400 (EDT) Mathieu Desnoyers <mathieu.desnoyers@xxxxxxxxxxxx> wrote: > > Anyway, I don't see any issues with the current patch set as is > > (besides the documentation fix, which I already updated locally). And > > will add this to my queue for linux-next. > > The only thing I would change in the documentation is to word this as > "here is a trampoline trick which can be used to work-around rare cases > of tracepoint header circular dependency issues" rather than "always use > this when instrumenting a header". > I rather not have tracepoints in headers. Period! It's not just about circular dependencies, it also bloats the code. -- Steve
