Fix: incorrect support for multi-context
authorMathieu Desnoyers <mathieu.desnoyers@efficios.com>
Mon, 13 May 2013 10:02:55 +0000 (06:02 -0400)
committerMathieu Desnoyers <mathieu.desnoyers@efficios.com>
Mon, 13 May 2013 10:02:55 +0000 (06:02 -0400)
commit83e43212040a26aca40a4f8fefaa3682e9edaa57
tree5dc429c7e756593b616f4c15387a320050e6b343
parenta51ac6d9a011ba10b0be396dc1e801b2fc829651
Fix: incorrect support for multi-context

* Zifei Tong <soariez@gmail.com> wrote:
> I did some debugging one this issue. The problem only occurs when we
> have more than one context field.
> So this will not work, too:
>
> lttng create
> lttng enable-event -a -u
> lttng add-context -u -t vpid
> lttng add-context -u -t vtid
> lttng start
> $@
> lttng stop
> sleep 1
> lttng view
> lttng destroy
>
> The problem I found out is wrong `fields` argument passed into
> `ustcomm_register_channel`.
> The `fields` argument passed is a pointer to the `event_field` of the
> first element in a `lttng_ctx_field` array, but not a
> `lttng_event_field` array as expected.

Fixes #529

Reported-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
include/ust-comm.h
liblttng-ust-comm/lttng-ust-comm.c
liblttng-ust/lttng-events.c
This page took 0.024481 seconds and 4 git commands to generate.