On Fri, Sep 25, 2015 at 11:11:51AM +0000, He Kuang wrote: SNIP
> > +static int config_term_tracepoint(struct perf_event_attr *attr, > + struct parse_events_term *term, > + struct parse_events_error *err) > +{ > + switch (term->type_term) { > + case PARSE_EVENTS__TERM_TYPE_CALLGRAPH: > + case PARSE_EVENTS__TERM_TYPE_STACKSIZE: > + return config_term_common(attr, term, err); > + default: > + if (err) { > + err->idx = term->err_term; > + err->str = strdup("unknown term"); > + err->help = strdup( > + "valid terms: call-graph,stack-size\n"); please keep that on one line, we dont follow 80 chars rule strictly if it results in ugly formatting like above thanks, jirka -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/