Mirror of the lttng-dev mailing list
 help / color / mirror / Atom feed
From: michael.sills.lavoie@gmail.com (Michael Sills-Lavoie)
Subject: [ltt-dev] lttng-agent and ust issues
Date: Fri, 20 Aug 2010 14:07:55 -0400	[thread overview]
Message-ID: <4C6EC47B.2070908@gmail.com> (raw)
In-Reply-To: <4C6EBE59.3030500@polymtl.ca>

  On 2010-08-20 13:41, Alexandre Montplaisir wrote:
>
>> Hello everyone,
>>
>> [...]
>>
>> In addition to being a counterintuitive way to do things, this is not 
>> the way it
>> works for the kernel provider. This could cause problems for 
>> higher-level tools
>> like the eclipse control plugin, which, I believe, makes no 
>> distinction between
>> a ust trace and a kernel trace.
>
> Afaik, the UI in Eclipse-control shows both "kernel" and "user traces" 
> providers, and they go through different code paths, so the API 
> doesn't necessarily need to be identical for both. Still, as you say, 
> it would make it clear to have them at least similar.
In fact, the Eclipse UI shows both providers, but trust me, there are no 
different code paths in eclipse to control the two different providers. 
It just enumerates the different providers and tries to get information 
about both.

Michael
>
> Good stuff!
>
> Alexandre
>
>> There are already ustcmd functions to alloc and start the trace. 
>> Adding one to
>> only do the setup would help fix the problems with allocTrace,
>> setChannelSubbufNum, setChannelSubbufSize, setupTrace and 
>> writeTraceNetwork. I
>> believe libust is already setup to do this, it just needs to be exposed
>> throught ustcmd.
>>
>> getActiveTraceInfo, getActiveTraces, setChannelEnable, 
>> setChannelOverwrite and
>> getTraces would also need added functionality in ustcmd.
>>
>> I'm not sure about the implications of implementing 
>> stopWriteTraceLocal, as the
>> local tracing is currently done in another process (ustd).
>>
>> setChannelTimer does not set a per-channel timer, but instead uses a 
>> single
>> timer for all channels and for all traces the agent monitors. This 
>> was done as
>> a first step and will probably need to be elaborated a bit.
>>
>> As for the remaining commands, setTraceTransport is not needed, and I 
>> see no
>> obstacle to the implemention of setMarkerEnable.
>>
>> Hopefully these issues can be addressed in the near future, but in 
>> the meantime
>> they are documented in the manual.
>>
>> Alexis
>>
>> _______________________________________________
>> ltt-dev mailing list
>> ltt-dev at lists.casi.polymtl.ca
>> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
>
> _______________________________________________
> ltt-dev mailing list
> ltt-dev at lists.casi.polymtl.ca
> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev




  reply	other threads:[~2010-08-20 18:07 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-20 15:56 Alexis Hallé
2010-08-20 17:41 ` Alexandre Montplaisir
2010-08-20 18:07   ` Michael Sills-Lavoie [this message]
2010-08-24  3:45 ` Pierre-Marc Fournier

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=4C6EC47B.2070908@gmail.com \
    --to=michael.sills.lavoie@gmail.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