Mirror of the lttng-dev mailing list
 help / color / mirror / Atom feed
From: capricornalpha2@gmail.com (Anders Carlsson)
Subject: [ltt-dev] UST double-buffering won't always trigger consumer daemon
Date: Wed, 2 Nov 2011 09:19:04 +0100	[thread overview]
Message-ID: <CAM7ttQixitD6eCE9BkY594jU5Pp30Mt0X+6Zaw+k8H-Fi+qNQA@mail.gmail.com> (raw)
Message-ID: <20111102081904.7kAIWIdYqftTRa_fQgJKqBXGLyzU8G9ax7fpCRu8fRw@z> (raw)
In-Reply-To: <20111101171859.GA8645@Krystal>

Thanks Mathieu,

I had to leave work before being able to post a reply myself.

Best regards
Anders

On Tue, Nov 1, 2011 at 6:18 PM, Mathieu Desnoyers <
compudj at krystal.dyndns.org> wrote:

> * Mathieu Desnoyers (compudj at krystal.dyndns.org) wrote:
> > * Anders Carlsson (capricornalpha2 at gmail.com) wrote:
> > > Hello!
> > >
> > > I working on a UST port, but I cannot _reliably_ get trace data trough
> to
> > > the consumer daemon. To me it seems the traced application fails to
> trigger
> > > the consumer daemon when a sub-buffer boundary is crossed.
> >
> > Can you specify which UST version you are working on ?
> >
> > This looks like one of the few design issues that I fixed in the generic
> > ring buffer that I introduce in userspace in UST 2.0.
>
> For everyone's benefit, I received a private reply stating that the
> problem was not in UST upstream, but rather in the specific adaptation
> Anders is working on.
>
> Thanks,
>
> Mathieu
>
> >
> > Thanks,
> >
> > Mathieu
> >
> > >
> > > From buffers.h:
> > >
> > > static __inline__ void ltt_check_deliver(struct ust_channel *chan,
> > >         struct ust_buffer *buf,
> > >         long offset, long commit_count, long idx)
> > > {
> > >     long old_commit_count = commit_count - chan->subbuf_size;
> > >
> > >     /* Check if all commits have been done */
> > >     if (unlikely((BUFFER_TRUNC(offset, chan)
> > >             >> chan->n_subbufs_order)
> > >             - (old_commit_count
> > >                & chan->commit_count_mask) == 0)) {
> > > ...
> > >
> > > This condition becomes true only when commit_count equals a multiple of
> > > chan->subbuf_size, but when I add a printout of the values it appears
> that
> > > the buffer boundary is often crossed without commit_count ever becoming
> > > equal to the subbuffer size. In those cases, the trace ceases to
> operate.
> > > This seems to depend on the various tracepoint data sizes, and the
> order of
> > > commit etc.
> > >
> > > I suppose this could be taken care of by aligning the reservations to
> the
> > > subbuffer boundaries. It seems to me there is already code to handle
> that
> > > (in ltt_relay_try_reserve_slow()) Am I correct?
> > >
> > > Am I missing something (e.g. alignment differences between ARM and
> x86)?
> > >
> > > Should I look somewhere specific?
> > >
> > > Thank you for reading this far. Any help is appreciated.
> > >
> > > Kind regards
> > > Anders
> >
> > > _______________________________________________
> > > ltt-dev mailing list
> > > ltt-dev at lists.casi.polymtl.ca
> > > http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
> >
> >
> > --
> > Mathieu Desnoyers
> > Operating System Efficiency R&D Consultant
> > EfficiOS Inc.
> > http://www.efficios.com
> >
> > _______________________________________________
> > ltt-dev mailing list
> > ltt-dev at lists.casi.polymtl.ca
> > http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
> >
>
> --
> Mathieu Desnoyers
> Operating System Efficiency R&D Consultant
> EfficiOS Inc.
> http://www.efficios.com
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.casi.polymtl.ca/pipermail/lttng-dev/attachments/20111102/2849b03c/attachment-0002.htm>


  reply	other threads:[~2011-11-02  8:19 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CAM7ttQhXvoC01+sSYfCO2RkAht82PrAHmdzj9o4pfzS9vdyo9w@mail.gmail.com>
     [not found] ` <20111031222642.GA32286@Krystal>
2011-11-01 17:18   ` Mathieu Desnoyers
2011-11-02  8:19     ` Anders Carlsson [this message]
2011-11-02  8:19       ` Anders Carlsson
2011-11-02  8:19       ` Anders Carlsson

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=CAM7ttQixitD6eCE9BkY594jU5Pp30Mt0X+6Zaw+k8H-Fi+qNQA@mail.gmail.com \
    --to=capricornalpha2@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