[libvirt] [PATCH V2] Do not drop kernel cmdline for xen pv domains
Jim Fehlig
jfehlig at novell.com
Mon Jul 11 15:16:11 UTC 2011
Eric Blake wrote:
> On 07/07/2011 05:37 PM, Jim Fehlig wrote:
>
>> 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.
>>>
>>> V2: Fix existing testcase broken by this patch and add new testcases
>>>
>>>
>> Hmm, now domainschematest is failing
>>
>> on these two xml files. If I'm reading domain.rng correctly, kernel must
>> be specified?? Can it be optional like initrd and cmdline?
>>
>
> Reading domain_conf.c agrees with that interpretation. I would be fine
> with you squashing this in:
>
> diff --git i/docs/schemas/domain.rng w/docs/schemas/domain.rng
> index c01801e..b659da9 100644
> --- i/docs/schemas/domain.rng
> +++ w/docs/schemas/domain.rng
> @@ -553,9 +553,11 @@
> </define>
> <define name="osbootkernel">
> <interleave>
> - <element name="kernel">
> - <ref name="absFilePath"/>
> - </element>
> + <optional>
> + <element name="kernel">
> + <ref name="absFilePath"/>
> + </element>
> + </optional>
> <optional>
> <element name="initrd">
> <ref name="absFilePath"/>
>
> ACK, once you fix that and address Matthias' comment about spacing in
> the .xml file:
>
Squashed in your change to domain.rng, removed spaces noted by Matthias,
and pushed.
Regards,
Jim
More information about the libvir-list
mailing list