Steven Rostedt
Thu, 02 Sep 2010 12:25:59 -0700
On Thu, 2010-09-02 at 14:38 -0400, Steven Rostedt wrote: > I think I will add a: > > void *pevent_get_field_raw(stuct trace_seq *s, > struct event_format *event, > char *field_name, > struct record *record, > int **field_len); > > which will return a pointer into record->data and set the field_len to > the actual size. > > This will be useful for dynamic arrays too, since it is more complex > than just using a field->offset.
I just pushed out the changes and my new example looks like this:
static int
kvm_emulate_insn_handler(struct trace_seq *s, struct record *record,
struct event_format *event, void *context)
{
unsigned char *data;
int len;
int i;
data = pevent_get_field_raw(s, event, "insn", record,
&len, 1);
if (!data)
return -1;
trace_seq_puts(s, "insn: ");
for (i = 0; i < len; i++) {
trace_seq_printf(s, "%s%02x",
i ? "," : "",
data[i]);
}
return 0;
}
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html