lttng-dev.lists.lttng.org archive mirror
 help / color / mirror / Atom feed
* Re: [lttng-dev] lttng-dev Digest, Vol 156, Issue 8
       [not found] <mailman.1.1617811201.6072.lttng-dev@lists.lttng.org>
@ 2021-04-07 16:04 ` Ramesh Errabolu via lttng-dev
  0 siblings, 0 replies; only message in thread
From: Ramesh Errabolu via lttng-dev @ 2021-04-07 16:04 UTC (permalink / raw)
  To: lttng-dev


[-- Attachment #1.1: Type: text/plain, Size: 19277 bytes --]

Is there a dummy test application and libraries one can download and build.
I could then validate my setup against it.

Regards,
Ramesh


On Wed, Apr 7, 2021 at 11:00 AM <lttng-dev-request@lists.lttng.org> wrote:

> Send lttng-dev mailing list submissions to
>         lttng-dev@lists.lttng.org
>
> To subscribe or unsubscribe via the World Wide Web, visit
>         https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
> or, via email, send a message with subject or body 'help' to
>         lttng-dev-request@lists.lttng.org
>
> You can reach the person managing the list at
>         lttng-dev-owner@lists.lttng.org
>
> When replying, please edit your Subject line so it is more specific
> than "Re: Contents of lttng-dev digest..."
>
>
> Today's Topics:
>
>    1. Re: lttng-dev Digest, Vol 156, Issue 3 (Jonathan Rajotte-Julien)
>    2. Re: lttng-dev Digest, Vol 156, Issue 3 (Ramesh Errabolu)
>    3. Re: [PATCH lttng-tools] testapp: gen-ust-events: added help
>       and sync-before-first-event (Mathieu Desnoyers)
>    4. Re: [PATCH lttng-tools] Fix: test code assumes that child
>       process is schedule to run before parent (Mathieu Desnoyers)
>
>
> ----------------------------------------------------------------------
>
> Message: 1
> Date: Tue, 6 Apr 2021 19:46:21 -0400
> From: Jonathan Rajotte-Julien <jonathan.rajotte-julien@efficios.com>
> To: Ramesh Errabolu <ramesh.errabolu@gmail.com>
> Cc: lttng-dev@lists.lttng.org
> Subject: Re: [lttng-dev] lttng-dev Digest, Vol 156, Issue 3
> Message-ID: <20210406234621.GA913141@joraj-alpa>
> Content-Type: text/plain; charset=us-ascii
>
> > I followed your command sequence and noticed a bunch of files being
> > created. However when I tried to run babeltrace on these outputs I don't
> > see any of the functions that should have been called.  Including below
> the
> > search for symbols.
>
> At least userspace tracing seems to work.
>
> I would recommended, again, that you first try tracing on a dummy
> application, a
> really simple one. Then move to the usage of cyg_profile on that dummy app
> then
> to your application.
>
> The cyg profile events are the following:
>
>   lttng_ust_cyg_profile_fast:func_entry
>   lttng_ust_cyg_profile_fast:func_exit
>
> and, when using liblttng-ust-cyg-profile.so:
>
>   lttng_ust_cyg_profile:func_entry
>   lttng_ust_cyg_profile:func_exit
>
> I would recommend that you first start grep-ing (lttng_ust_cyg) for this
> in the
> trace to see if any is getting hit and recorded. If it is not the case,
> take a
> step back try with a dummy app and if nothing works with the dummy app we
> can at
> least try to help you from there and remove all other variable since you
> will be
> able to share the dummy app with us.
>
> Cheers
>
>
> ------------------------------
>
> Message: 2
> Date: Tue, 6 Apr 2021 18:56:39 -0500
> From: Ramesh Errabolu <ramesh.errabolu@gmail.com>
> To: Jonathan Rajotte-Julien <jonathan.rajotte-julien@efficios.com>
> Cc: lttng-dev@lists.lttng.org
> Subject: Re: [lttng-dev] lttng-dev Digest, Vol 156, Issue 3
> Message-ID:
>         <
> CAFGSPrwnihLeY_CQm8jYb9B0R2RT52U2SaPeVLVSDsqCyxYKSQ@mail.gmail.com>
> Content-Type: text/plain; charset="utf-8"
>
> Will try the dummy app to check if the basic setup is good. Wondering if
> being able to trace shared libraries is something not supported.
>
> Regards,
> Ramesh
>
>
> On Tue, Apr 6, 2021 at 6:46 PM Jonathan Rajotte-Julien <
> jonathan.rajotte-julien@efficios.com> wrote:
>
> > > I followed your command sequence and noticed a bunch of files being
> > > created. However when I tried to run babeltrace on these outputs I
> don't
> > > see any of the functions that should have been called.  Including below
> > the
> > > search for symbols.
> >
> > At least userspace tracing seems to work.
> >
> > I would recommended, again, that you first try tracing on a dummy
> > application, a
> > really simple one. Then move to the usage of cyg_profile on that dummy
> app
> > then
> > to your application.
> >
> > The cyg profile events are the following:
> >
> >   lttng_ust_cyg_profile_fast:func_entry
> >   lttng_ust_cyg_profile_fast:func_exit
> >
> > and, when using liblttng-ust-cyg-profile.so:
> >
> >   lttng_ust_cyg_profile:func_entry
> >   lttng_ust_cyg_profile:func_exit
> >
> > I would recommend that you first start grep-ing (lttng_ust_cyg) for this
> > in the
> > trace to see if any is getting hit and recorded. If it is not the case,
> > take a
> > step back try with a dummy app and if nothing works with the dummy app we
> > can at
> > least try to help you from there and remove all other variable since you
> > will be
> > able to share the dummy app with us.
> >
> > Cheers
> >
> -------------- next part --------------
> An HTML attachment was scrubbed...
> URL: <
> https://lists.lttng.org/pipermail/lttng-dev/attachments/20210406/f3c5c020/attachment-0001.htm
> >
>
> ------------------------------
>
> Message: 3
> Date: Wed, 7 Apr 2021 11:29:48 -0400 (EDT)
> From: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
> To: Anders Wallin <wallinux@gmail.com>
> Cc: lttng-dev <lttng-dev@lists.lttng.org>
> Subject: Re: [lttng-dev] [PATCH lttng-tools] testapp: gen-ust-events:
>         added help and sync-before-first-event
> Message-ID:
>         <760733085.60423.1617809388155.JavaMail.zimbra@efficios.com>
> Content-Type: text/plain; charset=utf-8
>
> ----- On Apr 1, 2021, at 12:36 PM, lttng-dev lttng-dev@lists.lttng.org
> wrote:
>
> > - added verbose option to print traces
> > - added sync-before-first-event to wait on file before first event
> > - added help option
> > - added missing short option "-e"
> > - don't allow negative wait times
>
> I don't think this is needed. Will explain in the next patch.
>
> Thanks,
>
> Mathieu
>
> >
> > Signed-off-by: Anders Wallin <wallinux@gmail.com>
> > ---
> > .../testapp/gen-ust-events/gen-ust-events.c   | 95 ++++++++++++++++---
> > 1 file changed, 82 insertions(+), 13 deletions(-)
> >
> > diff --git a/tests/utils/testapp/gen-ust-events/gen-ust-events.c
> > b/tests/utils/testapp/gen-ust-events/gen-ust-events.c
> > index df1e58e4..5fefb2d8 100644
> > --- a/tests/utils/testapp/gen-ust-events/gen-ust-events.c
> > +++ b/tests/utils/testapp/gen-ust-events/gen-ust-events.c
> > @@ -28,6 +28,15 @@
> > #define TRACEPOINT_DEFINE
> > #include "tp.h"
> >
> > +/* #define NDEBUG */
> > +
> > +#ifndef NDEBUG
> > +int verbose;
> > +#define TRACE(format, ...) { if (verbose) printf(" ---TRACE: " format,
> > __VA_ARGS__); }
> > +#else
> > +#define TRACE(format, ...)
> > +#endif
> > +
> > static struct option long_options[] =
> > {
> >       /* These options set a flag. */
> > @@ -38,9 +47,33 @@ static struct option long_options[] =
> >       {"sync-before-last-event-touch", required_argument, 0, 'c'},
> >       {"sync-before-exit", required_argument, 0, 'd'},
> >       {"sync-before-exit-touch", required_argument, 0, 'e'},
> > +     {"sync-before-first-event", required_argument, 0, 'f'},
> > +     {"help", no_argument, 0, 'h'},
> > +#ifndef NDEBUG
> > +     {"verbose", no_argument, 0, 'v'},
> > +#endif
> >       {0, 0, 0, 0}
> > };
> >
> > +const char *program_name;
> > +
> > +static void print_usage(void)
> > +{
> > +     printf("Usage:  %s options\n", program_name);
> > +     printf("  -h  --help                                    Display
> this usage
> > information\n"
> > +#ifndef NDEBUG
> > +            "  -v  --verbose                                 Print
> verbose
> > messages\n"
> > +#endif
> > +            "  -i  --iter [iterations]                       Number of
> iterations,
> > default=100, forever=-1\n"
> > +            "  -w  --wait [usec]                             Time in
> usec between
> > events, default=0\n"
> > +            "  -a  --sync-after-first-event [filename]       Create
> file after
> > first event\n"
> > +            "  -b  --sync-before-last-event [filename]       Wait for
> file before
> > writing last event\n"
> > +            "  -c  --sync-before-last-event-touch [filename] Create
> file after last
> > event\n"
> > +            "  -d  --sync-before-exit [filename]             Wait for
> file before
> > exit\n"
> > +            "  -e  --sync-before-exit-touch [filename]       Create
> file before
> > exit\n"
> > +            "  -f  --sync-before-first-event [filename]      Wait for
> file before
> > creating the first event\n");
> > +}
> > +
> > int main(int argc, char **argv)
> > {
> >       unsigned int i, netint;
> > @@ -54,7 +87,7 @@ int main(int argc, char **argv)
> >       uint32_t net_values[] = { 1, 2, 3 };
> >       int nr_iter = 100, ret = 0, first_event_file_created = 0;
> >       useconds_t nr_usec = 0;
> > -     char *after_first_event_file_path = NULL;
> > +     char *after_first_event_file_path_touch = NULL;
> >       char *before_last_event_file_path = NULL;
> >       /*
> >        * Touch a file to indicate that all events except one were
> > @@ -65,16 +98,20 @@ int main(int argc, char **argv)
> >       char *before_exit_file_path_touch = NULL;
> >       /* Wait on file before exiting */
> >       char *before_exit_file_path = NULL;
> > +     /* Wait on file before starting */
> > +     char *before_first_event_file_path = NULL;
> > +
> > +     program_name = argv[0];
> >
> >       for (i = 0; i < 3; i++) {
> >               net_values[i] = htonl(net_values[i]);
> >       }
> >
> > -     while ((option = getopt_long(argc, argv, "i:w:a:b:c:d:",
> > +     while ((option = getopt_long(argc, argv, "i:w:a:b:c:d:e:f:hv",
> >                       long_options, &option_index)) != -1) {
> >               switch (option) {
> >               case 'a':
> > -                     after_first_event_file_path = strdup(optarg);
> > +                     after_first_event_file_path_touch = strdup(optarg);
> >                       break;
> >               case 'b':
> >                       before_last_event_file_path = strdup(optarg);
> > @@ -88,22 +125,30 @@ int main(int argc, char **argv)
> >               case 'e':
> >                       before_exit_file_path_touch = strdup(optarg);
> >                       break;
> > +             case 'f':
> > +                     before_first_event_file_path = strdup(optarg);
> > +                     break;
> >               case 'i':
> >                       nr_iter = atoi(optarg);
> >                       break;
> > +#ifndef NDEBUG
> > +             case 'v':
> > +                     verbose = 1;
> > +                     break;
> > +#endif
> >               case 'w':
> > -                     nr_usec = atoi(optarg);
> > +                     /* only postive values are valid */
> > +                     nr_usec = (atoi(optarg) < 0) ? 0 : atoi(optarg);
> >                       break;
> > -             case '?':
> > -                     /* getopt_long already printed an error message. */
> > +             case 'h':
> >               default:
> > -                     ret = -1;
> > +                     print_usage();
> >                       goto end;
> >               }
> >       }
> >
> >       if (optind != argc) {
> > -             fprintf(stderr, "Error: takes long options only.\n");
> > +             fprintf(stderr, "Error: takes options only.\n");
> >
> >               /*
> >                * Aborting the test program for now because callers
> typically don't check
> > @@ -113,6 +158,7 @@ int main(int argc, char **argv)
> >                * we should eventually ensure that all scripts test and
> report the test
> >                * app return values.
> >                */
> > +             print_usage();
> >               abort();
> >
> >               ret = -1;
> > @@ -125,9 +171,23 @@ int main(int argc, char **argv)
> >               goto end;
> >       }
> >
> > +     TRACE("iter: %i\n", nr_iter);
> > +     TRACE("wait: %i (usec)\n", nr_usec);
> > +
> > +     if (before_first_event_file_path) {
> > +             TRACE("before_first_event_file_path(%i): %s\n",
> > +                   -1, before_first_event_file_path);
> > +             ret = wait_on_file(before_first_event_file_path);
> > +             if (ret != 0) {
> > +                     goto end;
> > +             }
> > +     }
> > +
> >       for (i = 0; nr_iter < 0 || i < nr_iter; i++) {
> >               if (nr_iter >= 0 && i == nr_iter - 1) {
> >                       if (before_last_event_file_path_touch) {
> > +
>  TRACE("before_last_event_file_path_touch(%i): %s\n",
> > +                                   i,
> before_last_event_file_path_touch);
> >                               ret =
> create_file(before_last_event_file_path_touch);
> >                               if (ret != 0) {
> >                                       goto end;
> > @@ -139,6 +199,8 @@ int main(int argc, char **argv)
> >                        * event.
> >                        */
> >                       if (before_last_event_file_path) {
> > +                             TRACE("before_last_event_file_path(%i):
> %s\n",
> > +                                   i, before_last_event_file_path);
> >                               ret =
> wait_on_file(before_last_event_file_path);
> >                               if (ret != 0) {
> >                                       goto end;
> > @@ -153,9 +215,10 @@ int main(int argc, char **argv)
> >                * First loop we create the file if asked to indicate
> >                * that at least one tracepoint has been hit.
> >                */
> > -             if (after_first_event_file_path &&
> first_event_file_created == 0) {
> > -                     ret = create_file(after_first_event_file_path);
> > -
> > +             if (after_first_event_file_path_touch &&
> first_event_file_created == 0) {
> > +                     TRACE("after_first_event_file_path_touch(%i):
> %s\n",
> > +                           i, after_first_event_file_path_touch);
> > +                     ret =
> create_file(after_first_event_file_path_touch);
> >                       if (ret != 0) {
> >                               goto end;
> >                       } else {
> > @@ -164,7 +227,7 @@ int main(int argc, char **argv)
> >               }
> >
> >               if (nr_usec) {
> > -                     if (usleep_safe(nr_usec)) {
> > +                     if (usleep_safe(nr_usec)) {
> >                               ret = -1;
> >                               goto end;
> >                       }
> > @@ -175,22 +238,28 @@ int main(int argc, char **argv)
> >       }
> >
> >       if (before_exit_file_path_touch) {
> > +             TRACE("before_exit_file_path_touch(%i): %s\n",
> > +                   i, before_exit_file_path_touch);
> >               ret = create_file(before_exit_file_path_touch);
> >               if (ret != 0) {
> >                       goto end;
> >               }
> >       }
> >       if (before_exit_file_path) {
> > +             TRACE("before_exit_file_path(%i): %s\n",
> > +                   i, before_exit_file_path);
> >               ret = wait_on_file(before_exit_file_path);
> >               if (ret != 0) {
> >                       goto end;
> >               }
> >       }
> > end:
> > -     free(after_first_event_file_path);
> > +     TRACE("end status: %i\n", ret);
> > +     free(after_first_event_file_path_touch);
> >       free(before_last_event_file_path);
> >       free(before_last_event_file_path_touch);
> >       free(before_exit_file_path);
> >       free(before_exit_file_path_touch);
> > +     free(before_first_event_file_path);
> >       exit(!ret ? EXIT_SUCCESS : EXIT_FAILURE);
> > }
> > --
> > 2.31.1
> >
> > _______________________________________________
> > lttng-dev mailing list
> > lttng-dev@lists.lttng.org
> > https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
>
> --
> Mathieu Desnoyers
> EfficiOS Inc.
> http://www.efficios.com
>
>
> ------------------------------
>
> Message: 4
> Date: Wed, 7 Apr 2021 11:31:46 -0400 (EDT)
> From: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
> To: Anders Wallin <wallinux@gmail.com>
> Cc: lttng-dev <lttng-dev@lists.lttng.org>
> Subject: Re: [lttng-dev] [PATCH lttng-tools] Fix: test code assumes
>         that child process is schedule to run before parent
> Message-ID:
>         <667805472.60438.1617809506670.JavaMail.zimbra@efficios.com>
> Content-Type: text/plain; charset=utf-8
>
> ----- On Apr 1, 2021, at 12:37 PM, lttng-dev lttng-dev@lists.lttng.org
> wrote:
>
> > the following tests fails
> > - test_event_vpid_tracker ust 0 "${EVENT_NAME}"
> > - test_event_vpid_track_untrack ust 0 "${EVENT_NAME}"
> > - test_event_pid_tracker ust 0 "${EVENT_NAME}"
> > - test_event_pid_track_untrack ust 0 "${EVENT_NAME}"
>
> There is indeed an issue with these tests: there is a missing "untrack all
> pids"
> which should be done at the very beginning of each test to have the
> expected
> behavior. AFAIU this explains why there is a small window where traced
> applications
> can generate events when none are expected.
>
> Fixing this should solve the issues without requiring any addition to the
> test
> program.
>
> Thanks,
>
> Mathieu
>
> >
> > Signed-off-by: Anders Wallin <wallinux@gmail.com>
> > ---
> > tests/regression/tools/tracker/test_event_tracker | 15 ++++++---------
> > 1 file changed, 6 insertions(+), 9 deletions(-)
> >
> > diff --git a/tests/regression/tools/tracker/test_event_tracker
> > b/tests/regression/tools/tracker/test_event_tracker
> > index 711690af..78e9310b 100755
> > --- a/tests/regression/tools/tracker/test_event_tracker
> > +++ b/tests/regression/tools/tracker/test_event_tracker
> > @@ -5,7 +5,7 @@
> > #
> > # SPDX-License-Identifier: GPL-2.0-only
> >
> > -TEST_DESC="LTTng - Event traker test"
> > +TEST_DESC="LTTng - Event tracker test"
> >
> > CURDIR=$(dirname "$0")/
> > TESTDIR="$CURDIR/../../.."
> > @@ -30,27 +30,24 @@ SCRIPT_GROUPNAME="$(id -gn)"
> >
> > CHILD_PID=-1
> > WAIT_PATH=
> > -AFTER_FIRST_PATH=
> > -BEFORE_LAST_PATH=
> > +BEFORE_FIRST_PATH=
> >
> > source $TESTDIR/utils/utils.sh
> >
> > function prepare_ust_app
> > {
> > -     AFTER_FIRST_PATH=$(mktemp -u)
> > -     BEFORE_LAST_PATH=$(mktemp -u)
> > +     BEFORE_FIRST_PATH=$(mktemp -u)
> >
> > -     $TESTAPP_BIN -i $NR_ITER -w $NR_USEC_WAIT -a "$AFTER_FIRST_PATH" -b
> > "$BEFORE_LAST_PATH" &
> > +     $TESTAPP_BIN -i $NR_ITER -w $NR_USEC_WAIT --sync-before-first-event
> > "$BEFORE_FIRST_PATH" &
> >       CHILD_PID=$!
> > }
> >
> > function trace_ust_app
> > {
> > -     touch "$BEFORE_LAST_PATH"
> > +     touch "$BEFORE_FIRST_PATH"
> >       wait
> >       ok $? "Traced application stopped."
> > -     rm "$BEFORE_LAST_PATH"
> > -     rm "$AFTER_FIRST_PATH"
> > +     rm "$BEFORE_FIRST_PATH"
> > }
> >
> > function prepare_kernel_app
> > --
> > 2.31.1
> >
> > _______________________________________________
> > lttng-dev mailing list
> > lttng-dev@lists.lttng.org
> > https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
>
> --
> Mathieu Desnoyers
> EfficiOS Inc.
> http://www.efficios.com
>
>
> ------------------------------
>
> Subject: Digest Footer
>
> _______________________________________________
> lttng-dev mailing list
> lttng-dev@lists.lttng.org
> https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
>
>
> ------------------------------
>
> End of lttng-dev Digest, Vol 156, Issue 8
> *****************************************
>

[-- Attachment #1.2: Type: text/html, Size: 27266 bytes --]

[-- Attachment #2: Type: text/plain, Size: 156 bytes --]

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

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2021-04-07 16:05 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <mailman.1.1617811201.6072.lttng-dev@lists.lttng.org>
2021-04-07 16:04 ` [lttng-dev] lttng-dev Digest, Vol 156, Issue 8 Ramesh Errabolu via lttng-dev

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).