Messages in this thread | | | Date | Tue, 3 Oct 2023 15:58:10 -0400 | From | Steven Rostedt <> | Subject | Re: [PATCH v1] dynamic_debug: add support for logs destination |
| |
On Mon, 2 Oct 2023 14:49:20 -0600 jim.cromie@gmail.com wrote:
> hi Lukasz, > > sorry my kernel-time has been in my own trees. > > What I dont understand is why +T is insufficient. > > IIUC, tracefs is intended for production use. > thats why each event can be enabled / disabled > - to select and minimize whats traced, and not impact the system > > and +T can forward all pr_debugs to trace, > (by 1-few trace events defined similarly to others) > or very few, giving yet another selection mechanism > to choose or eliminate specific pr-debugs and reduce traffic to > interesting stuff. > > Once your debug is in the trace-buf, > shouldnt user-space be deciding what to do with it ? > a smart daemon could leverage tracefs to good effect. > > IMO the main value of +T is that it allows feeding existing pr_debugs > into the place where other trace-data is already integrated and managed. > > At this point, I dont see any extra destination handling as prudent. >
I'm fine with either approach. I kind of like the creation of the instance, as that allows the user to keep this debug separate from other tracing going on. We are starting to have multiple applications using the tracing buffer (although most are using instances, which is why I'm trying to make them lighter weight with the eventfs code).
-- Steve
| |