linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Ftrace Data Export
@ 2017-05-17  8:08 Felipe Balbi
  2017-05-17 12:45 ` Chunyan Zhang
  0 siblings, 1 reply; 5+ messages in thread
From: Felipe Balbi @ 2017-05-17  8:08 UTC (permalink / raw)
  To: Chunyan Zhang
  Cc: Alexander Shishkin, Steven Rostedt, Ingo Molnar, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 314 bytes --]


Hi Chunyan,

When you wrote your patchset to provide ftrace exports, why did you
choose to export only function trace? Why not tracepoints,
function_graph, hwlat, irqsoff and all the other possibilities?

Is there an underlying reason why you chose to export only function
trace?

Thanks

-- 
balbi

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Ftrace Data Export
  2017-05-17  8:08 Ftrace Data Export Felipe Balbi
@ 2017-05-17 12:45 ` Chunyan Zhang
  2017-06-02 10:24   ` Felipe Balbi
  0 siblings, 1 reply; 5+ messages in thread
From: Chunyan Zhang @ 2017-05-17 12:45 UTC (permalink / raw)
  To: Felipe Balbi
  Cc: Alexander Shishkin, Steven Rostedt, Ingo Molnar, linux-kernel,
	Mathieu Poirier

Hi Felipe,

On 17 May 2017 at 16:08, Felipe Balbi <felipe.balbi@linux.intel.com> wrote:
>
> Hi Chunyan,
>
> When you wrote your patchset to provide ftrace exports, why did you
> choose to export only function trace? Why not tracepoints,

In fact, I tried submitting patches[1] to do exporting tracepoint to
STM, but Ingo and Steven commented that would introduce certain amount
of overhead, and that was not acceptable. I also used
'benchmark_event' to see the additional overhead caused by printing
tracepoint message to STM.  I cannot remember the exact data though,
the increased time consuming indeed was non-ignorable.

So at the end I gave up that idea, and later on switched to the way of
implementation you see in the kernel now.

> function_graph, hwlat, irqsoff and all the other possibilities?

I haven't thought about these clear enough :)
Any suggestion?

Thanks,
Chunyan

[1] https://lkml.org/lkml/2015/7/7/230

>
> Is there an underlying reason why you chose to export only function
> trace?
>
> Thanks
>
> --
> balbi

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Ftrace Data Export
  2017-05-17 12:45 ` Chunyan Zhang
@ 2017-06-02 10:24   ` Felipe Balbi
  2017-06-02 12:22     ` Chunyan Zhang
  0 siblings, 1 reply; 5+ messages in thread
From: Felipe Balbi @ 2017-06-02 10:24 UTC (permalink / raw)
  To: Chunyan Zhang
  Cc: Alexander Shishkin, Steven Rostedt, Ingo Molnar, linux-kernel,
	Mathieu Poirier

