[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] x86: remove shadow dom0 command line option



On 24/03/17 18:07, Roger Pau Monné wrote:
> On Fri, Mar 24, 2017 at 05:51:41PM +0000, Wei Liu wrote:
>> It has been broken for years and couldn't possibly be configured after
>> 4045953.
>>
>> Remove code and documentation.
>>
>> Signed-off-by: Wei Liu <wei.liu2@xxxxxxxxxx>
>> ---
>> Cc: Jan Beulich <jbeulich@xxxxxxxx>
>> Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
>> Cc: Tim Deegan <tim@xxxxxxx>
>> Cc: George Dunlap <george.dunlap@xxxxxxxxxxxxx>
>> ---
>>  docs/misc/xen-command-line.markdown | 13 +------------
>>  xen/arch/x86/dom0_build.c           | 10 +---------
>>  xen/arch/x86/pv/dom0_build.c        |  5 -----
>>  xen/arch/x86/setup.c                |  3 +--
>>  4 files changed, 3 insertions(+), 28 deletions(-)
>>
>> diff --git a/docs/misc/xen-command-line.markdown 
>> b/docs/misc/xen-command-line.markdown
>> index bdbdb8a53b..f86b1bb7d2 100644
>> --- a/docs/misc/xen-command-line.markdown
>> +++ b/docs/misc/xen-command-line.markdown
>> @@ -652,11 +652,6 @@ restrictions set up here. Note that the values to be 
>> specified here are
>>  ACPI PXM ones, not Xen internal node numbers. `relaxed` sets up vCPU
>>  affinities to prefer but be not limited to the specified node(s).
>>  
>> -### dom0\_shadow
>> -> `= <boolean>`
>> -
>> -This option is deprecated, please use `dom0=shadow` instead.
>> -
>>  ### dom0\_vcpus\_pin
>>  > `= <boolean>`
>>  
>> @@ -665,7 +660,7 @@ This option is deprecated, please use `dom0=shadow` 
>> instead.
>>  Pin dom0 vcpus to their respective pcpus
>>  
>>  ### dom0
>> -> `= List of [ pvh | shadow ]`
>> +> `= List of [ pvh ]`
> 
> The shadow option here needs to be kept, in case someone wants to create a
> PVHv2 Dom0 with shadow instead of hap.

You can't change the meaning of "dom0=shadow" from "Make a PV dom0
running in autotranslate mode" to "Make a PVH dom0 running in shadow
mode rather than HAP". :-)

 -George


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.