Mirror of the lttng-dev mailing list
 help / color / mirror / Atom feed
From: Brian Hutchinson via lttng-dev <lttng-dev@lists.lttng.org>
To: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Cc: lttng-dev@lists.lttng.org
Subject: Re: [lttng-dev] I'm still getting empty ust traces using tracef
Date: Tue, 20 Jun 2023 18:02:38 -0400	[thread overview]
Message-ID: <CAFZh4h-MPXsqHETtYGv2LqMgvt_me33eGSU-UhhZQxmjMEUTig@mail.gmail.com> (raw)
In-Reply-To: <91d1c0ca-af15-2972-da1e-8f07a148158b@efficios.com>

On Thu, May 11, 2023 at 2:14 PM Mathieu Desnoyers
<mathieu.desnoyers@efficios.com> wrote:
>
> On 2023-05-11 14:13, Mathieu Desnoyers via lttng-dev wrote:
> > On 2023-05-11 12:36, Brian Hutchinson via lttng-dev wrote:
> >> ... more background.  I've always used ltt in the kernel so I don't
> >> have much experience with the user side of it and especially
> >> multi-threaded, multi-core so I'm probably missing some fundamental
> >> concepts that I need to understand.
> >
> > Which are the exact versions of LTTng-UST and LTTng-Tools you are using
> > now ? (2.13.N or which git commit ?)
> >
>
> Also, can you try using lttng-ust stable-2.13 branch, which includes the following commit ?
>
> commit be2ca8b563bab81be15cbce7b9f52422369f79f7
> Author: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
> Date:   Tue Feb 21 14:29:49 2023 -0500
>
>      Fix: Reevaluate LTTNG_UST_TRACEPOINT_DEFINE each time tracepoint.h is included
>
>      Fix issues with missing symbols in use-cases where tracef.h is included
>      before defining LTTNG_UST_TRACEPOINT_DEFINE, e.g.:
>
>       #include <lttng/tracef.h>
>       #define LTTNG_UST_TRACEPOINT_DEFINE
>       #include <provider.h>
>
>      It is caused by the fact that tracef.h includes tracepoint.h in a
>      context which has LTTNG_UST_TRACEPOINT_DEFINE undefined, and this is not
>      re-evaluated for the following includes.
>
>      Fix this by lifting the definition code in tracepoint.h outside of the
>      header include guards, and #undef the old LTTNG_UST__DEFINE_TRACEPOINT
>      before re-defining it to its new semantic. Use a new
>      _LTTNG_UST_TRACEPOINT_DEFINE_ONCE include guard within the
>      LTTNG_UST_TRACEPOINT_DEFINE defined case to ensure symbols are not
>      duplicated.
>
>      Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
>      Change-Id: I0ef720435003a7ca0bfcf29d7bf27866c5ff8678
>

I applied this patch and if I use "tracef" type calls in our
application that is made up of a bunch of static libs ... the UST
trace calls work.  I verified that traces that were called from
several different static libs all worked.

But as soon as I include a "tracepoint" style tracepoint (that uses
trace provider include files etc.) then doing a "lttng list -u"
returns "None" for UST events.

Is there some kind of rule that says a file can't use both tracef and
tracepoint calls?  Is there something special you have to do to use
tracef and tracepoints in same file?  Doing so appears to have broken
everything.

Thanks,

Brian
_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

  parent reply	other threads:[~2023-06-20 22:02 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-05-11 15:53 Brian Hutchinson via lttng-dev
2023-05-11 16:36 ` Brian Hutchinson via lttng-dev
2023-05-11 18:13   ` Mathieu Desnoyers via lttng-dev
2023-05-11 18:15     ` Mathieu Desnoyers via lttng-dev
     [not found]       ` <CAFZh4h9M0eA1SpHhfquoyrqtwRhHV54f0O4hCkkiojwUmPTC9Q@mail.gmail.com>
     [not found]         ` <fdbcfbd9-742d-dc61-3667-48a00130d6aa@efficios.com>
2023-05-12 13:34           ` Mathieu Desnoyers via lttng-dev
2023-05-12 14:52           ` Brian Hutchinson via lttng-dev
2023-05-12 14:57             ` Mathieu Desnoyers via lttng-dev
2023-06-20 22:02       ` Brian Hutchinson via lttng-dev [this message]
2023-06-21 20:21         ` Mathieu Desnoyers via lttng-dev
2023-06-21 22:02           ` Brian Hutchinson via lttng-dev
2023-06-28 22:02             ` Brian Hutchinson via lttng-dev
2023-05-11 18:38     ` Brian Hutchinson via lttng-dev
2023-05-11 18:14   ` Michael Jeanson via lttng-dev
2023-05-11 18:42     ` Brian Hutchinson via lttng-dev
2023-05-11 18:57   ` Brian Hutchinson via lttng-dev

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAFZh4h-MPXsqHETtYGv2LqMgvt_me33eGSU-UhhZQxmjMEUTig@mail.gmail.com \
    --to=lttng-dev@lists.lttng.org \
    --cc=b.hutchman@gmail.com \
    --cc=mathieu.desnoyers@efficios.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox