Mirror of the lttng-dev mailing list
 help / color / mirror / Atom feed
From: loic.gelle@polymtl.ca (Loïc Gelle)
Subject: [lttng-dev] RFC: More flexible support for UST event collection in containers
Date: Wed, 01 Aug 2018 17:36:14 +0000	[thread overview]
Message-ID: <20180801173614.Horde.ZvKFlt_4hfXqTXVGAy1Ck_S@www.imp.polymtl.ca> (raw)

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 1499 bytes --]

Hi all,

I am writing a proof of concept for LTTng (ust + tools) for better  
support of application tracing in containers. My point is that we  
should be able to collect all UST events emitted from within  
containers using a single sessiond / consumerd, at least if you want  
to. The main problem to that approach was that the session deamon was  
unable to handle PID clashes when two apps with the same vpid tried to  
register. I fix this by patching the UST client for it to declare both  
its PID and its PID namespace inode number during registration (and I  
also propagated the change to lttng-tools for app registration in the  
hashtable). My patched versions of tools and UST are here, in the  
branch "containers":

https://github.com/loicgelle/lttng-ust
https://github.com/loicgelle/lttng-tools

One problem that I have now is that LTTng-tools currently uses  
/proc/[pid]/cmdline to fetch the name of each application when listing  
the available tracepoints. This is a problem since we have no (simple)  
way to know the PID of the registered apps in the context of the PID  
namespace of sessiond, so this step basically fails. I thought that a  
fix for this would be to ask the applications themselves to declare  
their name using their dedicated socket. Do you think that would be  
possible and relevant?

Also, if you like the idea that I'm pushing, there is more work ahead  
to make it a feature (for example propagating the changes to pid  
tracking).

Thanks,
Loïc.



             reply	other threads:[~2018-08-01 17:36 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-08-01 17:36 Loïc Gelle [this message]
2018-08-27 15:26 ` Loïc Gelle

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=20180801173614.Horde.ZvKFlt_4hfXqTXVGAy1Ck_S@www.imp.polymtl.ca \
    --to=loic.gelle@polymtl.ca \
    /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