Re: [PATCH 01/17] tools lib traceevent: Add state member to struct trace_seq

From: Namhyung Kim
Date: Wed Jan 15 2014 - 00:03:32 EST


On Tue, 14 Jan 2014 21:56:55 -0500, Steven Rostedt wrote:
> On Wed, 15 Jan 2014 11:49:28 +0900
> Namhyung Kim <namhyung@xxxxxxxxxx> wrote:
>> Oh, it looks better. But I'd like to TRACE_SEQ_CHECK() as is for some
>> cases. How about this?
>
> Looks good to me.
>
> Acked-by: Steven Rostedt <rostedt@xxxxxxxxxxx>
>
> I'll try to look at the rest of the patches tomorrow.

Thanks, and I found that trace_seq_destroy() should use
TRACE_SEQ_TRACE_RET instead. Here's an updated patch.

Thanks,
Namhyung