From: "Raphaël Beamonte" <raphael.beamonte@gmail.com>
To: lttng-dev@lists.lttng.org, mathieu.desnoyers@efficios.com
Cc: "Raphaël Beamonte" <raphael.beamonte@gmail.com>
Subject: [lttng-ust RFC PATCH v2 0/1] Identify UST threads by changing thread names
Date: Fri, 3 Jun 2016 11:16:59 -0400 [thread overview]
Message-ID: <cover.1464966729.git.raphael.beamonte__35385.8027061962$1464967063$gmane$org@gmail.com> (raw)
In-Reply-To: <cover.1461269886.git.raphael.beamonte@gmail.com>
Hi,
This is the update of the RFC patch following your comments.
This patch updates the pthread_setname_np/pthread_set_name_np version of
the RFC patch, and includes the following changes:
- Check in configure.ac whether the pthread_setname_np or
pthread_set_name_np functions exists. If it's not the case,
current behavior is preserved.
- Update compat.h to define a macro for the pthread_set*name_np
function to use in order to reduce repeated code.
- Update compat.h and lttng-ust-comm.c to use a macro to call the
lttng_ust_setustprocname function in order for that macro to be
empty when the required pthread_setn*ame_np is not available.
- Update compat.h to fix sprintf usage with the same variable to
read from and write to.
- The patch has been rebased on current lttng-ust's master branch.
This patch still contains the link to the pthread library in the
liblttng-ust/Makefile.am file, which still seems required to avoid
compilation warnings about implicit function declaration.
Thoughts and comments?
Thanks!
Raphaël
Raphaël Beamonte (1):
Add -ust to the name of UST threads of the application
configure.ac | 5 +++++
liblttng-ust/Makefile.am | 1 +
liblttng-ust/compat.h | 48 ++++++++++++++++++++++++++++++++++++++++++-
liblttng-ust/lttng-ust-comm.c | 6 ++++++
4 files changed, 59 insertions(+), 1 deletion(-)
--
2.8.1
_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
next prev parent reply other threads:[~2016-06-03 15:17 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <cover.1464966729.git.raphael.beamonte@gmail.com>
[not found] ` <80ee3399800d43b5d3f51aa095ebca160d4f20b9.1464966729.git.raphael.beamonte@gmail.com>
2016-06-03 15:47 ` [lttng-ust RFC PATCH v2 1/1] Add -ust to the name of UST threads of the application Sebastien Boisvert
[not found] ` <5751A690.9000501@gydle.com>
2016-06-03 15:54 ` Mathieu Desnoyers
[not found] ` <1992157209.27136.1464969273436.JavaMail.zimbra@efficios.com>
2016-06-03 15:55 ` Mathieu Desnoyers
[not found] ` <1936873452.27140.1464969327630.JavaMail.zimbra@efficios.com>
2016-06-03 16:12 ` Raphaël Beamonte
[not found] ` <CAE_Gge2Av=iucY8abMCHpU8PjgptdCLhvyoWk7nwfazt6ei3VQ@mail.gmail.com>
2016-06-03 16:19 ` Mathieu Desnoyers
2016-06-03 16:24 ` Sebastien Boisvert
[not found] ` <5751AF3F.4000405@gydle.com>
2016-06-03 16:31 ` Mathieu Desnoyers
[not found] ` <cover.1461269886.git.raphael.beamonte@gmail.com>
2016-04-21 20:50 ` [RFC PATCH 1/2] " Raphaël Beamonte
2016-04-21 20:50 ` [RFC PATCH 2/2] " Raphaël Beamonte
2016-06-01 15:04 ` [RFC PATCH 0/2] Identify UST threads by changing thread names Raphaël Beamonte
[not found] ` <CAE_Gge2jacOMKQaPdp_dXf3NP+yJTS+jWgJQ-SUnv1-B6WgcGw@mail.gmail.com>
2016-06-03 6:10 ` Jérémie Galarneau
[not found] ` <CA+jJMxvmHRYYtHWYjumXrS1BrBCih3WJnRA5yGFBXTw7jL3HiA@mail.gmail.com>
2016-06-03 6:11 ` Jérémie Galarneau
[not found] ` <CA+jJMxs9yNOeeqtYBfGTz1ADOqqiVO=XZd-7qm48BS8dzVo-iw@mail.gmail.com>
2016-06-03 6:18 ` Mathieu Desnoyers
[not found] ` <1192105907.26502.1464934699922.JavaMail.zimbra@efficios.com>
2016-06-03 6:24 ` Mathieu Desnoyers
[not found] ` <293416701.26545.1464935065348.JavaMail.zimbra@efficios.com>
2016-06-03 14:12 ` Raphaël Beamonte
[not found] ` <CAE_Gge2032JkDubp=vpaqNzoQre7MfvQC4UGm-wXPmux-Fwh6g@mail.gmail.com>
2016-06-03 15:38 ` Mathieu Desnoyers
2016-06-03 15:16 ` Raphaël Beamonte [this message]
2016-06-03 15:17 ` [lttng-ust RFC PATCH v2 1/1] Add -ust to the name of UST threads of the application Raphaël Beamonte
2016-06-03 16:45 ` [lttng-ust PATCH v3 0/1] Identify UST threads by changing thread names Raphaël Beamonte
2016-06-03 16:45 ` [lttng-ust PATCH v3 1/1] Add -ust to the name of UST threads of the application Raphaël Beamonte
[not found] <cover.1464971957.git.raphael.beamonte@gmail.com>
[not found] ` <1d3230f091658e56c8ccf12279c3afd89b4062c7.1464971957.git.raphael.beamonte@gmail.com>
2016-06-03 16:51 ` Sebastien Boisvert
2016-06-05 19:19 ` Mathieu Desnoyers
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='cover.1464966729.git.raphael.beamonte__35385.8027061962$1464967063$gmane$org@gmail.com' \
--to=raphael.beamonte@gmail.com \
--cc=lttng-dev@lists.lttng.org \
--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;
as well as URLs for NNTP newsgroup(s).