[-- Attachment #1: Type: text/plain, Size: 1468 bytes --]


Hi,

(sorry for the long delay, just back from vacations)

Chunyan Zhang <zhang.chunyan@linaro.org> writes:
> Hi Felipe,
>
> On 17 May 2017 at 16:08, Felipe Balbi <felipe.balbi@linux.intel.com> wrote:
>>
>> Hi Chunyan,
>>
>> When you wrote your patchset to provide ftrace exports, why did you
>> choose to export only function trace? Why not tracepoints,
>
> In fact, I tried submitting patches[1] to do exporting tracepoint to
> STM, but Ingo and Steven commented that would introduce certain amount
> of overhead, and that was not acceptable. I also used
> 'benchmark_event' to see the additional overhead caused by printing
> tracepoint message to STM.  I cannot remember the exact data though,
> the increased time consuming indeed was non-ignorable.
>
> So at the end I gave up that idea, and later on switched to the way of
> implementation you see in the kernel now.

Were you decoding the data before off-loading it to the trace export?

Maybe that's why they consider it an extra overhead? Have you considered
off-loading raw data for further post processing?

>> function_graph, hwlat, irqsoff and all the other possibilities?
>
> I haven't thought about these clear enough :)
> Any suggestion?

I think we should be able to export everything and anything :-p But, of
course, we would need tooling to decode it after the fact.

> [1] https://lkml.org/lkml/2015/7/7/230

hmm, lkml.org seems to be down.

-- 
balbi

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Ftrace Data Export
  2017-06-02 10:24   ` Felipe Balbi
@ 2017-06-02 12:22     ` Chunyan Zhang
  2017-06-05  6:17       ` Felipe Balbi
  0 siblings, 1 reply; 5+ messages in thread
From: Chunyan Zhang @ 2017-06-02 12:22 UTC (permalink / raw)
  To: Felipe Balbi
  Cc: Alexander Shishkin, Steven Rostedt, Ingo Molnar, linux-kernel,
	Mathieu Poirier, Arnd Bergmann, Mark Brown

On 2 June 2017 at 18:24, Felipe Balbi <felipe.balbi@linux.intel.com> wrote:
>
> Hi,
>
> (sorry for the long delay, just back from vacations)
>
> Chunyan Zhang <zhang.chunyan@linaro.org> writes:
>> Hi Felipe,
>>
>> On 17 May 2017 at 16:08, Felipe Balbi <felipe.balbi@linux.intel.com> wrote:
>>>
>>> Hi Chunyan,
>>>
>>> When you wrote your patchset to provide ftrace exports, why did you
>>> choose to export only function trace? Why not tracepoints,
>>
>> In fact, I tried submitting patches[1] to do exporting tracepoint to
>> STM, but Ingo and Steven commented that would introduce certain amount
>> of overhead, and that was not acceptable. I also used
>> 'benchmark_event' to see the additional overhead caused by printing
>> tracepoint message to STM.  I cannot remember the exact data though,
>> the increased time consuming indeed was non-ignorable.
>>
>> So at the end I gave up that idea, and later on switched to the way of
>> implementation you see in the kernel now.
>
> Were you decoding the data before off-loading it to the trace export?

Indeed :)

>
> Maybe that's why they consider it an extra overhead? Have you considered
> off-loading raw data for further post processing?

Yes, that's also the way off-loading function trace has been implemented now.
And like you said below, I also believe we can do the similar things
to other tracers.
I'd like to do this, but I have some other tasks in hands recently :-(

>
>>> function_graph, hwlat, irqsoff and all the other possibilities?
>>
>> I haven't thought about these clear enough :)
>> Any suggestion?
>
> I think we should be able to export everything and anything :-p But, of
> course, we would need tooling to decode it after the fact.

Yes, tools for decoding these raw data with kernel binary is one
thing, and how large storage STM can use to collect traces will also
affect how much value doing this will bring in and perhaps will
influence how we implement off-loading ftrace to trace export.

Since I haven't played Intel STM, how large are the storages connected
to STM on Intel platforms in general?

Thanks,
Chunyan

>
>> [1] https://lkml.org/lkml/2015/7/7/230
>
> hmm, lkml.org seems to be down.
>
> --
> balbi

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Ftrace Data Export
  2017-06-02 12:22     ` Chunyan Zhang
@ 2017-06-05  6:17       ` Felipe Balbi
  0 siblings, 0 replies; 5+ messages in thread
From: Felipe Balbi @ 2017-06-05  6:17 UTC (permalink / raw)
  To: Chunyan Zhang
  Cc: Alexander Shishkin, Steven Rostedt, Ingo Molnar, linux-kernel,
	Mathieu Poirier, Arnd Bergmann, Mark Brown

[-- Attachment #1: Type: text/plain, Size: 1289 bytes --]


Hi,

Chunyan Zhang <zhang.chunyan@linaro.org> writes:
>> Maybe that's why they consider it an extra overhead? Have you considered
>> off-loading raw data for further post processing?
>
> Yes, that's also the way off-loading function trace has been implemented now.
> And like you said below, I also believe we can do the similar things
> to other tracers.
> I'd like to do this, but I have some other tasks in hands recently :-(

fair enough

>>>> function_graph, hwlat, irqsoff and all the other possibilities?
>>>
>>> I haven't thought about these clear enough :)
>>> Any suggestion?
>>
>> I think we should be able to export everything and anything :-p But, of
>> course, we would need tooling to decode it after the fact.
>
> Yes, tools for decoding these raw data with kernel binary is one
> thing, and how large storage STM can use to collect traces will also
> affect how much value doing this will bring in and perhaps will
> influence how we implement off-loading ftrace to trace export.
>
> Since I haven't played Intel STM, how large are the storages connected
> to STM on Intel platforms in general?

that I don't know :-) My interest here is to off-load it via USB. I
suppose Alex knows the size of STM storage on Intel systems.

-- 
balbi

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2017-06-05  6:18 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-17  8:08 Ftrace Data Export Felipe Balbi
2017-05-17 12:45 ` Chunyan Zhang
2017-06-02 10:24   ` Felipe Balbi
2017-06-02 12:22     ` Chunyan Zhang
2017-06-05  6:17       ` Felipe Balbi

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).