[libvirt] [PATCH] Do not drop kernel cmdline for xen pv domains
Daniel P. Berrange
berrange at redhat.com
Thu Jul 7 09:31:31 UTC 2011
On Wed, Jul 06, 2011 at 07:36:26PM -0600, Jim Fehlig wrote:
> Kernel cmdline args can be passed to xen pv domains even when a
> bootloader is specified. The current config-to-sxpr mapping
> ignores cmdline when bootloader is present.
>
> Since the xend sub-driver is used with many xen toolstack versions,
> this patch takes conservative approach of adding an else block to
> existing !def->os.bootloader, and only appends sxpr if def->os.cmdline
> is non-NULL.
> ---
> src/xenxs/xen_sxpr.c | 6 ++++++
> 1 files changed, 6 insertions(+), 0 deletions(-)
>
> diff --git a/src/xenxs/xen_sxpr.c b/src/xenxs/xen_sxpr.c
> index 13ca015..bd770bc 100644
> --- a/src/xenxs/xen_sxpr.c
> +++ b/src/xenxs/xen_sxpr.c
> @@ -2261,6 +2261,12 @@ xenFormatSxpr(virConnectPtr conn,
> }
>
> virBufferAddLit(&buf, "))");
> + } else {
> + /* PV domains accept kernel cmdline args */
> + if (def->os.cmdline) {
> + virBufferEscapeSexpr(&buf, "(image (linux (args '%s')))",
> + def->os.cmdline);
> + }
> }
ACK, assuming the test cases still pass - I'm surprised this
doesn't require any testcase fixes
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
More information about the libvir-list
mailing list