From mboxrd@z Thu Jan 1 00:00:00 1970 From: Frank Van Der Linden Subject: Re: [PATCH] Move smp_num_siblings to the per-cpu data Date: Wed, 21 Jan 2009 09:18:15 -0700 Message-ID: <49774AC7.9070300@Sun.COM> References: Mime-Version: 1.0 Content-Type: text/plain; format=flowed; charset=ISO-8859-1 Content-Transfer-Encoding: 7BIT Return-path: In-reply-to: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Keir Fraser Cc: xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org Keir Fraser wrote: > On 20/01/2009 23:36, "Frank van der Linden" > wrote: > > >> Make the num_siblings CPU parameter per-CPU. >> >> While it is unlikely that a system has a different number of siblings >> for different physical CPUs, make this parameter per-CPU for >> consistency, and deal with the (so far theoretical) case that this is >> different per CPU package. >> >> Signed-off-by: Frank van der Linden >> > > It's a pretty odd thing to decide to knock together one day. Do you have an > underlying near-term motive for adding this flexibility? > > -- Keir > > > I originally did this as part of an extension to the MCE support hypercall. We needed some detailed physical CPU information, plus some MSR info. When implementing this, and extracting infro from cpu_data[], I noticed the smp_num_siblings inconsistency, so I changed it. The other changes will be submitted as a separate patch, after they've been discussed with AMD & Intel. - Frank