From: ebiederm@xmission.com (Eric W. Biederman)
To: Fernando Luis Vazquez Cao <fernando@intellilink.co.jp>
Cc: linux-kernel@vger.kernel.org, fastboot@lists.osdl.org
Subject: Re: [PATCH & RFC] kdump and stack overflows
Date: Mon, 28 Nov 2005 11:29:29 -0700 [thread overview]
Message-ID: <m1hd9wfwxi.fsf@ebiederm.dsl.xmission.com> (raw)
In-Reply-To: <1133200815.2425.98.camel@localhost.localdomain> (Fernando Luis Vazquez Cao's message of "Tue, 29 Nov 2005 03:00:14 +0900")
Fernando Luis Vazquez Cao <fernando@intellilink.co.jp> writes:
> On Mon, 2005-11-28 at 06:39 -0700, Eric W. Biederman wrote:
>> Fernando Luis Vazquez Cao <fernando@intellilink.co.jp> writes:
> Regarding the stack overflow audit of the nmi path, we have the problem
> that both nmi_enter and nmi_exit in do_nmi (see code below) make heavy
> use of "current" indirectly (specially through the kernel preemption
> code).
Ok. I wonder if it would be saner to simply replace the nmi trap
handler on the crash dump path?
>> I believe we have a separate interrupt stack that
>> should help but..
> Yes, when using 4K stacks we have a separate interrupt stack that should
> help, but I am afraid that crash dumping is about being paranoid.
Oh I agree. If we had a private 4K stack for the nmi handler we
would not need to worry about overflow in that case. (baring
nmi happening during nmis) Hmm. Is there anything to keep
us doing something bad in that case?
I guess as long as we don't clear the high bit of port 0x70 we
should be reasonably safe from the nmi firing multiple times.
Eric
next prev parent reply other threads:[~2005-11-28 18:30 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-11-28 13:11 Fernando Luis Vazquez Cao
2005-11-28 13:39 ` Eric W. Biederman
2005-11-28 18:00 ` Fernando Luis Vazquez Cao
2005-11-28 18:29 ` Eric W. Biederman [this message]
2005-11-29 13:27 ` [Fastboot] " Vivek Goyal
2005-11-29 16:46 ` Eric W. Biederman
2005-11-29 15:43 ` Fernando Luis Vazquez Cao
2005-11-30 6:09 ` [Fastboot] " Vivek Goyal
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=m1hd9wfwxi.fsf@ebiederm.dsl.xmission.com \
--to=ebiederm@xmission.com \
--cc=fastboot@lists.osdl.org \
--cc=fernando@intellilink.co.jp \
--cc=linux-kernel@vger.kernel.org \
/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
all inboxes | Powered by JetHome®