From: Andi Kleen <andi@firstfloor.org>
To: Stefani Seibold <stefani@seibold.net>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
linux-mm <linux-mm@kvack.org>,
Peter Zijlstra <a.p.zijlstra@chello.nl>,
Ingo Molnar <mingo@elte.hu>, Joerg Engel <joern@logfs.org>
Subject: Re: Detailed Stack Information Patch [0/3]
Date: Tue, 31 Mar 2009 17:49:05 +0200 [thread overview]
Message-ID: <87eiwdn15a.fsf@basil.nowhere.org> (raw)
In-Reply-To: <1238511498.364.60.camel@matrix> (Stefani Seibold's message of "Tue, 31 Mar 2009 16:58:18 +0200")
Stefani Seibold <stefani@seibold.net> writes:
>
> - Get out of virtual memory by creating a lot of threads
> (f.e. the developer did assign each of them the default size)
The application just fails then? I don't think that needs
a new monitoring tool.
> - Misuse the thread stack for big temporary data buffers
That would be better checked for at compile time
(except for alloca, but that is quite rare)
> - Thread stack overruns
Your method would be racy at best to determine this because
you don't keep track of the worst case, only the current case.
So e.g. if you monitoring app checks once per second the stack
could overflow between your monitoring intervals, but already
have bounced back before the checker comes in.
gcc has support to generate stack overflow checking code,
that would be more reliable. Alternatively you could keep
track of consumption in the VMA that has the stack, but
that can't handle very large jumps (like f() { char x[1<<30]; } )
The later can only be handled well by the compiler.
-Andi
--
ak@linux.intel.com -- Speaking for myself only.
next prev parent reply other threads:[~2009-03-31 15:49 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-03-31 14:58 Detailed Stack Information Patch [0/3] Stefani Seibold
2009-03-31 15:49 ` Andi Kleen [this message]
2009-03-31 18:22 ` Stefani Seibold
2009-03-31 19:02 ` Jörn Engel
2009-03-31 20:30 ` Andi Kleen
2009-03-31 21:25 ` Stefani Seibold
-- strict thread matches above, loose matches on Subject: below --
2009-01-20 10:16 Stefani Seibold
2009-01-22 19:41 ` Jörn Engel
2009-01-22 21:39 ` Stefani Seibold
2009-01-23 9:20 ` Jörn Engel
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=87eiwdn15a.fsf@basil.nowhere.org \
--to=andi@firstfloor.org \
--cc=a.p.zijlstra@chello.nl \
--cc=joern@logfs.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=mingo@elte.hu \
--cc=stefani@seibold.net \
/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 NNTP newsgroup(s).