On 25 August 2016 at 10:56, Daniel Wagner <daniel.wag...@bmw-carit.de> wrote: > Hi Binoy, > > On 08/24/2016 01:17 PM, Binoy Jayan wrote: >> >> Histogram output: >> cat /sys/kernel/debug/tracing/events/latency/latency_irqs/hist >> cat /sys/kernel/debug/tracing/events/latency/latency_preempt/hist >> cat /sys/kernel/debug/tracing/events/latency/latency_critical_timings/hist >> cat >> /sys/kernel/debug/tracing/events/latency/latency_hrtimer_interrupt/hist > > > [...] > >> Changes from v1 as per comments from Steven/Daniel >> - Use single tracepoint for irq/preempt/critical timings by introducing >> a trace type field to differentiate trace type in the same tracepoint. > > > Did you send out the right patches? This version still looks like the > previous one in this regard. And wouldn't be the 'Histogram output' have > only one file? Maybe I just understood something wrong here. > > cheers, > daniel
Hi Daniel, This patch is after incorporating changes w.r.t. comments by steven. And regarding using one tracepoint, I have mentioned the same in the cover letter. I have sent you (only) another patch with that change. When I do it like that I get an RCU error, the first time the "type" key is used. It is weird that it happens only for the first time something is echo-ed to the trigger file. I haven't been able to figure out why yet. Binoy