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

RE: [Xen-ia64-devel] Re: [Xen-devel] PATCH: split smpboot.c and createcp

To: "Tristan Gingold" <Tristan.Gingold@xxxxxxxx>, "Keir Fraser" <Keir.Fraser@xxxxxxxxxxxx>
Subject: RE: [Xen-ia64-devel] Re: [Xen-devel] PATCH: split smpboot.c and createcpuhotplug.c
From: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
Date: Tue, 23 May 2006 17:08:50 +0800
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx, xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Tue, 23 May 2006 02:09:24 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcZ+OkLEuDbp6sSuTVGTE4OsBmc6lwACsT3w
Thread-topic: [Xen-ia64-devel] Re: [Xen-devel] PATCH: split smpboot.c and createcpuhotplug.c
>From: Tristan Gingold
>Sent: 2006年5月23日 15:30
>
>Le Lundi 22 Mai 2006 17:44, Keir Fraser a écrit :
>> On 22 May 2006, at 08:57, Tristan Gingold wrote:
>> > this patch creates a new file: cpuhotplug.c.  The content is the
>xenbus
>> > handler part of smpboot.c.  The purpose is to be able to share this
>> > part with
>> > other architectures.
>> >
>> > Tested by compiling on i386.
>>
>> This raises the obvious question: why don't you use the rest of our
>> smpboot.c to implement SMP bringup? It's all generic except the bit
>> that sets up VCPU initial context.
>I can see three reasons:
>
>* Only a 10 lines change (IPI handling) was required in the linux kernel to
>run SMP on Xen.  All the other changes were in Xen.
>
>* xen/core/smpboot.c is not a drop in replacement for
>arch/ia64/kernel/smpboot.c  Surgery will be required.
>
>* If we used xen/core/smpboot.c, we'd lose transparent virtualization.
>To
>re-enable it we had to do surgery again.
>
>Tristan.

Can't is_running_on_xen help you with transparent virtualization? Checks 
on several major entry paths should be enough... 

The 2nd point seems more reasonable to me if logic does differ a lot.

Thanks,
Kevin

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

<Prev in Thread] Current Thread [Next in Thread>
  • RE: [Xen-ia64-devel] Re: [Xen-devel] PATCH: split smpboot.c and createcpuhotplug.c, Tian, Kevin <=