diff mbox

docs: Clarify scope of reboot= and noreboot Xen command line options

Message ID 1482418973-26941-1-git-send-email-ian.jackson@eu.citrix.com (mailing list archive)
State New, archived
Headers show

Commit Message

Ian Jackson Dec. 22, 2016, 3:02 p.m. UTC
Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
CC: Jan Beulich <JBeulich@suse.com>
---
 docs/misc/xen-command-line.markdown | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Comments

Jan Beulich Dec. 22, 2016, 3:30 p.m. UTC | #1
>>> On 22.12.16 at 16:02, <ian.jackson@eu.citrix.com> wrote:
> @@ -1356,7 +1357,9 @@ The following resources are available:
>  
>  > Default: `0`
>  
> -Specify the host reboot method.
> +Specify the host reboot method,
> +used when Xen crashes.
> +(This does not affect deliberate reboots initiated by dom0.)

This should be moved down to where `no` is being described, as
it affects only that sub-option. The reboot methods, otoh, affect
all kinds of reboots.

Jan
Ian Jackson Dec. 22, 2016, 6:59 p.m. UTC | #2
Jan Beulich writes ("Re: [PATCH] docs: Clarify scope of reboot= and noreboot Xen command line options"):
> On 22.12.16 at 16:02, <ian.jackson@eu.citrix.com> wrote:
> > -Specify the host reboot method.
> > +Specify the host reboot method,
> > +used when Xen crashes.
> > +(This does not affect deliberate reboots initiated by dom0.)
> 
> This should be moved down to where `no` is being described, as
> it affects only that sub-option. The reboot methods, otoh, affect
> all kinds of reboots.

So you might say
  reboot=triple reboot=efi reboot=no
and this would mean to do requested reboots with efi and crash reboots
not at all, with `reboot=triple' being completely ignored ?

This is ... a funny way for an option with a `=' to behave.
Perhaps we should deprecate this.

Ian.
Jan Beulich Dec. 27, 2016, 3:36 p.m. UTC | #3
>>> Ian Jackson <ian.jackson@eu.citrix.com> 12/22/16 8:00 PM >>>
>Jan Beulich writes ("Re: [PATCH] docs: Clarify scope of reboot= and noreboot Xen command line options"):
>> On 22.12.16 at 16:02, <ian.jackson@eu.citrix.com> wrote:
>> > -Specify the host reboot method.
>> > +Specify the host reboot method,
>> > +used when Xen crashes.
>> > +(This does not affect deliberate reboots initiated by dom0.)
>> 
>> This should be moved down to where `no` is being described, as
>> it affects only that sub-option. The reboot methods, otoh, affect
>> all kinds of reboots.
>
>So you might say
>reboot=triple reboot=efi reboot=no
>and this would mean to do requested reboots with efi and crash reboots
>not at all, with `reboot=triple' being completely ignored ?

Yes.

>This is ... a funny way for an option with a `=' to behave.
>Perhaps we should deprecate this.

I don't think we should. This override behavior has a use - when something is preset
in a config file, and you want to override it on the command line (which can be easily
the case namely under EFI). And this behavior is the same for most if not all other
command line options.

Jan
diff mbox

Patch

diff --git a/docs/misc/xen-command-line.markdown b/docs/misc/xen-command-line.markdown
index 0138978..68c81e6 100644
--- a/docs/misc/xen-command-line.markdown
+++ b/docs/misc/xen-command-line.markdown
@@ -1260,6 +1260,7 @@  the **vga** option, which relies on real mode to set the video mode.
 Do not automatically reboot after an error.  This is useful for
 catching debug output.  Defaults to automatically reboot after 5
 seconds.
+This is equivalent to `reboot=no`
 
 ### nosmp
 > `= <boolean>`
@@ -1356,7 +1357,9 @@  The following resources are available:
 
 > Default: `0`
 
-Specify the host reboot method.
+Specify the host reboot method,
+used when Xen crashes.
+(This does not affect deliberate reboots initiated by dom0.)
 
 `warm` instructs Xen to not set the cold reboot flag.