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-devel] PVops domain 0 crash on NUMA system only Node==1 present

To: Jeremy Fitzhardinge <jeremy@xxxxxxxx>
Subject: Re: [Xen-devel] PVops domain 0 crash on NUMA system only Node==1 present (Was: Re: Bug#603632: linux-image-2.6.32-5-xen-amd64: Linux kernel 2.6.32/xen/amd64 booting fine on bare metal, but not as dom0 with Xen 4.0.1 (Dell R410))
From: Ian Campbell <Ian.Campbell@xxxxxxxxxx>
Date: Fri, 3 Dec 2010 09:49:24 +0000
Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, Keir Fraser <keir@xxxxxxx>, Vincent CARON <zerodeux@xxxxxxxxxxxx>, Vincent Caron <vcaron@xxxxxxxxxxxxx>, Cris Daniluk <cris.daniluk@xxxxxxxxx>, "603632@xxxxxxxxxxxxxxx" <603632@xxxxxxxxxxxxxxx>
Delivery-date: Fri, 03 Dec 2010 01:50:25 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <1291366377.5514.15046.camel@xxxxxxxxxxxxxxxxxxxxxx>
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>
Organization: Citrix Systems, Inc.
References: <20101115233253.11935.35707.reportbug@zerohal> <1290513067.31507.7699.camel@xxxxxxxxxxxxxxxxxxxxxx> <1290538327.9844.26.camel@xxxxxxxxxxxxxxxxxxxxx> <1290550323.9844.85.camel@xxxxxxxxxxxxxxxxxxxxx> <1290550726.9844.90.camel@xxxxxxxxxxxxxxxxxxxxx> <1291333639.25131.48.camel@zerohal> <4CF83606.6050904@xxxxxxxx> <1291366377.5514.15046.camel@xxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Fri, 2010-12-03 at 08:52 +0000, Ian Campbell wrote:
> 
> > IanC: it looks like passing a node id of "-1" is the correct way to
> > say "I don't care". 
> 
> I thought so too but convinced myself from staring at the code that it
> wouldn't work in this case -- I'll double check before I resubmit. 

I was mislead because node gets placed verbatim into desc->node which at
first glance seemed invalid but on second look it seems to be fine,
everything which reads that value is prepared to accept the -1. There is
precedent for using -1 in hpet_assign_irq, although most other places
seem to use numa_node_id().

I've tested with -1 and it seems ok. I see (too late) that you've
already made the change so I guess you knew this ;-)

Are you going to put this change into next-2.6.32 as well as 37?

Ian.


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

<Prev in Thread] Current Thread [Next in Thread>