Re: [Xen-devel] [PATCH v3 04/17] SUPPORT.md: Add core ARM features

[George Dunlap <george.dunlap@xxxxxxxxxx>]

On 11/23/2017 11:11 AM, Julien Grall wrote:
> Hi George,
> 
> On 22/11/17 19:20, George Dunlap wrote:
>> Hardware support and guest type.
>>
>> Signed-off-by: George Dunlap <george.dunlap@xxxxxxxxxx>
>> ---
>> Changes since v2:
>> - Moved SMMUv* into generic IOMMU section
>>
>> CC: Ian Jackson <ian.jackson@xxxxxxxxxx>
>> CC: Wei Liu <wei.liu2@xxxxxxxxxx>
>> CC: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
>> CC: Jan Beulich <jbeulich@xxxxxxxx>
>> CC: Stefano Stabellini <sstabellini@xxxxxxxxxx>
>> CC: Konrad Wilk <konrad.wilk@xxxxxxxxxx>
>> CC: Tim Deegan <tim@xxxxxxx>
>> CC: Julien Grall <julien.grall@xxxxxxx>
>> ---
>>   SUPPORT.md | 25 ++++++++++++++++++++++++-
>>   1 file changed, 24 insertions(+), 1 deletion(-)
>>
>> diff --git a/SUPPORT.md b/SUPPORT.md
>> index a4cf2da50d..5945ab4926 100644
>> --- a/SUPPORT.md
>> +++ b/SUPPORT.md
>> @@ -22,6 +22,14 @@ for the definitions of the support status levels etc.
>>         Status: Supported
>>   +### ARM v7 + Virtualization Extensions
>> +
>> +    Status: Supported
>> +
>> +### ARM v8
>> +
>> +    Status: Supported
>> +
>>   ## Host hardware support
>>     ### Physical CPU Hotplug
>> @@ -35,6 +43,7 @@ for the definitions of the support status levels etc.
>>   ### Host ACPI (via Domain 0)
>>         Status, x86 PV: Supported
>> +    Status, ARM: Experimental
>>     ### x86/Intel Platform QoS Technologies
>>   @@ -44,6 +53,14 @@ for the definitions of the support status levels
>> etc.
>>         Status, AMD IOMMU: Supported
>>       Status, Intel VT-d: Supported
>> +    Status, ARM SMMUv1: Supported
>> +    Status, ARM SMMUv2: Supported
>> +
>> +### ARM/GICv3 ITS
>> +
>> +    Status: Experimental
>> +
>> +Extension to the GICv3 interrupt controller to support MSI.
>>     ## Guest Type
>>   @@ -67,12 +84,18 @@ Requires hardware virtualisation support (Intel
>> VMX / AMD SVM)
>>         Status: Supported
>>   -PVH is a next-generation paravirtualized mode
>> +PVH is a next-generation paravirtualized mode
> 
> I am not sure to see the difference between the 2 lines. Is it intented?

The difference is the whitespace at the end -- this change should have
been made in the previous patch instead.

> The rest looks good.

Thanks. With that moved, can it have your Ack?

 -George


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel