From: "Huang, Ying" <ying.huang@intel.com>
To: Vivek Goyal <vgoyal@redhat.com>
Cc: nigel@nigel.suspend2.net,
Kexec Mailing List <kexec@lists.infradead.org>,
linux-kernel@vger.kernel.org, "Rafael J. Wysocki" <rjw@sisk.pl>,
"Eric W. Biederman" <ebiederm@xmission.com>,
Pavel Machek <pavel@ucw.cz>,
Andrew Morton <akpm@linux-foundation.org>,
linux-pm@lists.linux-foundation.org
Subject: Re: [PATCH -mm] kexec jump -v9
Date: Wed, 28 May 2008 09:35:52 +0800 [thread overview]
Message-ID: <1211938552.12819.122.camel@caritas-dev.intel.com> (raw)
In-Reply-To: <20080527221514.GA20037@redhat.com>
On Tue, 2008-05-27 at 18:15 -0400, Vivek Goyal wrote:
[...]
> > But, because IOAPIC may need to be in original state during
> > suspend/resume, so it is not appropriate to call disable_IO_APIC() in
> > ioapic_suspend(). So I think we can call disable_IO_APIC() in new
> > hibernation/restore callback.
>
> My hunch is suspend/resume will still work if we put this call in
> ioapic_suspend() but I would not recommend that. suspend/resume does
> not need to put IOAPIC in legacy mode.
>
> I am not sure what is "new hibernation/restore callback"? Are you
> referring to new patches from Rafel?
Yes. Rafel has a new patch to separate suspend and hibernation device
call backs.
http://kerneltrap.org/Linux/Separating_Suspend_and_Hibernation
> I think this issue is specifc to kexec and kjump so probably we should
> not tweaking any suspend/resume related bit.
>
> How about calling disable_IO_APIC() in kexec_jump()? We can probably even
> optimize it by calling it only when we are transitioning into new image
> for the first time and not for subsquent transitions (by keeping some kind of
> count in kimage). This is little hackish but, should work...
Yes. This issue is kexec/kjump specific. We can call it in kexec_jump().
Maybe we also need call something other in native_machine_shutdown()?
BTW: I have a new version -v10: http://lkml.org/lkml/2008/5/22/106, do
you have time to review it?
Best Regards,
Huang Ying
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
WARNING: multiple messages have this Message-ID (diff)
From: "Huang, Ying" <ying.huang@intel.com>
To: Vivek Goyal <vgoyal@redhat.com>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>,
Pavel Machek <pavel@ucw.cz>,
nigel@nigel.suspend2.net, "Rafael J. Wysocki" <rjw@sisk.pl>,
Andrew Morton <akpm@linux-foundation.org>,
linux-kernel@vger.kernel.org,
linux-pm@lists.linux-foundation.org,
Kexec Mailing List <kexec@lists.infradead.org>
Subject: Re: [PATCH -mm] kexec jump -v9
Date: Wed, 28 May 2008 09:35:52 +0800 [thread overview]
Message-ID: <1211938552.12819.122.camel@caritas-dev.intel.com> (raw)
In-Reply-To: <20080527221514.GA20037@redhat.com>
On Tue, 2008-05-27 at 18:15 -0400, Vivek Goyal wrote:
[...]
> > But, because IOAPIC may need to be in original state during
> > suspend/resume, so it is not appropriate to call disable_IO_APIC() in
> > ioapic_suspend(). So I think we can call disable_IO_APIC() in new
> > hibernation/restore callback.
>
> My hunch is suspend/resume will still work if we put this call in
> ioapic_suspend() but I would not recommend that. suspend/resume does
> not need to put IOAPIC in legacy mode.
>
> I am not sure what is "new hibernation/restore callback"? Are you
> referring to new patches from Rafel?
Yes. Rafel has a new patch to separate suspend and hibernation device
call backs.
http://kerneltrap.org/Linux/Separating_Suspend_and_Hibernation
> I think this issue is specifc to kexec and kjump so probably we should
> not tweaking any suspend/resume related bit.
>
> How about calling disable_IO_APIC() in kexec_jump()? We can probably even
> optimize it by calling it only when we are transitioning into new image
> for the first time and not for subsquent transitions (by keeping some kind of
> count in kimage). This is little hackish but, should work...
Yes. This issue is kexec/kjump specific. We can call it in kexec_jump().
Maybe we also need call something other in native_machine_shutdown()?
BTW: I have a new version -v10: http://lkml.org/lkml/2008/5/22/106, do
you have time to review it?
Best Regards,
Huang Ying
next prev parent reply other threads:[~2008-05-28 1:34 UTC|newest]
Thread overview: 254+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-06 3:13 [PATCH -mm] kexec jump -v9 Huang, Ying
2008-03-06 3:13 ` Huang, Ying
2008-03-11 21:10 ` Vivek Goyal
2008-03-11 21:10 ` Vivek Goyal
2008-03-11 21:10 ` Vivek Goyal
2008-03-11 21:59 ` Nigel Cunningham
2008-03-11 21:59 ` Nigel Cunningham
2008-03-11 21:59 ` Nigel Cunningham
2008-03-11 23:55 ` Eric W. Biederman
2008-03-11 23:55 ` Eric W. Biederman
2008-03-11 23:55 ` Eric W. Biederman
2008-03-12 0:09 ` david
2008-03-12 0:09 ` david
2008-03-12 0:09 ` david
2008-03-12 2:14 ` Huang, Ying
2008-03-12 2:14 ` Huang, Ying
2008-03-12 18:53 ` Vivek Goyal
2008-03-12 18:53 ` Vivek Goyal
2008-03-12 18:53 ` Vivek Goyal
2008-03-13 0:01 ` Eric W. Biederman
2008-03-13 0:01 ` Eric W. Biederman
2008-03-13 0:01 ` Eric W. Biederman
2008-03-12 2:14 ` Huang, Ying
2008-03-11 22:18 ` Rafael J. Wysocki
2008-03-11 22:18 ` Rafael J. Wysocki
2008-03-12 2:02 ` Eric W. Biederman
2008-03-12 2:02 ` Eric W. Biederman
2008-03-12 2:02 ` Eric W. Biederman
2008-03-12 2:26 ` Huang, Ying
2008-03-12 2:26 ` Huang, Ying
2008-03-12 2:26 ` Huang, Ying
2008-03-11 22:18 ` Rafael J. Wysocki
2008-03-11 23:24 ` Pavel Machek
2008-03-11 23:24 ` Pavel Machek
2008-03-11 23:49 ` Rafael J. Wysocki
2008-03-11 23:49 ` Rafael J. Wysocki
2008-03-11 23:49 ` Rafael J. Wysocki
2008-03-12 1:55 ` Huang, Ying
2008-03-12 1:55 ` Huang, Ying
2008-03-12 15:01 ` [linux-pm] " Alan Stern
2008-03-12 15:01 ` Alan Stern
2008-03-12 21:53 ` Rafael J. Wysocki
2008-03-12 21:53 ` Rafael J. Wysocki
2008-03-13 0:33 ` Eric W. Biederman
2008-03-13 0:33 ` [linux-pm] " Eric W. Biederman
2008-03-13 0:33 ` Eric W. Biederman
2008-03-13 17:03 ` Rafael J. Wysocki
2008-03-13 17:03 ` Rafael J. Wysocki
2008-03-13 23:07 ` Eric W. Biederman
2008-03-13 23:07 ` [linux-pm] " Eric W. Biederman
2008-03-13 23:07 ` Eric W. Biederman
2008-03-14 1:31 ` Rafael J. Wysocki
2008-03-14 1:31 ` [linux-pm] " Rafael J. Wysocki
2008-03-14 1:31 ` Rafael J. Wysocki
2008-03-18 16:56 ` Eric W. Biederman
2008-03-18 23:52 ` Pavel Machek
2008-03-18 23:52 ` [linux-pm] " Pavel Machek
2008-03-18 23:52 ` Pavel Machek
2008-03-19 0:08 ` Rafael J. Wysocki
2008-03-19 0:08 ` [linux-pm] " Rafael J. Wysocki
2008-03-19 0:08 ` Rafael J. Wysocki
2008-03-19 2:33 ` Alan Stern
2008-03-19 2:33 ` [linux-pm] " Alan Stern
2008-03-19 2:33 ` Alan Stern
2008-03-19 3:25 ` Eric W. Biederman
2008-03-19 3:25 ` Eric W. Biederman
2008-03-19 15:01 ` Alan Stern
2008-03-19 15:01 ` [linux-pm] " Alan Stern
2008-03-19 15:01 ` Alan Stern
2008-03-19 19:28 ` Rafael J. Wysocki
2008-03-19 19:28 ` [linux-pm] " Rafael J. Wysocki
2008-03-19 19:28 ` Rafael J. Wysocki
2008-03-20 10:40 ` Pavel Machek
2008-03-20 10:40 ` [linux-pm] " Pavel Machek
2008-03-20 10:40 ` Pavel Machek
2008-03-20 22:45 ` Rafael J. Wysocki
2008-03-20 22:45 ` Rafael J. Wysocki
2008-03-20 23:01 ` Alan Stern
2008-03-20 23:01 ` Alan Stern
2008-03-20 23:22 ` Pavel Machek
2008-03-20 23:22 ` Pavel Machek
2008-03-20 23:40 ` Rafael J. Wysocki
2008-03-20 23:40 ` Rafael J. Wysocki
2008-03-21 0:36 ` Rafael J. Wysocki
2008-03-21 0:36 ` [linux-pm] " Rafael J. Wysocki
2008-03-21 0:36 ` Rafael J. Wysocki
2008-03-21 0:52 ` Alan Stern
2008-03-21 0:52 ` Alan Stern
2008-03-21 22:05 ` Nigel Cunningham
2008-03-21 22:05 ` [linux-pm] " Nigel Cunningham
2008-03-21 22:05 ` Nigel Cunningham
2008-03-22 16:21 ` Pavel Machek
2008-03-22 16:21 ` [linux-pm] " Pavel Machek
2008-03-22 16:21 ` Pavel Machek
2008-03-22 17:45 ` Rafael J. Wysocki
2008-03-22 17:45 ` Rafael J. Wysocki
2008-03-22 20:49 ` Alan Stern
2008-03-22 20:49 ` [linux-pm] " Alan Stern
2008-03-22 20:49 ` Alan Stern
2008-03-22 21:29 ` Rafael J. Wysocki
2008-03-22 21:29 ` [linux-pm] " Rafael J. Wysocki
2008-03-22 21:29 ` Rafael J. Wysocki
2008-05-14 22:38 ` Eric W. Biederman
2008-05-14 22:38 ` [linux-pm] " Eric W. Biederman
2008-05-14 22:38 ` Eric W. Biederman
2008-05-14 23:47 ` Rafael J. Wysocki
2008-05-14 23:47 ` Rafael J. Wysocki
2008-05-15 20:55 ` Eric W. Biederman
2008-05-15 20:55 ` [linux-pm] " Eric W. Biederman
2008-05-15 20:55 ` Eric W. Biederman
2008-05-15 21:20 ` Rafael J. Wysocki
2008-05-15 21:20 ` [linux-pm] " Rafael J. Wysocki
2008-05-15 21:20 ` Rafael J. Wysocki
2008-05-14 23:47 ` Rafael J. Wysocki
2008-03-22 17:45 ` Rafael J. Wysocki
2008-03-21 0:52 ` Alan Stern
2008-03-20 23:40 ` Rafael J. Wysocki
2008-03-20 23:22 ` Pavel Machek
2008-03-20 23:01 ` Alan Stern
2008-03-20 22:45 ` Rafael J. Wysocki
2008-05-14 20:41 ` [linux-pm] " Maxim Levitsky
2008-05-14 20:41 ` Maxim Levitsky
2008-05-14 23:34 ` Eric W. Biederman
2008-05-14 23:34 ` Eric W. Biederman
2008-05-14 23:34 ` Eric W. Biederman
2008-05-14 20:41 ` Maxim Levitsky
2008-03-18 16:56 ` Eric W. Biederman
2008-03-13 17:03 ` Rafael J. Wysocki
2008-03-12 21:53 ` Rafael J. Wysocki
2008-03-12 15:01 ` Alan Stern
2008-03-12 1:55 ` Huang, Ying
2008-03-12 8:57 ` Pavel Machek
2008-03-12 8:57 ` Pavel Machek
2008-03-12 8:57 ` Pavel Machek
2008-03-12 0:00 ` Nigel Cunningham
2008-03-12 0:00 ` Nigel Cunningham
2008-03-12 0:00 ` Nigel Cunningham
2008-03-11 23:24 ` Pavel Machek
2008-03-12 1:45 ` Huang, Ying
2008-03-12 1:45 ` Huang, Ying
2008-03-12 2:17 ` Eric W. Biederman
2008-03-12 2:17 ` Eric W. Biederman
2008-03-12 2:17 ` Eric W. Biederman
2008-03-12 6:54 ` Huang, Ying
2008-03-12 6:54 ` Huang, Ying
2008-03-12 6:54 ` Huang, Ying
2008-03-12 19:37 ` Vivek Goyal
2008-03-12 19:37 ` Vivek Goyal
2008-03-14 8:03 ` Huang, Ying
2008-03-14 8:03 ` Huang, Ying
2008-03-21 19:12 ` Vivek Goyal
2008-03-21 19:12 ` Vivek Goyal
2008-03-21 19:12 ` Vivek Goyal
2008-03-25 7:25 ` Huang, Ying
2008-03-25 7:25 ` Huang, Ying
2008-03-25 7:25 ` Huang, Ying
2008-03-14 8:03 ` Huang, Ying
2008-03-12 19:37 ` Vivek Goyal
2008-03-12 19:47 ` Vivek Goyal
2008-03-12 19:47 ` Vivek Goyal
2008-03-12 19:47 ` Vivek Goyal
2008-03-12 1:45 ` Huang, Ying
2008-04-09 9:34 ` Pavel Machek
2008-04-09 9:34 ` Pavel Machek
2008-04-09 9:34 ` Pavel Machek
2008-04-09 12:30 ` Vivek Goyal
2008-04-09 12:30 ` Vivek Goyal
2008-04-09 12:30 ` Vivek Goyal
2008-05-14 16:03 ` Vivek Goyal
2008-05-14 16:03 ` Vivek Goyal
2008-05-14 17:49 ` Vivek Goyal
2008-05-14 17:49 ` Vivek Goyal
2008-05-14 17:49 ` Vivek Goyal
2008-05-14 16:03 ` Vivek Goyal
2008-05-14 20:52 ` Vivek Goyal
2008-05-14 20:52 ` Vivek Goyal
2008-05-14 20:52 ` Vivek Goyal
2008-05-15 2:32 ` Huang, Ying
2008-05-15 2:32 ` Huang, Ying
2008-05-15 20:09 ` Vivek Goyal
2008-05-15 20:09 ` Vivek Goyal
2008-05-16 1:48 ` Huang, Ying
2008-05-16 1:48 ` Huang, Ying
2008-05-16 1:51 ` Vivek Goyal
2008-05-16 1:51 ` Vivek Goyal
2008-05-16 1:51 ` Vivek Goyal
2008-05-16 2:08 ` Huang, Ying
2008-05-16 2:08 ` Huang, Ying
2008-05-16 2:08 ` Huang, Ying
2008-05-16 12:13 ` Pavel Machek
2008-05-16 12:13 ` Pavel Machek
2008-05-16 12:13 ` Pavel Machek
2008-05-16 1:48 ` Huang, Ying
2008-05-15 20:09 ` Vivek Goyal
2008-05-15 2:32 ` Huang, Ying
2008-05-15 5:41 ` Huang, Ying
2008-05-15 5:41 ` Huang, Ying
2008-05-15 5:41 ` Huang, Ying
2008-05-15 18:42 ` Eric W. Biederman
2008-05-15 18:42 ` Eric W. Biederman
2008-05-15 18:42 ` Eric W. Biederman
2008-05-16 0:51 ` Vivek Goyal
2008-05-16 0:51 ` Vivek Goyal
2008-05-16 0:51 ` Vivek Goyal
2008-05-16 1:35 ` Eric W. Biederman
2008-05-16 1:35 ` Eric W. Biederman
2008-05-16 1:55 ` Huang, Ying
2008-05-16 1:55 ` Huang, Ying
2008-05-16 1:55 ` Huang, Ying
2008-05-16 1:35 ` Eric W. Biederman
2008-05-27 7:27 ` Huang, Ying
2008-05-27 7:27 ` Huang, Ying
2008-05-27 22:15 ` Vivek Goyal
2008-05-27 22:15 ` Vivek Goyal
2008-05-28 1:35 ` Huang, Ying [this message]
2008-05-28 1:35 ` Huang, Ying
2008-05-28 1:35 ` Huang, Ying
2008-05-27 22:15 ` Vivek Goyal
2008-05-27 7:27 ` Huang, Ying
2008-05-14 22:30 ` Eric W. Biederman
2008-05-14 22:30 ` Eric W. Biederman
2008-05-14 23:55 ` Rafael J. Wysocki
2008-05-14 23:55 ` Rafael J. Wysocki
2008-05-15 22:03 ` Eric W. Biederman
2008-05-15 22:03 ` Eric W. Biederman
2008-05-15 22:03 ` Eric W. Biederman
2008-05-15 23:20 ` Rafael J. Wysocki
2008-05-15 23:20 ` Rafael J. Wysocki
2008-05-15 23:20 ` Rafael J. Wysocki
2008-05-16 12:18 ` Pavel Machek
2008-05-16 12:18 ` Pavel Machek
2008-05-16 12:18 ` Pavel Machek
2008-05-16 14:20 ` [linux-pm] " Alan Stern
2008-05-16 14:20 ` Alan Stern
2008-05-16 14:20 ` Alan Stern
2008-05-14 23:55 ` Rafael J. Wysocki
2008-05-15 1:42 ` Huang, Ying
2008-05-15 1:42 ` Huang, Ying
2008-05-15 1:42 ` Huang, Ying
2008-05-15 19:05 ` Rafael J. Wysocki
2008-05-15 19:05 ` Rafael J. Wysocki
2008-05-15 19:05 ` Rafael J. Wysocki
2008-05-15 14:14 ` Alan Stern
2008-05-15 14:14 ` [linux-pm] " Alan Stern
2008-05-15 14:14 ` Alan Stern
2008-05-15 20:48 ` Eric W. Biederman
2008-05-15 20:48 ` Eric W. Biederman
2008-05-15 21:07 ` Alan Stern
2008-05-15 21:07 ` Alan Stern
2008-05-15 21:07 ` [linux-pm] " Alan Stern
2008-05-15 21:07 ` Alan Stern
2008-05-15 20:48 ` Eric W. Biederman
2008-05-14 22:30 ` Eric W. Biederman
-- strict thread matches above, loose matches on Subject: below --
2008-03-06 3:13 Huang, Ying
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=1211938552.12819.122.camel@caritas-dev.intel.com \
--to=ying.huang@intel.com \
--cc=akpm@linux-foundation.org \
--cc=ebiederm@xmission.com \
--cc=kexec@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@lists.linux-foundation.org \
--cc=nigel@nigel.suspend2.net \
--cc=pavel@ucw.cz \
--cc=rjw@sisk.pl \
--cc=vgoyal@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.