[PATCH] add phytium FT-2000+ and Tengyun-S2500 support on arm architecture.

yangshaojun19@163.com posted 1 patch 3 years, 5 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/1605661612-22980-2-git-send-email-yangshaojun19@163.com
There is a newer version of this series
src/cpu_map/arm_Phytium.xml | 10 ++++++++++
src/cpu_map/arm_vendors.xml |  1 +
src/cpu_map/index.xml       |  3 +++
3 files changed, 14 insertions(+)
create mode 100644 src/cpu_map/arm_Phytium.xml
[PATCH] add phytium FT-2000+ and Tengyun-S2500 support on arm architecture.
Posted by yangshaojun19@163.com 3 years, 5 months ago
From: Shaojun Yang <yangshaojun@phytium.com.cn>

Signed-off-by: Shaojun Yang <yangshaojun@phytium.com.cn>
---
 src/cpu_map/arm_Phytium.xml | 10 ++++++++++
 src/cpu_map/arm_vendors.xml |  1 +
 src/cpu_map/index.xml       |  3 +++
 3 files changed, 14 insertions(+)
 create mode 100644 src/cpu_map/arm_Phytium.xml

diff --git a/src/cpu_map/arm_Phytium.xml b/src/cpu_map/arm_Phytium.xml
new file mode 100644
index 0000000..f8b4852
--- /dev/null
+++ b/src/cpu_map/arm_Phytium.xml
@@ -0,0 +1,10 @@
+<cpus>
+  <model name='FT-2000+'>
+    <vendor name='Phytium'/>
+    <pvr value='0x662'/>
+  </model>
+  <model name='Tengyun-S2500'>
+    <vendor name='Phytium'/>
+    <pvr value='0x663'/>
+  </model>
+</cpus>
diff --git a/src/cpu_map/arm_vendors.xml b/src/cpu_map/arm_vendors.xml
index ff799ef..4465463 100644
--- a/src/cpu_map/arm_vendors.xml
+++ b/src/cpu_map/arm_vendors.xml
@@ -11,4 +11,5 @@
   <vendor name='Qualcomm' value='0x51'/>
   <vendor name='Marvell' value='0x56'/>
   <vendor name='Intel' value='0x69'/>
+  <vendor name='Phytium' value='0x70'/>
 </cpus>
diff --git a/src/cpu_map/index.xml b/src/cpu_map/index.xml
index 08f052e..e2c454b 100644
--- a/src/cpu_map/index.xml
+++ b/src/cpu_map/index.xml
@@ -103,5 +103,8 @@
 
     <!-- Hisilicon-based CPU models -->
     <include filename='arm_Kunpeng-920.xml'/>
+
+    <!-- Phytium-based CPU models -->
+    <include filename='arm_Phytium.xml'/>
   </arch>
 </cpus>
-- 
2.7.4


Re: [PATCH] add phytium FT-2000+ and Tengyun-S2500 support on arm architecture.
Posted by Han Han 3 years, 5 months ago
Just a reminder: remember to mention that in the NEWS after the patch is
merged.

On Wed, Nov 18, 2020 at 9:06 AM <yangshaojun19@163.com> wrote:

> From: Shaojun Yang <yangshaojun@phytium.com.cn>
>
> Signed-off-by: Shaojun Yang <yangshaojun@phytium.com.cn>
> ---
>  src/cpu_map/arm_Phytium.xml | 10 ++++++++++
>  src/cpu_map/arm_vendors.xml |  1 +
>  src/cpu_map/index.xml       |  3 +++
>  3 files changed, 14 insertions(+)
>  create mode 100644 src/cpu_map/arm_Phytium.xml
>
> diff --git a/src/cpu_map/arm_Phytium.xml b/src/cpu_map/arm_Phytium.xml
> new file mode 100644
> index 0000000..f8b4852
> --- /dev/null
> +++ b/src/cpu_map/arm_Phytium.xml
> @@ -0,0 +1,10 @@
> +<cpus>
> +  <model name='FT-2000+'>
> +    <vendor name='Phytium'/>
> +    <pvr value='0x662'/>
> +  </model>
> +  <model name='Tengyun-S2500'>
> +    <vendor name='Phytium'/>
> +    <pvr value='0x663'/>
> +  </model>
> +</cpus>
> diff --git a/src/cpu_map/arm_vendors.xml b/src/cpu_map/arm_vendors.xml
> index ff799ef..4465463 100644
> --- a/src/cpu_map/arm_vendors.xml
> +++ b/src/cpu_map/arm_vendors.xml
> @@ -11,4 +11,5 @@
>    <vendor name='Qualcomm' value='0x51'/>
>    <vendor name='Marvell' value='0x56'/>
>    <vendor name='Intel' value='0x69'/>
> +  <vendor name='Phytium' value='0x70'/>
>  </cpus>
> diff --git a/src/cpu_map/index.xml b/src/cpu_map/index.xml
> index 08f052e..e2c454b 100644
> --- a/src/cpu_map/index.xml
> +++ b/src/cpu_map/index.xml
> @@ -103,5 +103,8 @@
>
>      <!-- Hisilicon-based CPU models -->
>      <include filename='arm_Kunpeng-920.xml'/>
> +
> +    <!-- Phytium-based CPU models -->
> +    <include filename='arm_Phytium.xml'/>
>    </arch>
>  </cpus>
> --
> 2.7.4
>
>
>
Re: [PATCH] add phytium FT-2000+ and Tengyun-S2500 support on arm architecture.
Posted by Andrea Bolognani 3 years, 5 months ago
On Wed, 2020-11-18 at 09:06 +0800, yangshaojun19@163.com wrote:
> From: Shaojun Yang <yangshaojun@phytium.com.cn>
> 
> Signed-off-by: Shaojun Yang <yangshaojun@phytium.com.cn>
> ---
>  src/cpu_map/arm_Phytium.xml | 10 ++++++++++
>  src/cpu_map/arm_vendors.xml |  1 +
>  src/cpu_map/index.xml       |  3 +++
>  3 files changed, 14 insertions(+)
>  create mode 100644 src/cpu_map/arm_Phytium.xml

You only addressed a small part of my feedback. Please read

  https://www.redhat.com/archives/libvir-list/2020-November/msg00946.html

again and address *all* of the points I brought up.

-- 
Andrea Bolognani / Red Hat / Virtualization