19 messages in com.xensource.lists.xen-ia64-develRe: [Xen-ia64-devel] [Patch] Pass the...
FromSent OnAttachments
Akio Takebe18 Jul 2007 00:03.patch
Akio Takebe18 Jul 2007 01:48 
Xu, Anthony18 Jul 2007 06:25 
Akio Takebe18 Jul 2007 23:08 
Alex Williamson19 Jul 2007 16:20 
Alex Williamson19 Jul 2007 18:17 
Tristan Gingold19 Jul 2007 18:53 
Alex Williamson19 Jul 2007 19:05 
Akio Takebe20 Jul 2007 02:39 
Alex Williamson20 Jul 2007 13:58 
Alex Williamson20 Jul 2007 14:31 
tgin...@free.fr23 Jul 2007 01:19 
Akio Takebe23 Jul 2007 05:59.patch
Alex Williamson23 Jul 2007 08:39 
Akio Takebe24 Jul 2007 23:01.patch
Akio Takebe24 Jul 2007 23:43 
Isaku Yamahata25 Jul 2007 00:09 
Akio Takebe25 Jul 2007 00:45 
Isaku Yamahata25 Jul 2007 00:54 
Subject:Re: [Xen-ia64-devel] [Patch] Pass the bare LSAPIC ID to dom0
From:tgin...@free.fr (tgin@free.fr)
Date:07/23/2007 01:19:40 AM
List:com.xensource.lists.xen-ia64-devel

Quoting Akio Takebe <take@jp.fujitsu.com>:

Hi, Alex, Tristan, Yongkang and all

On Fri, 2007-07-20 at 03:53 +0200, Tristan Gingold wrote:

On Thu, Jul 19, 2007 at 05:20:26PM -0600, Alex Williamson wrote:

On Wed, 2007-07-18 at 16:03 +0900, Akio Takebe wrote:

Hi, all

This patch fix a issue which dom0 cannot boot with dom0_max_vcpus. Currently LSAPIC IDs are create by xen, but ACPI SRAT table is the bare table. So on some boxes node_cpuid[].phys_id are different from cpu_physical_id()s, and we cannot boot dom0. I think xen should pass the bare LSAPIC ID to dom0.

Hi Akio,

I am not sure this patch is complete. Some code of Xen assumes vcpu_id == id (get_lid, IPI). I think you'd better to patch the SRAT (if it is possible).

That may be the way to go, it should be possible. BTW, now that I look at it again, lsapic->id == 0 really looks like an implementation dependent check. Thanks,

I'm sorry, and I'll debug more. I'll check vcpu_id and SRAT code. Tristan's idea may be good, I'll try it.

Hi,

after more thoughts, I think trying NUMA in dom0 is bound to fail or to be useless. Memory mapping is virtualized and vcpu can move...

Tristan.