Re: [libvirt] [PATCH 1/3] cpu_map.xml: add cmt feature to x86

2015-10-29 Thread Jiri Denemark
On Thu, Oct 29, 2015 at 14:02:28 +0800, Qiaowei Ren wrote:
> Some Intel processor families (e.g. the Intel Xeon processor E5 v3
> family) introduced CMT (Cache Monitoring Technology) to measure the
> usage of cache by applications running on the platform. This patch
> add it into x86 part of cpu_map.xml.

When sending a series of patches, please use --cover-letter to create a
0/n patch where you describe what the series is trying to achieve. As a
nice side effect , individual patches will be sent as replies to the
cover letter, which is much better than sending 2..n patches as replies
to the first one.

> Signed-off-by: Qiaowei Ren 
> ---
>  .gnulib | 2 +-
>  src/cpu/cpu_map.xml | 3 +++
>  2 files changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/.gnulib b/.gnulib
> index f39477d..106a386 16
> --- a/.gnulib
> +++ b/.gnulib
> @@ -1 +1 @@
> -Subproject commit f39477dba778e99392948dd3dd19ec0d46aee932
> +Subproject commit 106a3866d01f9dd57ab4f10dbeb0d5a8db73a9f7

As Peter said, this hunk should be removed.

> diff --git a/src/cpu/cpu_map.xml b/src/cpu/cpu_map.xml
> index b9e95cf..14ccbd8 100644
> --- a/src/cpu/cpu_map.xml
> +++ b/src/cpu/cpu_map.xml
> @@ -317,6 +317,9 @@
>  
>
>  
> +
> +  
> +
>  
>
>  

This looks like it makes sense, but it really doesn't. This patches
causes libvirt to report cmt feature on host CPUs which support it, but
what's the point since we are apparently not interested in exposing the
feature to guests? Not to mention that even if the host CPU supports
cmt, the host kernel does not have to supported so advertising the CPU
feature doesn't really say whether it's usable or not.

Jirka

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list


Re: [libvirt] [PATCH 1/3] cpu_map.xml: add cmt feature to x86

2015-10-29 Thread Ren, Qiaowei

> -Original Message-
> From: Peter Krempa [mailto:pkre...@redhat.com]
> Sent: Thursday, October 29, 2015 4:43 PM
> To: Ren, Qiaowei
> Cc: libvir-list@redhat.com; Feng, Shaohe
> Subject: Re: [libvirt] [PATCH 1/3] cpu_map.xml: add cmt feature to x86
> 
> On Thu, Oct 29, 2015 at 14:02:28 +0800, Qiaowei Ren wrote:
> > Some Intel processor families (e.g. the Intel Xeon processor E5 v3
> > family) introduced CMT (Cache Monitoring Technology) to measure the
> > usage of cache by applications running on the platform. This patch add
> > it into x86 part of cpu_map.xml.
> >
> > Signed-off-by: Qiaowei Ren 
> > ---
> >  .gnulib | 2 +-
> >  src/cpu/cpu_map.xml | 3 +++
> >  2 files changed, 4 insertions(+), 1 deletion(-)
> >
> > diff --git a/.gnulib b/.gnulib
> > index f39477d..106a386 16
> > --- a/.gnulib
> > +++ b/.gnulib
> > @@ -1 +1 @@
> > -Subproject commit f39477dba778e99392948dd3dd19ec0d46aee932
> > +Subproject commit 106a3866d01f9dd57ab4f10dbeb0d5a8db73a9f7
> 
> This hunk should not be here. Gnulib versions are changed separately.
> Also I doubt that it's necessary.
> 
Yes. This should be not necessary here.

Thanks,
Qiaowei


--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list


Re: [libvirt] [PATCH 1/3] cpu_map.xml: add cmt feature to x86

2015-10-29 Thread Peter Krempa
On Thu, Oct 29, 2015 at 14:02:28 +0800, Qiaowei Ren wrote:
> Some Intel processor families (e.g. the Intel Xeon processor E5 v3
> family) introduced CMT (Cache Monitoring Technology) to measure the
> usage of cache by applications running on the platform. This patch
> add it into x86 part of cpu_map.xml.
> 
> Signed-off-by: Qiaowei Ren 
> ---
>  .gnulib | 2 +-
>  src/cpu/cpu_map.xml | 3 +++
>  2 files changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/.gnulib b/.gnulib
> index f39477d..106a386 16
> --- a/.gnulib
> +++ b/.gnulib
> @@ -1 +1 @@
> -Subproject commit f39477dba778e99392948dd3dd19ec0d46aee932
> +Subproject commit 106a3866d01f9dd57ab4f10dbeb0d5a8db73a9f7

This hunk should not be here. Gnulib versions are changed separately.
Also I doubt that it's necessary.

Peter


signature.asc
Description: Digital signature
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH 1/3] cpu_map.xml: add cmt feature to x86

2015-10-28 Thread Qiaowei Ren
Some Intel processor families (e.g. the Intel Xeon processor E5 v3
family) introduced CMT (Cache Monitoring Technology) to measure the
usage of cache by applications running on the platform. This patch
add it into x86 part of cpu_map.xml.

Signed-off-by: Qiaowei Ren 
---
 .gnulib | 2 +-
 src/cpu/cpu_map.xml | 3 +++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/.gnulib b/.gnulib
index f39477d..106a386 16
--- a/.gnulib
+++ b/.gnulib
@@ -1 +1 @@
-Subproject commit f39477dba778e99392948dd3dd19ec0d46aee932
+Subproject commit 106a3866d01f9dd57ab4f10dbeb0d5a8db73a9f7
diff --git a/src/cpu/cpu_map.xml b/src/cpu/cpu_map.xml
index b9e95cf..14ccbd8 100644
--- a/src/cpu/cpu_map.xml
+++ b/src/cpu/cpu_map.xml
@@ -317,6 +317,9 @@
 
   
 
+
+  
+
 
   
 
-- 
1.9.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list


[libvirt] [PATCH 1/3] cpu_map.xml: add cmt feature to x86

2015-07-05 Thread Qiaowei Ren
Some Intel processor families (e.g. the Intel Xeon processor E5 v3
family) introduced CMT (Cache Monitoring Technology) to measure the
usage of cache by applications running on the platform. This patch
add it into x86 part of cpu_map.xml.

Signed-off-by: Qiaowei Ren 
---
 .gnulib | 2 +-
 src/cpu/cpu_map.xml | 3 +++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/.gnulib b/.gnulib
index f39477d..106a386 16
--- a/.gnulib
+++ b/.gnulib
@@ -1 +1 @@
-Subproject commit f39477dba778e99392948dd3dd19ec0d46aee932
+Subproject commit 106a3866d01f9dd57ab4f10dbeb0d5a8db73a9f7
diff --git a/src/cpu/cpu_map.xml b/src/cpu/cpu_map.xml
index b9e95cf..14ccbd8 100644
--- a/src/cpu/cpu_map.xml
+++ b/src/cpu/cpu_map.xml
@@ -317,6 +317,9 @@
 
   
 
+
+  
+
 
   
 
-- 
1.9.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list