|  |  | 
  
    |  |  | 
 
  |   |  | 
  
    |  |  | 
  
    |  |  | 
  
    |   xen-devel
Re: [Xen-devel] RE: [PATCH] irq: Exclude percpu IRQs from being	fixed up 
| To: | "Li, Xin" <xin.li@xxxxxxxxx> |  
| Subject: | Re: [Xen-devel] RE: [PATCH] irq: Exclude percpu IRQs from being	fixed up |  
| From: | Ian Campbell <Ian.Campbell@xxxxxxxxxx> |  
| Date: | Tue, 3 May 2011 12:55:36 +0100 |  
| Cc: | Jeremy Fitzhardinge <jeremy@xxxxxxxx>,	xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, "Dong,	Eddie" <eddie.dong@xxxxxxxxx>, "'Zhang, Fengzhe'"	<IMCEAEX-_O=INTEL_OU=EXCHANGE+20ADMINISTRATIVE+20GROUP+20+28FYDIBOHF23SPDLT+29_CN=RECIPIENTS_CN=FZHAN17@xxxxxxxxx>,	Jan Beulich <JBeulich@xxxxxxxxxx> |  
| Delivery-date: | Tue, 03 May 2011 04:59:49 -0700 |  
| Envelope-to: | www-data@xxxxxxxxxxxxxxxxxxx |  
| In-reply-to: | <FC2FB65B4D919844ADE4BE3C2BB739AD5A2035D1@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> |  
| 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: | <1A42CE6F5F474C41B63392A5F80372B2335E978D@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>	<4D5BF2FE02000078000322EB@xxxxxxxxxxxxxxxxxx>	<4D5C68AF.3030807@xxxxxxxxx>	<4D5CE1DE0200007800032557@xxxxxxxxxxxxxxxxxx>	<4D5CDB77.2010607@xxxxxxxxx>	<FC2FB65B4D919844ADE4BE3C2BB739AD5A2035D1@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> |  
| Sender: | xen-devel-bounces@xxxxxxxxxxxxxxxxxxx |  
| On Thu, 2011-04-28 at 12:05 +0100, Li, Xin wrote:
> > >>>
> > >> IRQ_PER_CPU switch is not found in current Kconfig. I'm not sure if this
> > >
> > > kernel/irq/Kconfig (introduced as a generic option in 2.6.38-rc2). In
> > > prior kernel you'd have to add a respective Kconfig item in
> > > drivers/xen/Kconfig.
> > >
> > >> feature is going to be brought back in the short term. I remove the
> > >> ifdef to set IRQ_PER_CPU flag in desc by default but still leave the IRQ
> > >> handling logic unchanged. This is a temporary solution to fix system
> > >> crash on poweroff. And this is the fix with minimum impact among the
> > >> several solutions we tried.
> > >
> > > But it's more a hack than a fix. And making per-CPU IRQs properly
> > 
> > Yes, that's true. Switching on IRQ_PER_CPU can make this patch neat. But
> > that isn't my call.
> 
> Jeremy, would you consider the change to fix this issue?
There was no response to the several issues/questions I raised in
<1297933765.16356.1344.camel@xxxxxxxxxxxxxxxxxxxxxx>.
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
 | 
 
| <Prev in Thread] | Current Thread | [Next in Thread> |  | 
Re: [Xen-devel] RE: [PATCH] irq: Exclude percpu IRQs from being	fixed up,
Ian Campbell <=
 |  |  | 
  
    |  |  |