public inbox for systemtap@sourceware.org
 help / color / mirror / Atom feed
From: Mike Mason <mmlnx@us.ibm.com>
To: William Cohen <wcohen@redhat.com>
Cc: Sanket Somnath Hase <sanket@cmu.edu>, systemtap@sourceware.org
Subject: Re: stap -x option
Date: Sat, 28 Jul 2007 21:22:00 -0000	[thread overview]
Message-ID: <46AB8114.6090304@us.ibm.com> (raw)
In-Reply-To: <46AA594A.2010005@redhat.com>

William Cohen wrote:

[snip]

> 
> There are two issues with multiple probes. One is there will be 
> additional overhead if there are more probes to run at a probe point. 
> The other issue is that systemtap avoids recompiling the scripts when 
> possible. It checks to see if a kernel module has been previously 
> generated for a script, if so it reuses the module. If you attempt to 
> use multiple instances of a script, systemtap may indicate that it can't 
> load the module. The kernel only allow one instance of a module to be 
> loaded into the kernel. You may need to resort to using the "-m" option 
> to produce different modules to load.

While I think it's best to avoid running multiple instances of the same
script, it should still be allowed.  I ran into this yesterday when
I wanted to test the impact of running multiple instances of a script
I wrote.  Using -m to accomplish this is cumbersome.  Could we simply 
add an option that tells stap to ignore the cache?  It's simple, as 
the following diff shows:

Index: main.cxx
===================================================================
RCS file: /cvs/systemtap/src/main.cxx,v
retrieving revision 1.74
diff -r1.74 main.cxx
102a103
>     << "   -d         disable cache support" << endl
278c279
<       int grc = getopt (argc, argv, "hVMvtp:I:e:o:R:r:m:kgPc:x:D:bs:uq");
---
>       int grc = getopt (argc, argv, "hVMvtp:I:e:o:R:r:m:kgPc:x:D:bs:uqd");
311a313,316
>         case 'd':
>           s.use_cache = false;
>           break;
>

Then, if you run into module name conflicts, simply run stap with -d.

Mike

> 
> -Will

  parent reply	other threads:[~2007-07-28 17:47 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-27 18:23 Sanket Somnath Hase
2007-07-27 19:14 ` Martin Hunt
2007-07-28 17:47   ` Mike Mason
2007-07-28  0:05 ` William Cohen
2007-07-28  3:50   ` Sanket Somnath Hase
2007-07-28 21:22   ` Mike Mason [this message]
2007-07-28 21:46     ` Frank Ch. Eigler

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=46AB8114.6090304@us.ibm.com \
    --to=mmlnx@us.ibm.com \
    --cc=sanket@cmu.edu \
    --cc=systemtap@sourceware.org \
    --cc=wcohen@redhat.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 read-only IMAP folder(s) and NNTP newsgroup(s).