From: Andrew Cooper <andrew.cooper3@citrix.com>
To: Jan Beulich <JBeulich@suse.com>
Cc: "Keir (Xen.org)" <keir@xen.org>,
"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
Subject: Re: [PATCH] kexec/noreboot: Don't kexec_crash() if noreboot has been requested.
Date: Fri, 7 Sep 2012 14:28:30 +0100 [thread overview]
Message-ID: <5049F67E.3080709@citrix.com> (raw)
In-Reply-To: <504A0FDA0200007800099D10@nat28.tlf.novell.com>
On 07/09/12 14:16, Jan Beulich wrote:
>>>> On 07.09.12 at 15:05, Andrew Cooper <andrew.cooper3@citrix.com> wrote:
>> This issue came up when debugging pcpu linked list corruption (patches
>> for that issue to follow).
> Hmm, that's a matter of taste of course. Nor is running kdump
> really a (direct) reboot action. Both ways have their reasoning
> imo, so I'm not sure whether keeping things as is or applying the
> patch is the better thing.
>
> Jan
Hmm yes - I guess its not as clean cut as I thought, but I would still
argue that this patch is the lest surprising alternative.
One tweak I am thinking of working on soon is a "pause-on-reboot"
feature which is a bit like noreboot, but will accept a passphrase and
continue with the reboot, so you can 'pause', see the dying panic then
reboot without needing to invoke the server power management.
(This may have been as a result of being given details for the wrong
server power management interface IP address when debugging a server in
a data center 7 time zones away, and getting rather confused when it
didn't make any attempt to reboot. I imagine someone else might get
equally irate if their server was rebooting without warning.)
~Andrew
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel
--
Andrew Cooper - Dom0 Kernel Engineer, Citrix XenServer
T: +44 (0)1223 225 900, http://www.citrix.com
next prev parent reply other threads:[~2012-09-07 13:28 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-09-07 13:05 [PATCH] kexec/noreboot: Don't kexec_crash() if noreboot has been requested Andrew Cooper
2012-09-07 13:16 ` Jan Beulich
2012-09-07 13:28 ` Andrew Cooper [this message]
2012-09-07 15:03 ` Keir Fraser
2012-09-07 15:42 ` Andrew Cooper
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=5049F67E.3080709@citrix.com \
--to=andrew.cooper3@citrix.com \
--cc=JBeulich@suse.com \
--cc=keir@xen.org \
--cc=xen-devel@lists.xen.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 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.