|   | 
      | 
  
  
      | 
      | 
  
 
     | 
    | 
  
  
     | 
    | 
  
  
    |   | 
      | 
  
  
    | 
         
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
 
 |   
 
 | 
    | 
  
  
    |   | 
    |