On 15/12/15 07:22, Ian Campbell wrote:
+Daniel
On Fri, 2015-12-11 at 16:47 +0100, Juergen Gross wrote:
init-xenstore-domain takes only positional parameters today. Change
this to a more flexible parameter syntax allowing to specify additional
options or to omit some.
Today the supported usage is:
init-xenstore-domain <xenstore-kernel> <memory_mb> <flask-label>
[<ramdisk-file>]
Modify this to:
init-xenstore-domain --kernel <xenstore-kernel>
--memory <memory_mb>
[--flask <flask-label>]
[--ramdisk <ramdisk-file>]
The flask label is made optional in order to support xenstore domains
without the need of a flask enabled hypervisor.
Signed-off-by: Juergen Gross <jgr...@suse.com>
Looks fine to me
Acked-by: Ian Campbell <ian.campb...@citrix.com>
I'd like to see Daniel opinion of all the changes to init-xenstore-domain.c
in this series though.
This and the other command line changes in #7 look good, and make the
rather opaque invocation of the tool a bit nicer to read.
Acked-by: Daniel De Graaf <dgde...@tycho.nsa.gov>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
