public inbox for systemtap@sourceware.org
 help / color / mirror / Atom feed
From: "fche at redhat dot com" <sourceware-bugzilla@sourceware.org>
To: systemtap@sources.redhat.com
Subject: [Bug runtime/10189] STP_START gets lost in a warning flood
Date: Tue, 17 Nov 2009 17:31:00 -0000	[thread overview]
Message-ID: <20091117173120.17495.qmail@sourceware.org> (raw)
In-Reply-To: <20090522204505.10189.jistone@redhat.com>


------- Additional Comments From fche at redhat dot com  2009-11-17 17:31 -------
(In reply to comment #2)
> Current stap will keep allocating buffer for _stp_warn in probe until
> out of memory.

That's wrong; script-accessible constructs like error()/warn() should
not be able to deplete a dynamically allocated resource.  OTOH in this
case it looks like the transport is just using its own _stp_mempool_alloc,
which at least is a private resource.

> Maybe it's better to make _stp_warn utilize _stp_print instead of
> _stp_ctl_write.

Then we'd have the same problem with error().  The OOB messages do serve a
useful purpose at staprun time, so let's find a way of preserving OOB while
ensuring that the basic protocol can proceed.

For example, we can know that some of these kernel->staprun messages only
occur singly (i.e., are not queued en masse).  So for them, we could
preallocate a mem_buffer element, to avoid having them contend.  One way
would be to have separate mempools for each message type, and ensuring
that the singleton messages only allocate one or two elements total.


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=10189

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

  parent reply	other threads:[~2009-11-17 17:31 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-22 20:45 [Bug runtime/10189] New: " jistone at redhat dot com
2009-11-16  9:47 ` [Bug runtime/10189] " wenji dot huang at oracle dot com
2009-11-17  8:06 ` wenji dot huang at oracle dot com
2009-11-17 17:31 ` fche at redhat dot com [this message]
     [not found] <bug-10189-1110@http.sourceware.org/bugzilla/>
2010-11-08 20:02 ` fche at redhat dot com

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=20091117173120.17495.qmail@sourceware.org \
    --to=sourceware-bugzilla@sourceware.org \
    --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).