On Wed, 24 Feb 2016 18:08:58 +0100 (CET)
John Kacur <jkacur@xxxxxxxxxx> wrote:
> On Tue, 23 Feb 2016, Luiz Capitulino wrote:
>
> >
> > In short, this series allows you to run cyclictest under
> > trace-cmd and still get trace marks when the latency
> > specified with -b is execeded. More details in patch 4/4.
> >
> > This series is RFC because I'm not completely sure this
> > is the right thing to do. I'm wondering if we shouldn't
> > ditch all tracing support from cyclictest...
> >
> > Luiz Capitulino (4):
> > cyclictest: tracing(): check for notrace
> > cyclictest: move debugfs init code to its own function
> > cyclictest: move tracemark_fd handling to its own function
> > cyclictest: add --tracemark option
> >
> > src/cyclictest/cyclictest.c | 62 ++++++++++++++++++++++++++++++++++-----------
> > 1 file changed, 47 insertions(+), 15 deletions(-)
> >
> > --
>
> Hi. First of all, although I didn't look closely at the details yet, I
> like this set of patches. Just to let you and everyone else know where we
> are, Clark and I are going to put rt-tests into maintainence mode very
> shortly. We want to create a 1.0 version of it, which we will maintain
> with fixes, but no new features. This last set of patches would be a nice
> addition before we go into mainatinence mode so that the old code base
> would have a way of using trace-cmd.
OK, that's good to know because I have more patches pending but I
don't want to disturb your plans and workflow.
Regarding this series, what I want to achieve is:
1. Use trace-cmd for tracing
2. Get cyclictest to write to the trace_marker file when it
exceeds the -b latency treshold
By reading the code this seems to be impossible to achieve today,
as you need to pass --notrace to use trace-cmd for tracing and
--notrace will disable writing to the trace_maker.
If what I just described is correct, then we need this series.
> For the next development line of rt-tests, we would like to rip-out all of
> the tracing code that we can, and leave the bare minimum in place to
> interact with trace-cmd. With that in mind, could you go over your RFC
> patches, and if you are satisfied that they don't break anything, but
> allow tracing under trace-cmd, then this could be the last new feature we
> let in. Let me know and I'll review at that point. I'll make and official
> announcement about maintainence mode and a new development line in a
> separate mail as well.
OK, I'll do a second pass on this series, test it more, and send
a non-RFC version for inclusion.
I also have a small series with very minor fixups. I don't know
if they are important, but I'll post it shortly.
--
To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
