lkml.org 
[lkml]   [2014]   [Nov]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 3/3] tracing: add additional marks to signal very large delay
On Thu, Nov 13, 2014 at 10:38:01PM -0500, Steven Rostedt wrote:
> On Wed, 5 Nov 2014 16:18:46 +0900
> byungchul.park@lge.com wrote:
>
> > -static unsigned long preempt_mark_thresh_us = 100;
> > +DEFINE_MARK_STRUCT = {
> > + DEFINE_MARK(0ULL , ' '), /* 0 usecs */
> > + DEFINE_MARK(10000ULL , '+'), /* 10 usecs */
> > + DEFINE_MARK(100000ULL , '!'), /* 100 usecs */
> > + DEFINE_MARK(1000000ULL , '#'), /* 1000 usecs */
> > + DEFINE_MARK(1000000000ULL , '$'), /* 1 sec */
> > +};
>
> OK, I see why you did the macro magic here. But it's still obfuscates
> the code.
>
> Instead of having that find_trace_mark() be a static function, just
> make it defined in trace_output.c that function_graph calls. This code
> is used for outputting to userspace and a function call is not going to
> be noticed.

OK, I will make it global function and use it.

>
> As it would now be a global function (although only defined in
> kernel/trace), we need to make the name a bit more specific. Instead of
> calling it find_trace_mark(), call it trace_duration_mark().
>

OK, I'll rename it.

Thank you,
Byungchul.

> -- Steve
>
> >
> > static int
> > lat_print_timestamp(struct trace_iterator *iter, u64 next_ts)
> > @@ -506,8 +512,7 @@ lat_print_timestamp(struct trace_iterator *iter, u64 next_ts)
> > return trace_seq_printf(
> > s, " %4lldus%c: ",
> > abs_ts,
> > - rel_ts > preempt_mark_thresh_us ? '!' :
> > - rel_ts > 1 ? '+' : ' ');
> > + MARK(rel_ts * NSEC_PER_USEC));
> > } else { /* !verbose && !in_ns */
> > return trace_seq_printf(s, " %4lld: ", abs_ts);
> > }
> > @@ -692,7 +697,7 @@ int register_ftrace_event(struct trace_event *event)
> > goto out;
> >
> > } else {
> > -
> > +
> > event->type = next_event_type++;
> > list = &ftrace_event_list;
> > }
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/


\
 
 \ /
  Last update: 2014-11-14 08:01    [W:0.508 / U:0.004 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site