public inbox for systemtap@sourceware.org
 help / color / mirror / Atom feed
From: "Frank Ch. Eigler" <fche@redhat.com>
To: Martin Hunt <hunt@redhat.com>
Cc: systemtap@sources.redhat.com
Subject: Re: offline elfutils processing committed
Date: Mon, 06 Nov 2006 22:15:00 -0000	[thread overview]
Message-ID: <200611062152.kA6LqC82020963@ton.toronto.redhat.com> (raw)
In-Reply-To: <1162847898.2797.42.camel@dragon>


hunt wrote:

> [...]
> > What would be the harm in that?  IOW, what kind of concurrency problem
> > is the lock intended to prevent?
> 
> Module loads (and eventually unloads) will update the list of modules
> and their symbols. This cannot happen while a kprobe is looking through
> the list resolving some symbol.

OTOH if a module load is in progress, kprobes would be locked out.
Have you characterized how long such a duration could be?  (At last
it's mostly moot once the relocation call is taken out of its current
homestead in the $variable fetcher functions.)

> > > Regarding OOM killer, shouldn't we just set oom_adj to something
> > > like +15?  That would put us at the top of the victims list [...]
> > 
> > Who is that "us"?  The staprun process?  It's too late by then [...]
> Yeah, "us" means staprun and the systemtap module.
> 
> The point is damage control.  Systemtap allocates too much memory and
> oom killer gets active, the first thing it will kill is staprun and that
> should unload the module [...]

But it's too late by then.  Most or all the memory allocation attempts
take place in one big lump during one short interval, between checks
for the continued existence of the staprun process.  So if the
attempts are too aggressive, sure staprun will be zapped, but so will
many other programs, by the time the probe module would notice.

- FChE

  reply	other threads:[~2006-11-06 21:52 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-11-01  4:48 Frank Ch. Eigler
2006-11-01 20:37 ` Roland McGrath
2006-11-04  3:29   ` Frank Ch. Eigler
2006-11-04  4:05     ` Roland McGrath
2006-11-04 14:27       ` Frank Ch. Eigler
2006-11-06 20:55     ` Martin Hunt
2006-11-06 21:18       ` Frank Ch. Eigler
2006-11-06 21:52         ` Martin Hunt
2006-11-06 22:15           ` Frank Ch. Eigler [this message]
2006-11-01 22:22 Stone, Joshua I
2006-11-01 22:57 ` Frank Ch. Eigler
2006-11-02 16:42 Stone, Joshua I
2006-11-06 22:41 Stone, Joshua I
2006-11-07  4:37 ` Roland McGrath
2006-11-07  6:36 ` Martin Hunt
2006-11-07 15:13   ` Frank Ch. Eigler
2006-11-07 15:43     ` Martin Hunt
2006-11-07 16:17       ` Frank Ch. Eigler
2006-11-07 16:39         ` Martin Hunt
2006-11-08  1:26           ` 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=200611062152.kA6LqC82020963@ton.toronto.redhat.com \
    --to=fche@redhat.com \
    --cc=hunt@redhat.com \
    --cc=systemtap@sources.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).