WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

[Xen-devel] Re: [PATCH][RFC][0/2] x86 SMP: enables core-pair topology d

To: "Wei Huang" <wei.huang2@xxxxxxx>
Subject: [Xen-devel] Re: [PATCH][RFC][0/2] x86 SMP: enables core-pair topology detection for AMD CPUs
From: "Jan Beulich" <JBeulich@xxxxxxxxxx>
Date: Tue, 28 Jun 2011 08:38:59 +0100
Cc: George Dunlap <George.Dunlap@xxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Keir Fraser <keir@xxxxxxx>
Delivery-date: Tue, 28 Jun 2011 00:40:22 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4E08BD2D.9020403@xxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <4E03C386.9030009@xxxxxxx> <4E0862520200007800049BDB@xxxxxxxxxxxxxxxxxxxx> <4E08BD2D.9020403@xxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
>>> On 27.06.11 at 19:26, Wei Huang <wei.huang2@xxxxxxx> wrote:
> Thanks, Jan and George. I attach the latest version which fixes the 
> indentation issues.

Acked-by: Jan Beulich <jbeulich@xxxxxxxxxx>

> 
> -Wei
> 
> 
> On 06/27/2011 03:58 AM, Jan Beulich wrote:
>>>>> On 24.06.11 at 00:51, Wei Huang<wei.huang2@xxxxxxx>  wrote:
>>> Hi,
>>>
>>> The following two patches enable core-pair topology detection for AMD
>>> CPUs. The main idea is to record compute unit information into
>>> cpuinfo_x86 and use it when building sibling map. The sibling map will
>>> be used by Xen credit scheduler to coordinate VCPU scheduling in a
>>> fashion similar to SMT. This code is very similar to upstream Linux kernel.
>>>
>>> Regarding the performance improvement, I saw ~8% improvement running
>>> kernel compilation on Dom0 with this patch. Anrea Przywara ran a POVRay
>>> benchmark using this patch and he saw ~5% improvement. No perf
>>> degradation was seen so far.
>>>
>>> Please review.
>> Apart from some indentation issues these look fine to me.
>>
>> Jan
>>
>>> Thanks,
>>> -Wei
>>
>>
>>




_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel