From: ebiederm@xmission.com (Eric W. Biederman)
To: Khalid Aziz <khalid_aziz@hp.com>
Cc: linux-pm@lists.osdl.org,
Fastboot mailing list <fastboot@lists.osdl.org>,
netdev@vger.kernel.org
Subject: Re: [Fastboot] Re: Re: tg3: issue for reboot/kexec
Date: Thu, 18 Aug 2005 12:42:05 -0600 [thread overview]
Message-ID: <m1wtmjulxe.fsf@ebiederm.dsl.xmission.com> (raw)
In-Reply-To: <1124389821.2274.11.camel@lyra.fc.hp.com> (Khalid Aziz's message of "Thu, 18 Aug 2005 12:30:21 -0600")
[-- Attachment #1: Type: text/plain, Size: 1485 bytes --]
Khalid Aziz <khalid_aziz@hp.com> writes:
> On Thu, 2005-06-30 at 17:52 -0600, Eric W. Biederman wrote:
>
> I have found that I can not walk reboot_notifier list in all cases
> before kexec'ing a new kernel. For instance, when handling INIT on ia64,
> we are running in interrupt context and atleast some of the reboot
> notifier callbacks call schedule(). Calling schedule() is not gonna work
> when we are running in interrupt context. I have the same concern for
> when panic gets called in interrupt context. So I added a shutdown
> function to e1000 driver instead. Patch is attached. This patch has
> worked for me.
Thanks. I will look at your e1000 driver and see if it works on
the hardware I have here. You changes are at least to places
that could have fixed the problem I am seeing.
For the panic case calling any of the shutdown/reboot notifier bits
is actually wrong. That kernel that takes over needs to cope with
hardware left in any state.
> As soon as I have all the issues sorted out with kexec'ing on INIT on
> ia64, I will post a fully updated kexec patch for ia64. I now have kexec
> working solid on INIT with e1000 driver and it can handle multiple back
> to back INITs and come up in kexec'd kernel every time. I am now trying
> to sort some issues out with tg3 driver (another driver with no shutdown
> routine :(
In the normal case (not kexec on panic) user space should down the interfaces
before calling kexec. Is your user space doing that?
Eric
[-- Attachment #2: Type: text/plain, Size: 0 bytes --]
next prev parent reply other threads:[~2005-08-18 18:42 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <42BC9F09.1060002@us.ibm.com>
2005-06-25 0:46 ` tg3: issue for reboot/kexec randy_dunlap
2005-06-25 3:33 ` Greg KH
2005-06-30 23:21 ` [Fastboot] " Eric W. Biederman
2005-06-30 23:33 ` Greg KH
2005-06-30 23:52 ` Eric W. Biederman
2005-08-18 18:30 ` Khalid Aziz
2005-08-18 18:35 ` Khalid Aziz
2005-08-18 18:42 ` Eric W. Biederman [this message]
2005-08-18 19:06 ` Khalid Aziz
2005-07-01 1:18 ` Haren Myneni
2005-07-01 1:27 ` Eric W. Biederman
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=m1wtmjulxe.fsf@ebiederm.dsl.xmission.com \
--to=ebiederm@xmission.com \
--cc=fastboot@lists.osdl.org \
--cc=khalid_aziz@hp.com \
--cc=linux-pm@lists.osdl.org \
--cc=netdev@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;
as well as URLs for NNTP newsgroup(s).