[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/1] Add -incoming help text
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH 1/1] Add -incoming help text |
Date: |
Mon, 1 Jun 2015 10:29:32 +0100 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
* Markus Armbruster (address@hidden) wrote:
> Michael Tokarev <address@hidden> writes:
>
> > 29.05.2015 21:52, Dr. David Alan Gilbert (git) wrote:
> >> From: "Dr. David Alan Gilbert" <address@hidden>
> >>
> >> The help/man text for
> >>
> >> -incoming defer
> >>
> >> didn't make it through the merge of the code that implemented it.
> >>
> >> Signed-off-by: Dr. David Alan Gilbert <address@hidden>
> >> ---
> >> qemu-options.hx | 6 +++++-
> >> 1 file changed, 5 insertions(+), 1 deletion(-)
> >>
> >> diff --git a/qemu-options.hx b/qemu-options.hx
> >> index dad49cf..3085412 100644
> >> --- a/qemu-options.hx
> >> +++ b/qemu-options.hx
> >> @@ -3238,7 +3238,9 @@ DEF("incoming", HAS_ARG, QEMU_OPTION_incoming, \
> >> "-incoming fd:fd\n" \
> >> "-incoming exec:cmdline\n" \
> >> " accept incoming migration on given file
> >> descriptor\n" \
> >> - " or from given external command\n",
> >> + " or from given external command\n" \
> >> + "-incoming defer\n" \
> >> + " wait for the URI to be specified via
> >> migrate_incoming\n",
> >> QEMU_ARCH_ALL)
> >> STEXI
> >> @item -incoming tcp:address@hidden:@var{port}[,address@hidden,ipv4][,ipv6]
> >> @@ -3254,6 +3256,8 @@ Accept incoming migration from a given
> >> filedescriptor.
> >>
> >> @item -incoming exec:@var{cmdline}
> >> Accept incoming migration as an output from specified external command.
> >> address@hidden -incoming defer
> >> +Wait for the URI to be specified via migrate_incoming.
> >
> > I wonder what's the reason of mentioning it twice with
> > such a short and _identical_ text each time...
>
> Because that's the way our documentation works.
>
> The first hunk patches the fourth argument of DEF(), which goes into
> -help.
>
> The second hunk patches STEXI..ETEXI, which goes via qemu-options.texi
> into qemu.1 and qemu-doc.{dvi,info,html,pdf}.
>
> > Maybe
> > it's a good idea to reword the second message a bit.
> > Because actually it is not clear what does the whole
> > thing mean, what qemu will do when starting? Will it
> > accept other commands?
>
> You're right in that .texi allows more verbose explanations than -help.
>
> David's text looks okay to me as is. Perhaps your questions inspire
> further improvements.
How about changing the second part to.
address@hidden -incoming defer
+Wait for the URI to be specified via migrate_incoming. The monitor can
+be used to change settings (such as migration parameters) prior to issuing
+the migrate_incoming to allow the migration to begin.
Dave
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK