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. 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
> -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
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
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
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